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
|
|
|
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...
|
|
|
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!
|
|
You need to be logged in to post messages in the forums. New users may register here.