Bug #29178 for Tk-Wizard: callback_dirselect is hard to use

Tue Sep 04 21:43:14 2007 MTHURN [...] cpan.org - Ticket created

Subject:

callback_dirselect is hard to use

Wizard.pm contains a useful callback for the dirSelect page, but it's hard to use. The only way I know how is to supply a preNextButtonAction which looks at the current page number, and if we're on the dirSelect page, call the dirselect callback. It would be nice if there were a more convenient way, such as the ability to install callbacks on a per-page basis. -- - - Martin 'Kingpin' Thurn

Tue Sep 04 21:43:35 2007 MTHURN [...] cpan.org - Severity Normal changed to Wishlist

Thu Dec 20 09:30:01 2007 LGODDARD [...] cpan.org - Taken

Thu Dec 20 09:30:53 2007 LGODDARD [...] cpan.org - Correspondence added

Cuold you please explain the problems you find in using it? Since there is already support for pre- and post-button callbacks, I'm not sure what other callbacks you would like: could you please provide a use case? Thanks Lee

Thu Dec 20 09:30:54 2007 The RT System itself - Status changed from 'new' to 'open'

Thu Dec 20 09:30:55 2007 LGODDARD [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Dec 20 09:31:09 2007 LGODDARD [...] cpan.org - Status changed from 'resolved' to 'open'

Thu Dec 20 18:20:00 2007 MTHURN [...] cpan.org - Correspondence added

On Thu Dec 20 09:30:53 2007, LGODDARD wrote: Show quoted text

> Cuold you please explain the problems you find in using it?

The only way to set a callback is on the Wizard object. That means the same callback gets called for EVERY page. But of course we want the dirselect callback to be called only for a dirselect page, not any other pages. And in general, as the user flips from page to page, we only want to do checking for the things that happened on that page. If the same callback gets called for every page, the logic in that mother-of-all-callbacks gets very convoluted -- at best, it ends up looking like a big cascaded IF statement that uses the current (and/or next) page number to decide what to do... These are reasons that per-page callbacks make sense. -- - - Martin 'Kingpin' Thurn

Fri Dec 21 06:22:03 2007 LGODDARD [...] cpan.org - Correspondence added

I'm adding it now.

Tue Jan 29 23:56:23 2008 LGODDARD [...] cpan.org - Correspondence added

Added feature.

Tue Jan 29 23:56:27 2008 LGODDARD [...] cpan.org - Status changed from 'open' to 'resolved'