Update of
/var/cvs/contributions/CMSContainer/cmsc/portlets/src/java/com/finalist/cmsc/portlets
In directory
james.mmbase.org:/tmp/cvs-serv2545/cmsc/portlets/src/java/com/finalist/cmsc/portlets
Modified Files:
AbstractContentPortlet.java
Log Message:
CMSC-1028 - Better handling of missing content elements in portlets - Improved
thread listing view - Added thread dump cleaning line.
See also:
http://cvs.mmbase.org/viewcvs/contributions/CMSContainer/cmsc/portlets/src/java/com/finalist/cmsc/portlets
See also: http://www.mmbase.org/jira/browse/CMSC-1028
Index: AbstractContentPortlet.java
===================================================================
RCS file:
/var/cvs/contributions/CMSContainer/cmsc/portlets/src/java/com/finalist/cmsc/portlets/AbstractContentPortlet.java,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -b -r1.12 -r1.13
--- AbstractContentPortlet.java 21 Jun 2008 15:52:35 -0000 1.12
+++ AbstractContentPortlet.java 25 Jul 2008 12:17:20 -0000 1.13
@@ -12,23 +12,9 @@
import java.io.IOException;
import java.text.DateFormat;
import java.text.SimpleDateFormat;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Date;
-import java.util.Enumeration;
-import java.util.HashMap;
-import java.util.Map;
-import java.util.Set;
-
-import javax.portlet.ActionRequest;
-import javax.portlet.ActionResponse;
-import javax.portlet.PortletException;
-import javax.portlet.PortletMode;
-import javax.portlet.PortletModeException;
-import javax.portlet.PortletPreferences;
-import javax.portlet.PortletSession;
-import javax.portlet.RenderRequest;
-import javax.portlet.RenderResponse;
+import java.util.*;
+
+import javax.portlet.*;
import org.apache.commons.lang.StringUtils;
import org.mmbase.bridge.Cloud;
@@ -240,27 +226,21 @@
protected void setMetaData(RenderRequest req, String elementId) {
try {
ContentElement element =
ContentRepository.getContentElement(elementId);
-
+ if (element != null) { //When element not found, skip it.
PortletFragment portletFragment = getPortletFragment(req);
- portletFragment
- .addHeaderResource(new MetaHeaderResource(true, "title",
element.getTitle()));
- portletFragment.addHeaderResource(new MetaHeaderResource(true,
"subject", element
- .getKeywords()));
- portletFragment.addHeaderResource(new MetaHeaderResource(true,
"date", formatDate(element
- .getCreationdate())));
+ portletFragment.addHeaderResource(new MetaHeaderResource(true,
"title", element.getTitle()));
+ portletFragment.addHeaderResource(new MetaHeaderResource(true,
"subject", element.getKeywords()));
+ portletFragment.addHeaderResource(new MetaHeaderResource(true,
"date", formatDate(element.getCreationdate())));
portletFragment.addHeaderResource(new MetaHeaderResource(true,
"identifier", elementId));
portletFragment.addHeaderResource(new MetaHeaderResource(true,
"coverage",
formatDate(element.getPublishdate()) + " - " +
formatDate(element.getExpirydate())));
}
- catch (RuntimeException re) {
- if (re.getMessage().startsWith("Node not found")) {
- getLogger().debug("Node not found", re);
}
- else {
+ catch (RuntimeException re) {
getLogger().error(re);
}
}
- }
+
private String formatDate(Date date) {
if (date == null) {
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs
