You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@sling.apache.org by cz...@apache.org on 2010/01/12 14:28:36 UTC

svn commit: r898315 - /sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java

Author: cziegeler
Date: Tue Jan 12 13:28:35 2010
New Revision: 898315

URL: http://svn.apache.org/viewvc?rev=898315&view=rev
Log:
Fix copy/paste error: listener should be removed (not added) on remove

Modified:
    sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java

Modified: sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java
URL: http://svn.apache.org/viewvc/sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java?rev=898315&r1=898314&r2=898315&view=diff
==============================================================================
--- sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java (original)
+++ sling/whiteboard/portal/container-api/src/main/java/org/apache/sling/portal/container/api/PortletApplicationRegistry.java Tue Jan 12 13:28:35 2010
@@ -91,7 +91,7 @@
      */
     public synchronized void removeListener(PortletApplicationRegistryListener listener) {
         if ( listener != null ) {
-            this.listeners.add(listener);
+            this.listeners.remove(listener);
             for(final PortletApplication callback : callbacks) {
                 listener.notifyPortletApplicationRemoved(callback);
             }