I'm having trouble diagnosing an issue when using the Test Convert File function: when I select a format that I know will work, I get the following error:

An error occurred.
/ffmpeg_wrapper/js
(no information available).

Recent log entries doesn't show any more detail.

However, the converted file does exist in /tmp/ and works fine, so ffmpeg presumably returned an OK status code. In addition, if I disable JavaScript, I get the download link for the file.

Is there any reason why there's a JavaScript wrapper on the conversion process? Seems like it adds a layer of needless complexity, and at least in this case, breaks an otherwise successful conversion.

Comments

arthurf’s picture

This came from a function which allowed default settings to be defined and applied when a output file type was chosen. I've removed the functionality.

mark trapp’s picture

Status: Active » Fixed

Sweet, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

design10’s picture

error still exists

arthurf’s picture

@design10 can you confirm that you're using a recent dev version?

design10’s picture

StatusFileSize
new28.1 KB

I did not use the cvs to pull it from the server but downloaded the dev file from the project page. BTW thank you for the hard work you are putting into this module. It is really amazing. I was able to get the 1.0 beta series working and am really impressed with the work you are putting in to abstract the UI.

On a seperate issue "ffmpeg_wrapper_which_ffmpeg" function effectively overwrites the stored setting for ffmpeg_wrapper. Using the native system on a virtual server execution from php is often limited to the users home directory for obvious security reasons. To get around this I can simply copy the executable to a folder in my home directory and execute and resetting the path which returns true from this function to the file in my home directory. While this is not useful for contribution back to the development of ffmpeg_wrapper I thought it would be good to have some way to override the function for a virtual server configuration.

I will try to dl the latest

below is the module i loaded.

design10’s picture

I uploaded the most recent dev version from cvs and still get the error. Am I missing a file?

wbarber69’s picture

I'm having the same issue using the latest dev (nov 28)

arthurf’s picture

Status: Closed (fixed) » Active

Can you supply information on the settings and files that you're using? I haven't been able to duplicate this in my testing so there must be some difference in the kind of data that we're using.

sreyas’s picture

I am having same situation. Trying to use the test convert file option.
Input file format : wmv
Output format : flv
audio codec : libmp3lame /64K/44.1khz

Error getting as an alert the content of the alert is an html

An error occurred. /ffmpeg_wrapper/js 
Fatal error: Call to undefined function checkmarkup() in /home/cpuser/public_html/dev/sites/all/ modules/ffmpeg_wrapper/ffmpeg_wrapper_ui/ffmpeg_wrapper_ui.module on line 408

The video is getting converted and created in the /tmp file and this video do work. Have exactly the same size when we convert via command prompt.
Let me know if you need any further information .

darrick’s picture

Status: Active » Needs review
StatusFileSize
new571 bytes

Should be calling drupal_json($var = NULL) instead of drupal_to_js($var). The former prints the headers.

darrick’s picture

StatusFileSize
new927 bytes

Patch in #11 actually fixes the issue where the return from ffmpeg_wrapper/js was showing up in an alert box.

This patch has that but also the fix for #10. There was typo. checkmarkup should be check_markup.

jantoine’s picture

Status: Needs review » Needs work

After applying the patch from #12, it seems to correctly fix the checkmarkup issue in #10, but I am still seeing the error message described in the original issue, so the fix in #11 doesn't seem to be fixing that for me?

Cheers,

Antoine

arthurf’s picture

Fix in #11 is committed