portals-pluto-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From edalqu...@apache.org
Subject svn commit: r669675 - /portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java
Date Thu, 19 Jun 2008 20:23:00 GMT
Author: edalquist
Date: Thu Jun 19 13:22:59 2008
New Revision: 669675

URL: http://svn.apache.org/viewvc?rev=669675&view=rev
Log:
PLUTO-487 Remove call to internalStore from constructor, make internalStore() available as
a protected method to sub-classes

Modified:
    portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java

Modified: portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java
URL: http://svn.apache.org/viewvc/portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java?rev=669675&r1=669674&r2=669675&view=diff
==============================================================================
--- portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java
(original)
+++ portals/pluto/branches/pluto-1.1.x/pluto-container/src/main/java/org/apache/pluto/internal/impl/PortletPreferencesImpl.java
Thu Jun 19 13:22:59 2008
@@ -132,22 +132,9 @@
                 preferences.put(storedPreferences[i].getName(),
                                 storedPreferences[i]);
             }
-        	// Store the preferences retrieved from portlet.xml.
-            //   Portlet preferences are stored everytime when a
-            //   PortletPreferencesImpl instance is created.
-            //   So here we do not check the portlet request method ID.
-        	internalStore();
-
         } catch (PortletContainerException ex) {
             LOG.error("Error retrieving preferences.", ex);
             //TODO: Rethrow up the stack????
-        } catch (IOException ex) {
-            LOG.error("Error retrieving preferences.", ex);
-            //TODO: Rethrow up the stack????
-        } catch (ValidatorException ex) {
-            LOG.warn("ValidatorException initializing portlet preferences. "
-            		+ "This is not illegal at this point "
-            		+ "since we are just retreiving from portlet.xml.", ex);
         }
         if (LOG.isDebugEnabled()) {
         	LOG.debug("Merged stored preferences: " + toString());
@@ -302,7 +289,7 @@
      * @throws ValidatorException  if the portlet preferences are not valid.
      * @throws IOException  if an error occurs with the persistence mechanism.
      */
-    private void internalStore() throws IOException, ValidatorException {
+    protected final void internalStore() throws IOException, ValidatorException {
         // Validate the preferences before storing, if a validator is defined.
         //   If the preferences cannot pass the validation,
         //   an ValidatorException will be thrown out.



Mime
View raw message