Some comments

You need to be logged in to post messages in the forums. New users may register here.

Marcin Drozd

Member since:
09 January 2008

Posts: 2

Thursday 22 May 2008 2:41:53 am

Hi
Great ext :)

I have some comments about it:
1. no translation
There is
alt="Preview"

should be:
alt="{'Preview'|i18n( 'ezvideoflv/datatype' )}"

at linie 17 (file: ezvideoflv/design/standard/templates/content/datatype/edit/ezvideoflv.tpl

2. system via exec
My IPS disabled php functions: system, passthru, etc. but exec no (uff).
Maybe, it should be in ini? (what function do I want to use: system or exec)
file: ezvideoflv/datatypes/ezvideoflv/ezvideoflv.php
method: doConvert













































































































Up

Marcin Drozd

Member since:
09 January 2008

Posts: 2

Thursday 22 May 2008 2:56:34 am

Oh, I have a polish translation of this. If U want...
Up

Damien Pobel

Member since:
09 January 2008

Posts: 28

Tuesday 24 June 2008 2:45:56 pm

Hi Marcin,

Sorry for the delay, I'm quite busy...

1. you're right, that's a small bug
2. hum it seems quite specific to your setup. The ImageMagick image handler also use system() and I'm not for an ini settings has it can be really confusing.
3. the problem of the ffmpeg use instead of GD is that you can't choose the "middle" frame with [Preview]/Frame settings. But we can probably let the developer choose in the ini settings the command line he wants to use to generate the preview instead of GD.
4. You're right.

I'm also interesting in the polish translation !
Would you like a read/write access to the SVN ? Just register as a member of the project on http://projects.ez.no/index.php/ezvideoflv/team/members so that I can approve your membership.

Cheers!

Damien
Planet eZ Publish.fr : http://www.planet-ezpublish.fr
Certification : http://ez.no/certification/verify/308412
Publications about eZ Publish : http://pwet.fr/tags/keywords/weblog/ez_publish

Up

You need to be logged in to post messages in the forums. New users may register here.