the deletion api converted the confirm_form() arguments to the same kind of structure the l() and url() now use. this patch updates the rest of the confirm forms in core.

CommentFileSizeAuthor
confirm_form_0.patch17.41 KBhunmonk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hunmonk’s picture

hunmonk CreditAttribution: hunmonk commented
Status: Needs review » Reviewed & tested by the community

tested several confirm forms, all seems to be fine.

Steven’s picture

Steven CreditAttribution: Steven commented
Status: Reviewed & tested by the community » Fixed

Did some testing myself, and found a lingering help message in the format deletion that was easily fixed as well.

Committed to HEAD.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
Pancho’s picture

Pancho
Location UTC+2 🇪🇺 EU
CreditAttribution: Pancho commented
Status: Closed (fixed) » Closed (won't fix)

Annotation: This has been rolled back on Juli 1, 2007 by Dries

- Rollback of patch #147723: delete API. Talked to Steven and Gabor and we
unanimously agreed to rollback the deletion API. We all support the
features this patch added, yet not its actual design and implementation.
After some talk, we decided that it would be better for Drupal -- in the
long term -- not to go with a solution that isn't 100%. We also recognize
that in the short term, this patch would have been useful addition. So
let's figure out how we can implement this properly in D7.

Pancho’s picture

Pancho
Location UTC+2 🇪🇺 EU
CreditAttribution: Pancho commented
Status: Closed (won't fix) » Closed (fixed)