By default the Video Manipulation options is set to "No thumbnail - use video image".
When you set it to "Use mplayer to create thumbnails" it saves the option but then complains that mplayer is not found.
When you try to set it back to "No thumbnail - use video image" it says "Path to mplayer is not correct — video thumbnails will not be generated" and "The settings have not been saved because of the errors.", so you cannot turn off the option.
Solution:
Do not check "Path to mplayer" when the video thumbnail options is No thumbnail.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | acidfree.patch | 564 bytes | jax |
Comments
Comment #1
jax commentedThe patch that I attached to this post fixes the issue. But since it accesses $_POST directly I do not think it's the drupaly way to do it. Anyway, I hope it inspires someone to improve it.
The issue is that the _settings hook uses the current form settings (with variable_get) to validate in stead of the submitted values.
Comment #2
benthere commentedreally breaks acidfree if you don't have mplayer and try setting it, assuming it will work the same way as the -tran fields and fail gracefully.
Comment #3
vhmauery commentedfixed in CVS. it should get repackaged in 10 hours or so...
Comment #4
vhmauery commentedComment #5
paul-at-murphymaphia.com commentedThis patch keeps the error from happening but the 'Path to mplayer' setting is not stored or updated when the form is submitted. I had to enter the path in the 'variables' table manually to get mplayer working.
Comment #6
vhmauery commentedokay -- now it is fixed in cvs -- it should have been '#default_value' instead of '#value'.
Comment #7
vhmauery commented