Opened 12 years ago

Closed 11 years ago

#2351 closed defect (fixed)

passfunction of wizard pane is called for both buttons next and previous

Reported by: achim.hoeffmann@… Owned by: dante
Priority: high Milestone:
Component: Dojox Version: 0.4.1
Keywords: Cc:
Blocked By: Blocking:

Description

see the following thread on gmane.comp.web.dojo.user http://comments.gmane.org/gmane.comp.web.dojo.user/23506

Re: How to determine in which direction a wizard pane is left From: Karl Tiedt <ktiedt <at> gmail.com> Subject: Re: How to determine in which direction a wizard pane is left Newsgroups: gmane.comp.web.dojo.user Date: 2007-01-16 07:14:25 GMT

If your validation is fring everytime (back and forwards...) I would have to say its a bug based on this text:

_checkPass: function() {

summary: Called when the user presses the "next" button. Calls passFunction to see if it's OK to advance to next panel, and if it isn't, then display error. Returns true to advance, false to not advance.

If thats the case, you should file a bug at http://trac.dojotoolkit.org

-Karl

On 1/14/07, Achim Höffmann <achim.hoeffmann <at> online.de> wrote:

the passfunction of a wizard pane is called for both the next and the previous button. Typically, only for the next button some kind of processing (validation, ...) is necessary. So, I'm looking for something like ... if (<wizard's pane next button is clicked> ) ... to extend my passfunctions.

Attachments (2)

wizard_next_or_both.patch (5.0 KB) - added by ghenry@… 12 years ago.
Adds a checkBoth setting, with the default now to only check when the next button is clicked.
func.php (2.8 KB) - added by Slavon 5 years ago.
http://cureby.tumblr.com/

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by ghenry@…

Attaching a patch that adds a checkBoth="true" setting. The default is now to only check when the next button is clicked, but if you want the old behaviour, set checkBoth="false" in the WizardContainer?.

Gavin Henry.

http://www.suretecsystems.com/

Changed 12 years ago by ghenry@…

Attachment: wizard_next_or_both.patch added

Adds a checkBoth setting, with the default now to only check when the next button is clicked.

comment:2 Changed 12 years ago by bill

Component: WidgetsDojox
Owner: bill deleted

comment:3 Changed 11 years ago by dante

Owner: set to dante
Status: newassigned

comment:4 Changed 11 years ago by dante

Resolution: fixed
Status: assignedclosed

this doesn't seem to be happening after the refactor? i don't necessarily agree with the checkBoth logic (conditionally prevent people from changing their minds bout something) not sure i've ever seen a UI that would do that? reopen if important.

Changed 5 years ago by Slavon

Attachment: func.php added
Note: See TracTickets for help on using tickets.