Return-Path: Delivered-To: apmail-shale-issues-archive@locus.apache.org Received: (qmail 56605 invoked from network); 2 Jan 2007 21:02:53 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 2 Jan 2007 21:02:53 -0000 Received: (qmail 70900 invoked by uid 500); 2 Jan 2007 21:02:48 -0000 Delivered-To: apmail-shale-issues-archive@shale.apache.org Received: (qmail 70876 invoked by uid 500); 2 Jan 2007 21:02:48 -0000 Mailing-List: contact issues-help@shale.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@shale.apache.org Delivered-To: mailing list issues@shale.apache.org Received: (qmail 70826 invoked by uid 99); 2 Jan 2007 21:02:48 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Jan 2007 13:02:47 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Jan 2007 13:02:39 -0800 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 2C2A57142E4 for ; Tue, 2 Jan 2007 13:02:19 -0800 (PST) Message-ID: <25779700.1167771739177.JavaMail.jira@brutus> Date: Tue, 2 Jan 2007 13:02:19 -0800 (PST) From: "mailmarshal@indexmultimedia.com (JIRA)" To: issues@shale.apache.org Subject: [jira] Commented: (SHALE-188) Add a default outcome to the dialog manager MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ http://issues.apache.org/struts/browse/SHALE-188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39506 ] mailmarshal@indexmultimedia.com commented on SHALE-188: ------------------------------------------------------- Your message Subject: "[jira] Commented: (SHALE-188) Add a default outcome to the dialog manager" could not be delivered to some or all of the intended recipients. User - (550 5.1.1 User unknown) Additional Information ====================== Original Sender: Sender-MTA: <140.211.11.4> Remote-MTA: <192.168.240.2> Reporting-MTA: MessageName: Last-Attempt-Date: <22:00:30 Tue, 02 January 2007> > Add a default outcome to the dialog manager > ------------------------------------------- > > Key: SHALE-188 > URL: http://issues.apache.org/struts/browse/SHALE-188 > Project: Shale > Issue Type: Improvement > Components: Dialog > Environment: Debian Linux - JDK 1.5 > Reporter: Laure Goudon > Fix For: 1.0.4-SNAPSHOT > > > Version from manifest: Implementation-Version: 20060326 > Hello, > When we click on the browser back button while in a dialog and then click on another link or button, an Illegal argument exception comes (see stacktrace). This error occurs because Shale can't find any transition for this action. It would be a good idea that we could have a default transition (for instance named "*" or "default") that the dialog navigation handler class would use if no other transition was found for this state or dialog. > Thanks. > stacktrace : > java.lang.IllegalArgumentException: You have requested a transition outcome named "clusterMenu" from a state named "viewApplicationView" in a dialog named "viewApplication", but no transition definition can be found. Double check the spelling of the transition outcome name. > at org.apache.shale.dialog.faces.DialogNavigationHandler.transition(DialogNavigationHandler.java:526) > at org.apache.shale.dialog.faces.DialogNavigationHandler.postprocess(DialogNavigationHandler.java:353) > at org.apache.shale.dialog.faces.DialogNavigationHandler.handleNavigation(DialogNavigationHandler.java:219) > at org.apache.myfaces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:84) > at javax.faces.component.UICommand.broadcast(UICommand.java:106) > at org.apache.myfaces.custom.navmenu.htmlnavmenu.HtmlCommandNavigationItem.broadcast(HtmlCommandNavigationItem.java:228) > Proposed solution : > In the class DialogNavigationHandler in the method transition > /** Default outcome name constant as in : > * > */ > public static final String DEFAULT_OUTCOME_NAME = "*"; > // Identify the appropriate Transition > Transition transition = state.findTransition(outcome); > if (transition == null) { > transition = state.getDialog().findTransition(outcome); > // ---- Begin patch ---- > // If not found, check the default outcome for the current state > if (transition == null) { > Transition transition = state.findTransition(outcome); > // If not found, check the default outcome for the current dialog. > if (transition == null) { > transition = state.getDialog().findTransition(DEFAULT_OUTCOME_NAME); > } > } > // ---- End patch ---- > } > if (transition == null) { > throw new IllegalArgumentException(messages.getMessage("dialog.noTransition", > new Object[] { outcome, > state.getName(), > state.getDialog().getName() })); > } -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/struts/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira