[gatein-commits] gatein SVN: r6967 - portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/api/impl/content.

[do-not-reply at jboss.org]

Author: mwringe
Date: 2011-08-01 17:47:46 -0400 (Mon, 01 Aug 2011)
New Revision: 6967

Modified:
   portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/api/impl/content/CategoryImpl.java
Log:
Use the id of the application, not the id of the application content (would otherwise fail if the category name doesn't equal the web application's name).

Modified: portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/api/impl/content/CategoryImpl.java
===================================================================
--- portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/api/impl/content/CategoryImpl.java	2011-08-01 15:07:06 UTC (rev 6966)
+++ portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/api/impl/content/CategoryImpl.java	2011-08-01 21:47:46 UTC (rev 6967)
@@ -82,6 +82,7 @@
       Class<Content> contentClass = getContentClassFor(type);
       if (Gadget.class.isAssignableFrom(contentClass))
       {
+    	 //TODO: for a gadget, it should probably be using application.getID instead of getContentId
          return gateIn.gadgetId(application.getContentId());
       }
       else if (org.gatein.api.content.WSRP.class.isAssignableFrom(contentClass))
@@ -90,7 +91,7 @@
       }
       else if (org.gatein.api.content.Portlet.class.isAssignableFrom(contentClass))
       {
-         return gateIn.parsePortletId(application.getContentId());
+         return gateIn.parsePortletId(application.getId());
       }
       else
       {