Opened 10 years ago

Closed 9 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@… 10 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 3 years ago.
http://cureby.tumblr.com/

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 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 10 years ago by ghenry@…

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

comment:2 Changed 10 years ago by bill

  • Component changed from Widgets to Dojox
  • Owner bill deleted

comment:3 Changed 9 years ago by dante

  • Owner set to dante
  • Status changed from new to assigned

comment:4 Changed 9 years ago by dante

  • Resolution set to fixed
  • Status changed from assigned to closed

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 3 years ago by Slavon

Note: See TracTickets for help on using tickets.