svn commit: r1679970 - /myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java

[de...@apache.org]

Author: deki
Date: Mon May 18 09:01:55 2015
New Revision: 1679970

URL: http://svn.apache.org/r1679970
Log:
TOBAGO-1464: mark ActionListenerImpl as deprecated

Modified:
    myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java

Modified: myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java?rev=1679970&r1=1679969&r2=1679970&view=diff
==============================================================================
--- myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java (original)
+++ myfaces/tobago/trunk/tobago-core/src/main/java/org/apache/myfaces/tobago/application/ActionListenerImpl.java Mon May 18 09:01:55 2015
@@ -34,6 +34,10 @@ import javax.faces.event.AbortProcessing
 import javax.faces.event.ActionEvent;
 import javax.faces.event.ActionListener;
 
+/**
+ * @deprecated use JSF 2 exception handler
+ */
+@Deprecated
 public class ActionListenerImpl implements ActionListener {
 
   private static final Logger LOG = LoggerFactory.getLogger(ActionListenerImpl.class);
@@ -59,11 +63,14 @@ public class ActionListenerImpl implemen
           fe = fe.getCause();
         }
       }
-      LOG.error("Processing failed. Forwarding to error page. errorOutcome=" + errorOutcome, e.getCause());
       final FacesContext facesContext = FacesContext.getCurrentInstance();
+      String errorLogMsg = "Processing failed. Forwarding to error page. errorOutcome=" + errorOutcome;
       if (e.getCause() != null) {
-         final FacesMessage facesMessage = new FacesMessage(e.getCause().toString());
-         facesContext.addMessage(null, facesMessage);
+        LOG.error(errorLogMsg, e.getCause());
+        final FacesMessage facesMessage = new FacesMessage(e.getCause().toString());
+        facesContext.addMessage(null, facesMessage);
+      } else {
+        LOG.error(errorLogMsg, e);
       }
       final UIComponent source = event.getComponent();
       final ActionSource2 actionSource = (ActionSource2) source;