diff --git a/ccm-cms/src/WEB-INF/resources/cms-item-adapters.xml b/ccm-cms/src/WEB-INF/resources/cms-item-adapters.xml index 7fd2fd6b2..a9f768a73 100755 --- a/ccm-cms/src/WEB-INF/resources/cms-item-adapters.xml +++ b/ccm-cms/src/WEB-INF/resources/cms-item-adapters.xml @@ -7,7 +7,7 @@ - + @@ -32,6 +32,24 @@ + + + + + + + + + + + + + + + + + + diff --git a/ccm-cms/src/com/arsdigita/cms/dispatcher/ItemXML.java b/ccm-cms/src/com/arsdigita/cms/dispatcher/ItemXML.java index 17eec08b3..9bb9a201a 100755 --- a/ccm-cms/src/com/arsdigita/cms/dispatcher/ItemXML.java +++ b/ccm-cms/src/com/arsdigita/cms/dispatcher/ItemXML.java @@ -21,8 +21,8 @@ package com.arsdigita.cms.dispatcher; import com.arsdigita.cms.CMS; import com.arsdigita.cms.ContentItem; +import com.arsdigita.cms.ContentItemXMLRenderer; import com.arsdigita.dispatcher.RequestContext; -import com.arsdigita.domain.DomainObjectXMLRenderer; import com.arsdigita.xml.Document; import com.arsdigita.xml.Element; import java.io.IOException; @@ -58,8 +58,8 @@ public class ItemXML extends ResourceHandlerImpl { Element content = new Element("cms:item", CMS.CMS_XML_NS); - DomainObjectXMLRenderer renderer = - new DomainObjectXMLRenderer(content); + ContentItemXMLRenderer renderer = + new ContentItemXMLRenderer(content); renderer.setWrapAttributes(true); renderer.setWrapRoot(false); diff --git a/ccm-cms/src/com/arsdigita/cms/dispatcher/SimpleXMLGenerator.java b/ccm-cms/src/com/arsdigita/cms/dispatcher/SimpleXMLGenerator.java index aa25d27a8..5b3ea8ac7 100755 --- a/ccm-cms/src/com/arsdigita/cms/dispatcher/SimpleXMLGenerator.java +++ b/ccm-cms/src/com/arsdigita/cms/dispatcher/SimpleXMLGenerator.java @@ -22,6 +22,7 @@ package com.arsdigita.cms.dispatcher; import com.arsdigita.bebop.PageState; import com.arsdigita.cms.CMS; import com.arsdigita.cms.ContentItem; +import com.arsdigita.cms.ContentItemXMLRenderer; import com.arsdigita.cms.ContentSection; import com.arsdigita.cms.SecurityManager; import com.arsdigita.cms.UserDefinedContentItem; @@ -29,7 +30,6 @@ import com.arsdigita.cms.util.GlobalizationUtil; import com.arsdigita.domain.DataObjectNotFoundException; import com.arsdigita.domain.DomainObjectFactory; import com.arsdigita.domain.DomainObjectTraversal; -import com.arsdigita.domain.DomainObjectXMLRenderer; import com.arsdigita.domain.SimpleDomainObjectTraversalAdapter; import com.arsdigita.kernel.Kernel; import com.arsdigita.kernel.Party; @@ -144,8 +144,8 @@ public class SimpleXMLGenerator implements XMLGenerator { // This is the preferred method Element content = startElement(useContext); - DomainObjectXMLRenderer renderer = - new DomainObjectXMLRenderer(content); + ContentItemXMLRenderer renderer = + new ContentItemXMLRenderer(content); renderer.setWrapAttributes(true); renderer.setWrapRoot(false); diff --git a/ccm-cms/src/com/arsdigita/cms/search/XMLContentProvider.java b/ccm-cms/src/com/arsdigita/cms/search/XMLContentProvider.java index 76ef06e90..e987ed289 100755 --- a/ccm-cms/src/com/arsdigita/cms/search/XMLContentProvider.java +++ b/ccm-cms/src/com/arsdigita/cms/search/XMLContentProvider.java @@ -20,12 +20,12 @@ package com.arsdigita.cms.search; import com.arsdigita.cms.CMS; +import com.arsdigita.cms.ContentItemXMLRenderer; import com.arsdigita.cms.util.GlobalizationUtil; import com.arsdigita.search.ContentType; import com.arsdigita.search.ContentProvider; import com.arsdigita.domain.DomainObject; -import com.arsdigita.domain.DomainObjectXMLRenderer; import com.arsdigita.xml.Document; import com.arsdigita.xml.Element; import com.arsdigita.util.UncheckedWrapperException; @@ -60,8 +60,8 @@ public class XMLContentProvider implements ContentProvider { public byte[] getBytes() { Element root = new Element("cms:item", CMS.CMS_XML_NS); - DomainObjectXMLRenderer renderer = - new DomainObjectXMLRenderer(root); + ContentItemXMLRenderer renderer = + new ContentItemXMLRenderer(root); renderer.setWrapAttributes(true); renderer.walk(m_obj, m_adapterContext); diff --git a/ccm-cms/src/com/arsdigita/cms/ui/FolderContent.java b/ccm-cms/src/com/arsdigita/cms/ui/FolderContent.java index 36e68f889..964ba6a8a 100755 --- a/ccm-cms/src/com/arsdigita/cms/ui/FolderContent.java +++ b/ccm-cms/src/com/arsdigita/cms/ui/FolderContent.java @@ -22,11 +22,11 @@ import com.arsdigita.bebop.PageState; import com.arsdigita.bebop.SimpleComponent; import com.arsdigita.cms.CMS; import com.arsdigita.cms.ContentItem; +import com.arsdigita.cms.ContentItemXMLRenderer; import com.arsdigita.cms.ContentSection; import com.arsdigita.cms.ContentSectionCollection; import com.arsdigita.cms.Folder; import com.arsdigita.cms.dispatcher.SimpleXMLGenerator; -import com.arsdigita.domain.DomainObjectXMLRenderer; import com.arsdigita.xml.Element; @@ -205,7 +205,7 @@ public class FolderContent extends SimpleComponent { log.debug("Rendering item '"+item.getName()+"'."); Element itemElement = parent.newChildElement(TAG_ITEM, CMS.CMS_XML_NS); - DomainObjectXMLRenderer renderer = new DomainObjectXMLRenderer(itemElement); + ContentItemXMLRenderer renderer = new ContentItemXMLRenderer(itemElement); // not sure these are necessary renderer.setWrapAttributes(true); diff --git a/ccm-cms/src/com/arsdigita/cms/ui/portlet/ContentItemPortletRenderer.java b/ccm-cms/src/com/arsdigita/cms/ui/portlet/ContentItemPortletRenderer.java index c4dcb59e0..aefb34324 100755 --- a/ccm-cms/src/com/arsdigita/cms/ui/portlet/ContentItemPortletRenderer.java +++ b/ccm-cms/src/com/arsdigita/cms/ui/portlet/ContentItemPortletRenderer.java @@ -22,11 +22,11 @@ import com.arsdigita.bebop.PageState; import com.arsdigita.bebop.portal.AbstractPortletRenderer; import com.arsdigita.cms.CMS; import com.arsdigita.cms.ContentItem; +import com.arsdigita.cms.ContentItemXMLRenderer; import com.arsdigita.cms.SecurityManager; import com.arsdigita.cms.dispatcher.SimpleXMLGenerator; import com.arsdigita.cms.portlet.ContentItemPortlet; import com.arsdigita.dispatcher.AccessDeniedException; -import com.arsdigita.domain.DomainObjectXMLRenderer; import com.arsdigita.kernel.Kernel; import com.arsdigita.kernel.Party; import com.arsdigita.kernel.permissions.PermissionDescriptor; @@ -93,8 +93,7 @@ public class ContentItemPortletRenderer extends AbstractPortletRenderer { Element contentItem = content.newChildElement("cms:item", CMS.CMS_XML_NS); - DomainObjectXMLRenderer renderer = new DomainObjectXMLRenderer( - contentItem); + ContentItemXMLRenderer renderer = new ContentItemXMLRenderer(contentItem); renderer.setWrapAttributes(true); renderer.setWrapRoot(false); @@ -106,8 +105,12 @@ public class ContentItemPortletRenderer extends AbstractPortletRenderer { public String getCacheKey(PageState state) { ContentItem item = m_portlet.getContentItem(); - if( null == item ) return ""; - if( item.isLive() ) return item.getPublicVersion().getOID().toString(); + if( null == item ) { + return ""; + } + if( item.isLive() ) { + return item.getPublicVersion().getOID().toString(); + } // Don't cache it if it's not live return ""; diff --git a/ccm-core/src/com/arsdigita/domain/DomainObjectTraversal.java b/ccm-core/src/com/arsdigita/domain/DomainObjectTraversal.java index eaa403357..6bee0d1ee 100755 --- a/ccm-core/src/com/arsdigita/domain/DomainObjectTraversal.java +++ b/ccm-core/src/com/arsdigita/domain/DomainObjectTraversal.java @@ -253,7 +253,7 @@ public abstract class DomainObjectTraversal { walk(adapter, obj, "/object", context, null); } - private void walk(final DomainObjectTraversalAdapter adapter, + protected void walk(final DomainObjectTraversalAdapter adapter, final DomainObject obj, final String path, final String context,