right now the FAPI attribute #skip_duplicate_check can only be manipulated via hook_form_alter() -- it cannot be passed as an initial attribute in a form array, because drupal_prepare_form overwrites the value after the initial form array is passed in. this is rather inconvenient if you actually want to use this attribute. :)

attached patch simply adds a check for an existing setting before setting to the default value.

CommentFileSizeAuthor
skip_check_1.patch450 byteshunmonk

Comments

dries’s picture

Status: Needs review » Fixed

Committed to CVS HEAD. Thanks! :)

heine’s picture

Version: 6.x-dev » 5.x-dev
Status: Fixed » Reviewed & tested by the community

Applies with a small offset to 5.x as well.

chx’s picture

Priority: Normal » Critical

confirm_form is broken in DRUPAL-5 because of this and there is a fix which also needs to be backported -- because of this, I deem this critic.

drumm’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 5.

Doesn't look like 4.7 is needed.

Anonymous’s picture

Status: Fixed » Closed (fixed)