Scarab commit: svn commit: r10733 - trunk/src/java/org/tigris/scarab: notification om

Hussayn Dabbous <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: dabbous
Date: 2009-05-25 14:42:06-0700
New Revision: 10733

Modified:
   trunk/src/java/org/tigris/scarab/notification/ScarabNewNotificationManager.java
   trunk/src/java/org/tigris/scarab/om/NotificationStatus.java

Log:
SCB2873: workaround for DATA TOO LONG error in notification manager

Modified: trunk/src/java/org/tigris/scarab/notification/ScarabNewNotificationManager.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/notification/ScarabNewNotificationManager.java?view=diff&pathrev=10733&r1=10732&r2=10733
==============================================================================
--- trunk/src/java/org/tigris/scarab/notification/ScarabNewNotificationManager.java	(original)
+++ trunk/src/java/org/tigris/scarab/notification/ScarabNewNotificationManager.java	2009-05-25 14:42:06-0700
@@ -570,7 +570,7 @@
                 }
                 catch (Exception e)
                 {
-                    log.error("sendPendingNotifications(): Updating: " + e);
+                    log.error("sendPendingNotifications(): Updating: " + e, e);
                 }
             }
         }

Modified: trunk/src/java/org/tigris/scarab/om/NotificationStatus.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/NotificationStatus.java?view=diff&pathrev=10733&r1=10732&r2=10733
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/NotificationStatus.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/NotificationStatus.java	2009-05-25 14:42:06-0700
@@ -101,7 +101,22 @@
             this.setActivity(activity);
             att = activity.getActivitySet().getAttachment();
             if (att != null)
-            	this.setComment(att.getData());
+            {
+                String newComment = att.getData();
+                if ( newComment != null)
+                {
+                    if(newComment.length() > 255)
+                    {
+                        String truncatedPart = newComment.substring(256, newComment.length());
+                        newComment           = newComment.substring(0,256);
+                        this.getLog().warn("Truncated Comment to 255 characters:\n***BEGIN***\n" 
+                                     + newComment
+                                     + "\n***REST TRUNCATED***\n"
+                                     + truncatedPart);
+                    }
+            	    this.setComment(newComment);
+                }
+            }
             Integer receiverId = receiver.getUserId();
             if(receiverId == null)
             {

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2353642
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.