When using the file upload on a content type when the form is first rendered there is a hidden field present for video_id.

After the upload and the ahah completes another hidden form element with the same id is added that contains the video_id value. Now the same form element exist twice and when the content is saved the value does not get stored.

I can make the video actually save if I use firebug to delete the original form element for video_id after the ahah file upload. The one that is still empty after the file upload and ahah call. I remember initially having this issue in version 1.8 onward and rolled back to 1.7 which works in this regard.

I'm using an nginx server if that makes a difference, maybe for the ahah.

CommentFileSizeAuthor
#3 1110074-video-upload-does-not-save-4324440-D6.patch607 bytesjethro

Comments

jeremyheslop’s picture

jeremyheslop commented
Assigned: Unassigned » jeremyheslop

Jethro,

Can you download the new version and let me know how things go? Thanks,

Jeremy

jethro’s picture

jethro commented
Version: 6.x-2.0 » 6.x-2.1

Thanks for your efforts on this.

The same issue is still there. The hidden field field_viddlervideo[0][video_id] appears twice on the creation and edit form after a file upload and only the one that was originally there is used. New uploads are not saved.

One thing that has improved is that previously when the url or vid id input was enabled, after a video upload these form inputs would appear twice. This is no longer the case.

Thanks,
Jethro

Looking in to this, it seems like the field added on line 1070 of viddler.module needs to be part of the area that is replaced with the ahah callback, so if it's visible or not, it will reflect the viddler id of whatever has just been uploaded. Maybe the wrapper (line 1032) could basically replace the whole field but update the value of video_id element.

jethro’s picture

jethro commented
Status: Active » Needs review
StatusFileSize
new 1110074-video-upload-does-not-save-4324440-D6.patch607 bytes

Here is a simple fix which include the viddler video id in the ahah_wrapper. This worked for me and hopefully doesn't break anything.

jeremyheslop’s picture

jeremyheslop commented

Thanks jethro. There were a couple stray div's that would cause issues with ahah. I went through the code and removed any that were not needed and only left the ones ahah needed for wrappers. I should have the code updated today. Thanks again,

Jeremy

eodonnell’s picture

eodonnell commented
Version: 6.x-2.1 » 6.x-2.3

I'm still seeing this problem on version 6.x-2.3 - I'm using the aquia prosper theme, but also tested on bluemarine. You get the upload CCK field, it uploads the video and then it looks like it disappears. You can save the node, but the video doesn't appear. The issue is only w/ the uploading of a video file - recording straight to the site seems to work well.

jeremyheslop’s picture

jeremyheslop commented

eodonnell,

What are your viddler content type settings? and which fields do you have enabled (ie upload, webcam, id, url, etc)

Have you tried enabling debuging? and are there any drupal or apache errors when you post a video?

I'm not able to reproduce this on my dev or testing server. Thanks,

Jeremy

eodonnell’s picture

eodonnell commented

Thanks Jeremy - I had fivestar enabled and subscriptions. I disabled both of those, and now it is working.