the function system_settings_form_submit, unsets some form values, but not enough. Patch attached to catch 'op' and 'form_token'. This should be backported to 4.7 too for 'form_token'.

Also, I don't know why this related patch is labeled as being committed:
http://drupal.org/node/48209

but it apparently never was.

CommentFileSizeAuthor
#6 cleanup_settings_submit_47_6.diff733 bytespwolanin
#4 system_17.patch753 byteschx
cleanup_settings_submit_0.diff814 bytespwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Fixed

Committed to HEAD.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Version: 5.x-dev » 4.7.x-dev
Status: Fixed » Patch (to be ported)
killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented

are you sure this needs to be ported? That part of the code has changed between 4 and 5.

chx’s picture

chx CreditAttribution: chx commented
Status: Patch (to be ported) » Reviewed & tested by the community
FileSize
system_17.patch753 bytes

Yes, this needs porting and the porting is trivial.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

applied

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Fixed » Reviewed & tested by the community
FileSize
cleanup_settings_submit_47_6.diff733 bytes

@killes - I can't see that this change was applied to the 4.7 branch.

Attached is a patch for 4.7

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

strange. it is now applied.

Anonymous’s picture

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