My Configuration

Video 6.x-3.4
Acquia Drupal 1.2.21
CCK (Built In)

Steps to recreate

->configuring a node with a cck widget type Video
->Enable auto conversion video & Enable auto thumbnail video
->Make sure Title is a mandatory field(Its ok,If you have any other field as mandatory)
->Just Upload a video leaving the title empty
->Select a thumbnail and press save
->You get a Validation failed message & a Message saying that the file was queued for processing (see snapshot1.png)
->Now put in a title and save
->Now you get a user warning saying that there is a duplicate entry in the video_rendering table (see snapshot2.png)

CommentFileSizeAuthor
#6 uploadfield_convert.inc_691200-6.patch5.42 KBiLLin
#1 snapshot1.png60.06 KBvenkiparam
#1 snapshot2.png29.31 KBvenkiparam

Comments

venkiparam’s picture

venkiparam commented
StatusFileSize
new snapshot2.png29.31 KB
new snapshot1.png60.06 KB

adding the missed out snapshots

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented

Seems our insertion API is calling when validate videos, I will fix it.

venkiparam’s picture

venkiparam commented

Thanks heshanmw. Any updates on this issue ?. I checked the CVS Commit messages and didn't see any updates relating to this issue.

venkiparam’s picture

venkiparam commented

Any updates on this issue

iLLin’s picture

iLLin commented

I will look at this issue today.

iLLin’s picture

iLLin commented
StatusFileSize
new uploadfield_convert.inc_691200-6.patch5.42 KB

Ok I got this tracked. I am attaching a patch for the uploadfield_convert.inc file. There is also some "extra" stuff in there commented out for future stuff im working on that will include converting your video on save. I have it working but the module itself needs other files I have created, one being a conversion class. I'm working with Heshan to get this stuff worked in. Anyway this will fix your duplicate entry problem.

-d

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Active » Patch (to be ported)
navaneeth_r’s picture

navaneeth_r commented

Hi

I applied your patch. I got Fatal error: Call to undefined function video_tmp_load_video() errors. Function missing in this patch.

Thanks

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Version: 6.x-3.4 » 6.x-3.9

This patch worked for me. I have tested it. Did u upgrade your module to latest and add the patch in to it?

iLLin’s picture

iLLin commented
Status: Patch (to be ported) » Fixed

Ported to 3.x-dev and 4.x-dev.

Status: Fixed » Closed (fixed)

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