You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by at...@apache.org on 2008/08/20 23:52:58 UTC

svn commit: r687481 - /portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java

Author: ate
Date: Wed Aug 20 14:52:58 2008
New Revision: 687481

URL: http://svn.apache.org/viewvc?rev=687481&view=rev
Log:
Dropping left over java.util.prefs handling for portlet prefs.
Still todo: how/when/what (set of) prefs should be deleted when a portlet app is removed?

Modified:
    portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java

Modified: portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java
URL: http://svn.apache.org/viewvc/portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java?rev=687481&r1=687480&r2=687481&view=diff
==============================================================================
--- portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java (original)
+++ portals/jetspeed-2/portal/branches/JS2-869/components/jetspeed-registry/src/main/java/org/apache/jetspeed/components/portletregistry/PersistenceBrokerPortletRegistry.java Wed Aug 20 14:52:58 2008
@@ -23,8 +23,6 @@
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
-import java.util.prefs.BackingStoreException;
-import java.util.prefs.Preferences;
 
 import org.apache.jetspeed.cache.JetspeedCache;
 import org.apache.jetspeed.cache.JetspeedCacheEventListener;
@@ -230,21 +228,7 @@
     {
         getPersistenceBrokerTemplate().delete(app);
         
-        String appNodePath = MutablePortletApplication.PREFS_ROOT + "/" +((MutablePortletApplication)app).getName();
-        try
-        {
-            if(Preferences.systemRoot().nodeExists(appNodePath))
-            {                   
-                Preferences node = Preferences.systemRoot().node(appNodePath);
-               // log.info("Removing Application preference node "+node.absolutePath());
-                node.removeNode();
-            }
-        }
-        catch (BackingStoreException e)
-        {
-           throw new RegistryException(e.toString(), e);
-        }
-
+        // TODO: remove PortletPreferences and then what scope: everything (default, global and entity prefs) or only default (portlet scope)?
     }
 
     public void updatePortletApplication( PortletApplicationDefinition app ) throws RegistryException



---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org