onUpdate option doesn't work [#442942]

The onUpdate option for defining a callback function for updated forms doesn't work. (And yes, I did set updateMethod to "callback".) The reason it's not working is that onUpdate is not part of the defaults definition. The attached patch fixes the problem.

Comment	File	Size	Author
#1	popups-default-setting.patch	867 bytes	zoo33

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

zoo33 CreditAttribution: zoo33 commented 23 April 2009 at 15:44

File	Size
popups-default-setting.patch	867 bytes

Comment #2

vertazzar CreditAttribution: vertazzar commented 27 April 2009 at 20:44

thanks -

btw can you fix the problem with caching ..

if you aggregate js popups 6.x -2.0 has mayor problems, it just "gets stuck with loading "

and how to disable js includes in popups?

Comment #3

rszrama CreditAttribution: rszrama commented 15 August 2009 at 22:27

Status:

Needs review

» Reviewed & tested by the community

I ran into this same problem and arrived at the same fix. I came here to post an issue but was a good boy and searched first.

Patch +1. This is essential.

Comment #4

iboxsolutions CreditAttribution: iboxsolutions commented 22 August 2009 at 13:29

Hi,

I've read the release notes but I'm not sure how you add options to links to override the default settings. I'm using popup links on a slider and when the popup window closes it goes back to the first slide so I need to enable the noUpdate option but not sure how :-S

Any help much appreciated.

Also, any idea on when Alpha 6 will be released (I've never applied patches before)?

Thanks in advance,

Tim