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 dl...@apache.org on 2004/06/05 18:18:48 UTC
cvs commit: jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/tools/pamanager JetspeedDescriptorUtilities.java
dlestrat 2004/06/05 09:18:48
Modified: portal/src/java/org/apache/jetspeed/tools/pamanager
JetspeedDescriptorUtilities.java
Log:
Clean up. See http://nagoya.apache.org/jira/browse/JS2-65
Revision Changes Path
1.5 +4 -53 jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/tools/pamanager/JetspeedDescriptorUtilities.java
Index: JetspeedDescriptorUtilities.java
===================================================================
RCS file: /home/cvs/jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/tools/pamanager/JetspeedDescriptorUtilities.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- JetspeedDescriptorUtilities.java 25 Mar 2004 21:42:32 -0000 1.4
+++ JetspeedDescriptorUtilities.java 5 Jun 2004 16:18:48 -0000 1.5
@@ -33,8 +33,9 @@
public class JetspeedDescriptorUtilities
{
protected final static Log log = LogFactory.getLog(JetspeedDescriptorUtilities.class);
+
/**
- * Load a portlet.xml file into a Portlet Application tree
+ * Load jetspeed-portlet.xml file into a Portlet Application tree
*
* @param pathPortletXML The path to the portlet.xml file
* @return Application The Java object tree representing portlet.xml
@@ -49,61 +50,12 @@
FileReader reader = new java.io.FileReader(pathPortletXML);
Digester digester = new Digester();
- //digester.setLogger(log);
digester.setValidating(false);
digester.setNamespaceAware(true);
digester.push(app.getMetadata());
digester.addRuleSet(new MetadataRuleSet("portlet-app/"));
- /*
- digester.addRule("portlet-app/title", new LocalizedFieldRule());
- digester.addRule("portlet-app/contributor", new LocalizedFieldRule());
- digester.addRule("portlet-app/creator", new LocalizedFieldRule());
- digester.addRule("portlet-app/coverage", new LocalizedFieldRule());
- digester.addRule("portlet-app/description", new LocalizedFieldRule());
- digester.addRule("portlet-app/format", new LocalizedFieldRule());
- digester.addRule("portlet-app/identifier", new LocalizedFieldRule());
- digester.addRule("portlet-app/language", new LocalizedFieldRule());
- digester.addRule("portlet-app/publisher", new LocalizedFieldRule());
- digester.addRule("portlet-app/relation", new LocalizedFieldRule());
- digester.addRule("portlet-app/right", new LocalizedFieldRule());
- digester.addRule("portlet-app/source", new LocalizedFieldRule());
- digester.addRule("portlet-app/subject", new LocalizedFieldRule());
- digester.addRule("portlet-app/type", new LocalizedFieldRule());
- digester.addRule("portlet-app/metadata", new LocalizedFieldRule());
- */
-
- // digester.addSetNext("portlet-app", "setMetadata");
-
- //digester.addRule("portlet-app/portlet", new PortletRule(app));
-
- /*
- This will continually push an object onto the digester stack it will never pop it off.
- The only negative effect is increased memory/stack size.
-
- This is not a problem because the rules will work with the last object on the stack
- to pop the object off the stack, a new rule mapped to portlet-app/portlet would
- need to be added that all it did was pop the current object at the portlet end element
- */
- digester.addRule("portlet-app/portlet/portlet-name", new PortletRule(app));
-
- /*
- digester.addRule("portlet-app/portlet/title", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/contributor", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/creator", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/coverage", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/description", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/format", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/identifier", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/language", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/publisher", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/relation", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/right", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/source", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/subject", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/type", new LocalizedFieldRule());
- digester.addRule("portlet-app/portlet/metadata", new LocalizedFieldRule());
- */
+ digester.addRule("portlet-app/portlet/portlet-name", new PortletRule(app));
digester.addRuleSet(new MetadataRuleSet("portlet-app/portlet/"));
digester.parse(reader);
@@ -116,7 +68,6 @@
{
String msg = "Could not unmarshal \"" + pathPortletXML+"\". "+t.toString();
log.error(msg, t);
- //throw new PortletApplicationException(msg, t);
}
return result;
---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org