|
From: Arne L. (JIRA) <no...@at...> - 2006-05-15 15:33:47
|
[ http://opensource.atlassian.com/projects/spring/browse/RCP-267?page=comments#action_17623 ] Arne Limburg commented on RCP-267: ---------------------------------- OK, you are right. But, is there any opportunity at the current code to realize the behaviour of commiting on next without to much coding? - There is no onNextListener in the WizardContainer - You cannot get the buttons to register a listener - There is no onAboutToHide-Method in the WizardPage interface Am I missing a possibility to do it? My recommendation is to specify onAboutToHide in WizardPage and to implement an attribute commitOnNext in FormBackedWizardPage. For convenience there should be a second addForm-method in AbstractWizard with the signature addForm(Form form, boolean commitOnNext). Other proposals? > Forms in Wizards (with FormBackedWizardPages) are never commited by the WizardDialog (the only implementation of a WizardContainer). > ------------------------------------------------------------------------------------------------------------------------------------ > > Key: RCP-267 > URL: http://opensource.atlassian.com/projects/spring/browse/RCP-267 > Project: Spring Framework Rich Client Project > Type: Improvement > Environment: any > Reporter: Arne Limburg > Assignee: Oliver Hutchison > > Forms in Wizards (with FormBackedWizardPages) are never commited by the WizardDialog (the only implementation of a WizardContainer). I don't consider this as a bug, but this should be done, when a FormBackedWizardPage becomes invisible (due to a click on back or next or via showPage) if it is commitable. At least this should happen on a click on next. I suggest adding the methods isCommittable() and commit() to the FormBackedWizardPage, delegating to the backed Form's FormModel. Then the WizardDialog should call this methods in onNext(), onBack() and showPage() if the current page is an instance of FormBackedWizardPage. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/spring/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira |