Hopefully these will clear up a few of the other bugs in the issue queue as well!

1) CCK module does not use module_invoke_all('flashvideo_delete_file'... when updating nodes
2) _flashvideo_perform_postop does not pass vid if using filefield
3) Old Drupal 5 style usage of l() function in flashvideo-play.tpl.php

Kudos to stirlyn

Pobster

CommentFileSizeAuthor
flashvideo-pobster.patch4 KBpobster

Comments

pobster’s picture

pobster commented

Incidentally just to point out that 1 + 2 are sort of related and only probably useful if actually utilising the module invoke hooks...

Pobster

pobster’s picture

pobster commented
Title: Several fixes... » Couple of fixes...

Yeah maybe "several" is ambitious ;o)

travist’s picture

travist commented
Status: Needs review » Patch (to be ported)

Thanks for your efforts pobster. I took a look at the patch and it looks good to me. Mark, you can submit this or I can. It is up to you.

Thanks everyone.

attheshow’s picture

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

Thanks pobster. Just committed to dev version.

Status: Fixed » Closed (fixed)

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