[jboss-svn-commits] JBL Code SVN: r11760 - in labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums: util and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: dejp
Date: 2007-05-08 13:48:08 -0400 (Tue, 08 May 2007)
New Revision: 11760

Modified:
   labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/impl/ForumsModuleImpl.java
   labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/util/NotificationEngine.java
Log:
JBFORUMS-212

Modified: labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/impl/ForumsModuleImpl.java
===================================================================
--- labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/impl/ForumsModuleImpl.java	2007-05-08 17:47:40 UTC (rev 11759)
+++ labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/impl/ForumsModuleImpl.java	2007-05-08 17:48:08 UTC (rev 11760)
@@ -1562,6 +1562,8 @@
             throw new ModuleException("forum must not be null");
          }
 
+         session.save(poster);
+         
          ForumWatch forumWatch = new ForumWatchImpl();
          forumWatch.setPoster(poster);
          forumWatch.setForum(forum);
@@ -1652,6 +1654,8 @@
          {
             throw new ModuleException("topic must not be null");
          }
+         
+         session.save(poster);
 
          TopicWatch topicWatch = new TopicWatchImpl();
          topicWatch.setPoster(poster);

Modified: labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/util/NotificationEngine.java
===================================================================
--- labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/util/NotificationEngine.java	2007-05-08 17:47:40 UTC (rev 11759)
+++ labs/jbossforums/branches/forums22/forums/src/main/org/jboss/portlet/forums/util/NotificationEngine.java	2007-05-08 17:48:08 UTC (rev 11760)
@@ -65,6 +65,7 @@
 import org.jboss.portlet.forums.model.TopicWatch;
 import org.jboss.portlet.forums.model.Watch;
 import org.jboss.portlet.forums.ui.Constants;
+import org.jboss.portlet.forums.ui.PortalUtil;
 
 import EDU.oswego.cs.dl.util.concurrent.LinkedQueue;
 import EDU.oswego.cs.dl.util.concurrent.QueuedExecutor;
@@ -263,7 +264,7 @@
                   User watcher = watch.getPoster().getUser();
                   Object watcherId = watcher.getId();
                   
-                  if (!notifieds.contains(watcherId))
+                  if (!notifieds.contains(watcherId) && !watcherId.equals(PortalUtil.getUserNA().getId()))
                   {
                      
                      boolean securityFlag = true;
@@ -323,7 +324,7 @@
                       TopicWatch watch = (TopicWatch)i.next();
                       User watcher = watch.getPoster().getUser();
                       Object watcherId = watcher.getId();
-                      if (!notifieds.contains(watcherId))
+                      if (!notifieds.contains(watcherId) && !watcherId.equals(PortalUtil.getUserNA().getId()))
                       {
                           boolean securityFlag = true;