Line 731 should have the 'Confirm' sent through the t() function. Otherwise the condition will fail if the string "Confirm" has been translated:

if ($form_state['clicked_button']['#value'] == t('Confirm')) {

Comments

grub3’s picture

grub3 commented

Great catch. This fixes my problem. Thanks. This should be committed to CVS ASAP.

disterics’s picture

disterics commented
Status: Active » Fixed

Fix is commited.

mjourney2’s picture

mjourney2 commented

There is a second instance further down in the code where the same == 'Confirm' needs to be changed to == t('Confirm'). You may want to fix that also. Thanks.

disterics’s picture

disterics commented

Aaah! Stupid me. Will do.

Anonymous’s picture

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

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