[ 
https://jira.duraspace.org/browse/DS-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=22553#comment-22553
 ] 

Mark Diggory commented on DS-1004:
----------------------------------

To clarify what I dislike about this patch, it has to do with extending the 
HandleTypeMatcher to introduce "Wihdrawn" as a type of DSpaceObject when it is 
a state of a DSpace Object.



Index: src/main/java/org/dspace/app/xmlui/aspect/general/HandleTypeMatcher.java
===================================================================
--- src/main/java/org/dspace/app/xmlui/aspect/general/HandleTypeMatcher.java    
(revision 6579)
+++ src/main/java/org/dspace/app/xmlui/aspect/general/HandleTypeMatcher.java    
(working copy)
@@ -17,6 +17,7 @@
 import org.apache.cocoon.sitemap.PatternException;
 import org.dspace.app.xmlui.utils.HandleUtil;
 import org.dspace.content.DSpaceObject;
+import org.dspace.content.Item;
 import org.dspace.core.Constants;
 
 /**
@@ -37,10 +38,13 @@
     public static final String COMMUNITY_EXPRESSION = "community";
 
     /** The collection expression */
-    public static final String COLLECITON_EXPRESSION = "collection";
+    public static final String COLLECTION_EXPRESSION = "collection";
 
     /** The item expression */
     public static final String ITEM_EXPRESSION = "item";
+    
+    /** The withdrawn item expression */
+    public static final String WITHDRAWN_EXPRESSION = "withdrawn";
 
     /**
      * Match the encoded DSpaceObject against a specified type.
@@ -58,8 +62,9 @@
         for (String expression : expressions)
         {
             if (!(COMMUNITY_EXPRESSION.equals(expression)
-                    || COLLECITON_EXPRESSION.equals(expression) || 
ITEM_EXPRESSION
-                    .equals(expression)))
+                  || COLLECTION_EXPRESSION.equals(expression) 
+                  || WITHDRAWN_EXPRESSION.equals(expression) 
+                  || ITEM_EXPRESSION.equals(expression)))
                
> Add optional reason text to tombstone page (XMLUI)
> --------------------------------------------------
>
>                 Key: DS-1004
>                 URL: https://jira.duraspace.org/browse/DS-1004
>             Project: DSpace
>          Issue Type: Improvement
>          Components: XMLUI
>    Affects Versions: 1.8.0
>            Reporter: Richard Rodgers
>            Assignee: Richard Rodgers
>             Fix For: 1.8.0
>
>         Attachments: tombstone_dspace-xmlui-api.patch, 
> tombstone_dspace-xmlui-webapp.patch
>
>
> Companion to DS-587 - that is, a configurable tombstone 'reason'. The primary 
> difference is that XMLUI never displayed a tombstone page at all,
> just the (erroneous) RestrictedItem page. This restores the parity between 
> JSPUI and XMLUI

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.duraspace.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Dspace-devel mailing list
Dspace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to