[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-1747) SeamPhaseListener transaction debug log statements are misleading/confusing

[Matt Drees (JIRA)]

SeamPhaseListener transaction debug log statements are misleading/confusing
---------------------------------------------------------------------------

                 Key: JBSEAM-1747
                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1747
             Project: JBoss Seam
          Issue Type: Bug
    Affects Versions: 2.0.0.BETA1
         Environment: 20070727.1142
            Reporter: Matt Drees
            Priority: Trivial


Debug logs from SeamPhaseListener look something like this on a non-faces request:

DEBUG 31-07 11:34:31,467 (org.jboss.seam.jsf.SeamPhaseListener:begin:573) beginning transaction prior to phase: RESTORE_VIEW 1
DEBUG 31-07 11:34:31,560 (org.jboss.seam.jsf.SeamPhaseListener:commitOrRollback:589) committing transaction after phase: RESTORE_VIEW 1
DEBUG 31-07 11:34:31,560 (org.jboss.seam.jsf.SeamPhaseListener:begin:573) beginning transaction prior to phase: RENDER_RESPONSE 6
DEBUG 31-07 11:34:31,576 (org.jboss.seam.jsf.SeamPhaseListener:commitOrRollback:589) committing transaction after phase: INVOKE_APPLICATION 5
DEBUG 31-07 11:34:31,592 (org.jboss.seam.jsf.SeamPhaseListener:begin:573) beginning transaction prior to phase: INVOKE_APPLICATION 5
DEBUG 31-07 11:34:33,731 (org.jboss.seam.jsf.SeamPhaseListener:commitOrRollback:589) committing transaction after phase: RENDER_RESPONSE 6

It's misleading.  For one thing, there is no "INVOKE_APPLICATION" phase; but that phaseid is used for logging before and after page actions.  For another, the "before" and "after" statements are backwards (semantically).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira