svn commit: r1081998 - /incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java

[rgardler]

Author: rgardler
Date: Tue Mar 15 23:20:33 2011
New Revision: 1081998

URL: http://svn.apache.org/viewvc?rev=1081998&view=rev
Log:
More useful error reporting and logging - tell us what the actual problem is.

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java

Modified: 
incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java?rev=1081998&r1=1081997&r2=1081998&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java 
(original)
+++ incubator/wookie/trunk/src/org/apache/wookie/server/ContextListener.java 
Tue Mar 15 23:20:33 2011
@@ -214,24 +214,24 @@ public class ContextListener implements 
                                                        
persistenceManager.commit();
                                                } catch (IOException e) {
                             persistenceManager.rollback();
-                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.1");
+                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.1") + " - " + 
e.getLocalizedMessage();
                                                        
FlashMessage.getInstance().error(error);
-                                                       _logger.error(error);
+                                                       _logger.error(error, e);
                                                } catch 
(BadWidgetZipFileException e) {
                             persistenceManager.rollback();
-                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.2");
+                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.2") + " - " + 
e.getLocalizedMessage();
                                                        
FlashMessage.getInstance().error(error);
-                                                       _logger.error(error);
+                                                       _logger.error(error, e);
                                                } catch (BadManifestException 
e) {
                             persistenceManager.rollback();
-                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.3");
+                                                       String error = 
f.getName()+":"+localizedMessages.getString("WidgetHotDeploy.3") + " - " + 
e.getLocalizedMessage();
                                                        
FlashMessage.getInstance().error(error);
-                                                       _logger.error(error);
+                                                       _logger.error(error, e);
                                                } catch (Exception e) {
                             persistenceManager.rollback();
                                                        String error = 
f.getName()+":"+e.getLocalizedMessage();
                                                        
FlashMessage.getInstance().error(error);
-                                                       _logger.error(error);
+                                                       _logger.error(error, e);
                                                } finally {
                                            // close thread persistence manager
                                            
PersistenceManagerFactory.closePersistenceManager();