Sometimes a box class needs to take over the whole boxes edit form and not just add in the options. The patch to come will add a check to see if the box want to define the whole form and if it does, hand over the form to the box.

This is need for the entity_boxes module http://drupal.org/project/entity_boxes

CommentFileSizeAuthor
#7 133012-add-option-for-box-to-define-edit-form-d6.patch450 bytessmithmilner
#3 133012-add-option-for-box-to-define-edit-form.patch478 bytese2thex
#3 133012-add-option-for-box-to-define-edit-form-make.patch474 bytese2thex
#2 133012-add-option-for-box-to-define-edit-form-make.patch476 bytese2thex
#1 133012-add-option-for-box-to-define-edit-form.patch480 bytese2thex

Comments

e2thex’s picture

e2thex commented
StatusFileSize
new 133012-add-option-for-box-to-define-edit-form.patch480 bytes

This patch adds a check to see if the box has a full_form method, and if that method exists then it call and returned, instead of the standard form processing

e2thex’s picture

e2thex commented
StatusFileSize
new 133012-add-option-for-box-to-define-edit-form-make.patch476 bytes

This patch is the same as in #1 but it is a -p0 version for older drush make installs

e2thex’s picture

e2thex commented
StatusFileSize
new 133012-add-option-for-box-to-define-edit-form-make.patch474 bytes
new 133012-add-option-for-box-to-define-edit-form.patch478 bytes

change to calling the form method box_form

febbraro’s picture

febbraro commented
Status: Active » Fixed

Thanks Erik.

http://drupalcode.org/project/boxes.git/commit/ce6c5b0

Status: Fixed » Closed (fixed)

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

smithmilner’s picture

smithmilner commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Closed (fixed) » Needs review

Rerolled for d6 dev.

smithmilner’s picture

smithmilner commented
StatusFileSize
new 133012-add-option-for-box-to-define-edit-form-d6.patch450 bytes

attaching it would be useful...

tirdadc’s picture

tirdadc commented
Status: Needs review » Fixed

Changed it to box_form so it's consistent with the D7 version and committed.

Status: Fixed » Closed (fixed)

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