You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by "Craig McClanahan (JIRA)" <ji...@apache.org> on 2006/04/26 08:14:50 UTC

[jira] Moved: (SHALE-43) [shale] DialogNavigationHandler behavior depends on isDebugEnabled

     [ http://issues.apache.org/struts/browse/SHALE-43?page=all ]

Craig McClanahan moved STR-2483 to SHALE-43:
--------------------------------------------

      Project: Shale  (was: Struts Action 1)
          Key: SHALE-43  (was: STR-2483)
    Component:     (was: Shale)
      Version:     (was: Nightly Build)
    Assign To:     (was: Struts Developer Mailing List)

> [shale] DialogNavigationHandler  behavior depends on isDebugEnabled
> -------------------------------------------------------------------
>
>          Key: SHALE-43
>          URL: http://issues.apache.org/struts/browse/SHALE-43
>      Project: Shale
>         Type: Bug

>  Environment: Operating System: All
> Platform: All
>     Reporter: Keijo Nurmes
>     Priority: Minor

>
> In class org.apache.shale.dialog.faces.DialogNavigationHandler method transition:
> transition from a ViewState with null outcome, current state is returned  
> only if 'log.isDebugEnabled()' is true.
> Suggested patch:
> Index:
> /shale-trunk/core-library/src/java/org/apache/shale/dialog/faces/DialogNavigationHandler.java
> ===================================================================
> ---
> /shale-trunk/core-library/src/java/org/apache/shale/dialog/faces/DialogNavigationHandler.java
> (revision 170542)
> +++
> /shale-trunk/core-library/src/java/org/apache/shale/dialog/faces/DialogNavigationHandler.java
> (working copy)
> @@ -470,8 +470,8 @@
>          if ((state instanceof ViewState) && (outcome == null)) {
>              if (log.isDebugEnabled()) {
>                  log.debug("  --> Stay in current state");
> -                return state;
>              }
> +            return state;
>          }
>  
>          // Identify the appropriate Transition

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org