svn commit: r13447 - trunk/src_new/org/argouml/uml/ui/TabProps.java

Author: mvw
Date: 2007-08-25 13:52:21-0700
New Revision: 13447

Modified:
   trunk/src_new/org/argouml/uml/ui/TabProps.java

Log:
Fix for issue 4830: ModelListeners not removed from proppanels.
In fact, new proppanels were created at every target change.

Modified: trunk/src_new/org/argouml/uml/ui/TabProps.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/ui/TabProps.java?view=diff&rev=13447&p1=trunk/src_new/org/argouml/uml/ui/TabProps.java&p2=trunk/src_new/org/argouml/uml/ui/TabProps.java&r1=13446&r2=13447
==============================================================================
--- trunk/src_new/org/argouml/uml/ui/TabProps.java	(original)
+++ trunk/src_new/org/argouml/uml/ui/TabProps.java	2007-08-25 13:52:21-0700
@@ -304,9 +304,11 @@
     private TabModelTarget findPanelFor(Object trgt) {
         /* 1st attempt: get a panel that we created before: */
         TabModelTarget p = (TabModelTarget) panels.get(trgt.getClass());
-        if (p != null && LOG.isDebugEnabled()) {
-            LOG.debug("Getting prop panel for: " + trgt.getClass().getName()
-                    + ", " + "found (in cache?) " + p);
+        if (p != null) {
+            if (LOG.isDebugEnabled()) {
+                LOG.debug("Getting prop panel for: " + trgt.getClass().getName()
+                        + ", " + "found (in cache?) " + p);
+            }
             return p;
         }