I'm been working on updating everything for drupal 4.7, mainly for the new form api.

This version here mostly works except for a problem with the checkbox's. I'm posting it now incase anyone wants to test it right away. When I can get everything working i'll commit to cvs for more testing.

CommentFileSizeAuthor
#3 video.module_4.patch33.75 KBLukeLast
video_4.module68.76 KBLukeLast

Comments

eaton’s picture

eaton commented

I'd put money on the checkbox issue being related to http://drupal.org/node/35816 -- that patch fixes a number of problems with checkbox values, and should be committed shortly.

fax8’s picture

fax8 commented

can you please post it as a patch file???
Thanks.

Fabio

LukeLast’s picture

LukeLast commented
StatusFileSize
new video.module_4.patch33.75 KB

I'll wait until the checkbox bug fix is committed to HEAD before I commit this.

fax8’s picture

fax8 commented

good work..

you changed also the returning values of functions (return $output instead of print(..) )
and node_load() changes wich were on of my todos. Good!

ok for me.

Fabio

LukeLast’s picture

LukeLast commented
Status: Needs review » Fixed

CVS updated and the checkbox bug has been fixed.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)