Reverting previous commit and going back to revision 5115.

git-svn-id: http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2@5899 9c30dcfa-912a-0410-8fc2-9e0234be79fd

dspace-api/pom.xml

@@ -13,7 +13,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -39,9 +39,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace</url>
    </scm>
 
    <!--

dspace-api/src/main/java/org/dspace/app/bulkedit/BulkEditChange.java

@@ -66,11 +66,17 @@ public class BulkEditChange
     /** The ArrayList of the complete set of new values (constant + adds) */
     private ArrayList<DCValue> complete;
 
-    /** The Arraylist of old collections the item has been moved from */
-    private ArrayList<Collection> oldOwningCollections;
+    /** The Arraylist of old collections the item used to be mapped to */
+    private ArrayList<Collection> oldMappedCollections;
 
-    /** The Arraylist of new collections the item has been moved into */
-    private ArrayList<Collection> newOwningCollections;
+    /** The Arraylist of new collections the item has been mapped into */
+    private ArrayList<Collection> newMappedCollections;
+
+    /** The old owning collection */
+    private Collection oldOwningCollection;
+
+    /** The new owning collection */
+    private Collection newOwningCollection;
 
     /** Is this a new item */
     private boolean newItem;
@@ -88,14 +94,16 @@ public class BulkEditChange
         item = null;
         newItem = true;
         empty = true;
+        oldOwningCollection = null;
+        newOwningCollection = null;
 
         // Initialise the arrays
         adds = new ArrayList<DCValue>();
         removes = new ArrayList<DCValue>();
         constant = new ArrayList<DCValue>();
         complete = new ArrayList<DCValue>();
-        oldOwningCollections = new ArrayList<Collection>();
-        newOwningCollections = new ArrayList<Collection>();
+        oldMappedCollections = new ArrayList<Collection>();
+        newMappedCollections = new ArrayList<Collection>();
     }
 
     /**
@@ -115,8 +123,8 @@ public class BulkEditChange
         removes = new ArrayList<DCValue>();
         constant = new ArrayList<DCValue>();
         complete = new ArrayList<DCValue>();
-        oldOwningCollections = new ArrayList<Collection>();
-        newOwningCollections = new ArrayList<Collection>();
+        oldMappedCollections = new ArrayList<Collection>();
+        newMappedCollections = new ArrayList<Collection>();
     }
 
     /**
@@ -168,29 +176,76 @@ public class BulkEditChange
     }
 
     /**
-     * Add a new owning Collection
+     * Add a new mapped Collection
      *
-     * @param c The new owning Collection
+     * @param c The new mapped Collection
      */
-    public void registerNewOwningCollection(Collection c)
+    public void registerNewMappedCollection(Collection c)
     {
-        // Add the new owning Collection\
-        newOwningCollections.add(c);
+        // Add the new owning Collection
+        newMappedCollections.add(c);
         empty = false;
     }
 
     /**
-     * Add an old owning Collection
+     * Add an old mapped Collection
      *
-     * @param c The old owning Collection
+     * @param c The old mapped Collection
      */
-    public void registerOldOwningCollection(Collection c)
+    public void registerOldMappedCollection(Collection c)
     {
-        // Add the old owning Collection\
-        oldOwningCollections.add(c);
+        // Add the old owning Collection (if it isn't there already, or is an old collection)
+        boolean found = false;
+
+        if ((this.getOldOwningCollection() != null) &&
+            (this.getOldOwningCollection().getHandle().equals(c.getHandle())))
+        {
+            found = true;
+        }
+
+        for (Collection collection : oldMappedCollections)
+        {
+            if (collection.getHandle().equals(c.getHandle()))
+            {
+                found = true;
+            }
+        }
+
+        if (!found)
+        {
+            oldMappedCollections.add(c);
+            empty = false;
+        }
+    }
+
+    /**
+     * Register a change to the owning collection
+     *
+     * @param oldC The old owning collection
+     * @param newC The new owning collection
+     */
+    public void changeOwningCollection(Collection oldC, Collection newC)
+    {
+        // Store the old owning collection
+        oldOwningCollection = oldC;
+
+        // Store the new owning collection
+        newOwningCollection = newC;
         empty = false;
     }
 
+    /**
+     * Set the owning collection of an item
+     *
+     * @param newC The new owning collection
+     */
+    public void setOwningCollection(Collection newC)
+    {
+        // Store the new owning collection
+        newOwningCollection = newC;
+        //empty = false;
+    }
+
     /**
      * Get the DSpace Item that these changes are applicable to.
      *
@@ -247,25 +302,47 @@ public class BulkEditChange
     }
 
     /**
-     * Get the list of new owning Collections
+     * Get the list of new mapped Collections
      *
-     * @return the list of new owning collections
+     * @return the list of new mapped collections
      */
-    public ArrayList<Collection> getNewOwningCollections()
+    public ArrayList<Collection> getNewMappedCollections()
     {
         // Return the array
-        return newOwningCollections;
+        return newMappedCollections;
     }
 
     /**
-     * Get the list of old owning Collections
+     * Get the list of old mapped Collections
      *
-     * @return the list of old owning collections
+     * @return the list of old mapped collections
      */
-    public ArrayList<Collection> getOldOwningCollections()
+    public ArrayList<Collection> getOldMappedCollections()
     {
         // Return the array
-        return oldOwningCollections;
+        return oldMappedCollections;
+    }
+
+    /**
+     * Get the old owning collection
+     *
+     * @return the old owning collection
+     */
+    public Collection getOldOwningCollection()
+    {
+        // Return the old owning collection
+        return oldOwningCollection;
+    }
+
+    /**
+     * Get the new owning collection
+     *
+     * @return the new owning collection
+     */
+    public Collection getNewOwningCollection()
+    {
+        // Return the new owning collection
+        return newOwningCollection;
     }
 
     /**

dspace-api/src/main/java/org/dspace/app/bulkedit/DSpaceCSV.java

@@ -265,11 +265,19 @@ public class DSpaceCSV
         // Create the CSV line
         DSpaceCSVLine line = new DSpaceCSVLine(i.getID());
 
-        // Add in owning collections
+        // Add in owning collection
+        String owningCollectionHandle = i.getOwningCollection().getHandle();
+        line.add("collection", owningCollectionHandle);
+
+        // Add in any mapped collections
         Collection[] collections = i.getCollections();
         for (Collection c : collections)
         {
-           line.add("collection", c.getHandle());
+            // Only add if it is not the owning collection
+            if (!c.getHandle().equals(owningCollectionHandle))
+            {
+                line.add("collection", c.getHandle());   
+            }
         }
 
         // Populate it

dspace-api/src/main/java/org/dspace/app/bulkedit/MetadataImport.java

@@ -120,6 +120,10 @@ public class MetadataImport
                 {
                     // Get the item
                     Item item = Item.find(c, id);
+                    if (item == null)
+                    {
+                        throw new MetadataImportException("Unknown item ID " + id);
+                    }
                     BulkEditChange whatHasChanged = new BulkEditChange(item);
 
                     // Has it moved collection?
@@ -224,10 +228,19 @@ public class MetadataImport
                     }
 
                     // Record the addition to collections
+                    boolean first = true;
                     for (String handle : collections)
                     {
                         Collection extra = (Collection)HandleManager.resolveToObject(c, handle);
-                        whatHasChanged.registerNewOwningCollection(extra);
+                        if (first)
+                        {
+                            whatHasChanged.setOwningCollection(extra);   
+                        }
+                        else
+                        {
+                            whatHasChanged.registerNewMappedCollection(extra);
+                        }
+                        first = false;
                     }
 
                     // Create the new item?
@@ -454,10 +467,11 @@ public class MetadataImport
     }
 
     /**
-     * Compare changes between an items owning collections and what is in the CSV file
+     * Compare changes between an items owning collection and mapped collections
+     * and what is in the CSV file
      *
      * @param item The item in question
-     * @param collections The collection handles fro mthe CSV file
+     * @param collections The collection handles from the CSV file
      * @param actualCollections The Collections from the actual item
      * @param bechange The bulkedit change object for this item
      * @param change Whether or not to actuate a change
@@ -473,90 +487,138 @@ public class MetadataImport
                          BulkEditChange bechange,
                          boolean change) throws SQLException, AuthorizeException, IOException, MetadataImportException
     {
-        // First, loop through the strings from the CSV
+        // First, check the owning collection (as opposed to mapped collections) is the same of changed
+        String oldOwner = item.getOwningCollection().getHandle();
+        String newOwner = collections.get(0);
+        // Resolve the handle to the collection
+        Collection newCollection = (Collection)HandleManager.resolveToObject(c, newOwner);
+
+        // Check it resolved OK
+        if (newCollection == null)
+        {
+            throw new MetadataImportException("'" + newOwner + "' is not a Collection! You must specify a valid collection ID");
+        }
+
+        if (!oldOwner.equals(newOwner))
+        {
+            // Register the old and new owning collections
+            bechange.changeOwningCollection(item.getOwningCollection(), (Collection)HandleManager.resolveToObject(c, newOwner));
+        }
+
+        // Second, loop through the strings from the CSV of mapped collections
+        boolean first = true;
         for (String csvcollection : collections)
         {
-            // Look for it in the actual list of Collections
-            boolean found = false;
-            for (Collection collection : actualCollections)
+            // Ignore the first collection as this is the owning collection
+            if (!first)
             {
-                // Is it there?
-                if (csvcollection.equals(collection.getHandle()))
+                // Look for it in the actual list of Collections
+                boolean found = false;
+                for (Collection collection : actualCollections)
                 {
-                    found = true;
+                    if (collection.getID() != item.getOwningCollection().getID()) {
+                        // Is it there?
+                        if (csvcollection.equals(collection.getHandle()))
+                        {
+                            found = true;
+                        }
+                    }
                 }
-            }
 
-            // Was it found?
-            if (!found)
-            {
-                // Register than change
+                // Was it found?
                 DSpaceObject dso = HandleManager.resolveToObject(c, csvcollection);
                 if ((dso == null) || (dso.getType() != Constants.COLLECTION))
                 {
                     throw new MetadataImportException("Collection defined for item " + item.getID() +
                                                       " (" + item.getHandle() + ") is not a collection");
                 }
-                else
+                if (!found)
                 {
+                    // Register the new mapped collection
                     Collection col = (Collection)dso;
-                    bechange.registerNewOwningCollection(col);
-
-                    // Execute the change
-                    if (change)
-                    {
-                        // Add the item to the community
-                        col.addItem(item);
-                    }
+                    bechange.registerNewMappedCollection(col);
                 }
             }
+            first = false;
         }
 
-        // Second, loop through the strings from the current item
+        // Third, loop through the strings from the current item
         for (Collection collection : actualCollections)
         {
             // Look for it in the actual list of Collections
             boolean found = false;
+            first = true;
             for (String csvcollection : collections)
             {
-                // Is it there?
-                if (collection.getHandle().equals(csvcollection))
+                // Don't check the owning collection
+                if ((first) && (collection.getID() == item.getOwningCollection().getID()))
                 {
                     found = true;
                 }
+                else
+                {
+                    if (!first)
+                    {
+                        // Is it there?
+                        if (collection.getHandle().equals(csvcollection))
+                        {
+                            found = true;
+                        }
+
+                    }
+                }
+                first = false;
             }
 
             // Was it found?
             if (!found)
             {
-                // Reocrd that it isn't there any more
-                bechange.registerOldOwningCollection(collection);
-
-                // Execute the change
-                if (change)
-                {
-                    // Sanity check it is in another collection
-                    if (item.getCollections().length > 1)
-                    {
-                        collection.removeItem(item);
-                    }
-                    else
-                    {
-                        throw new MetadataImportException("Not removing item " + item.getHandle() +
-                                                          " from collection " + collection.getHandle() +
-                                                          " as it would leave it oprhaned!");
-                    }
-                }
+                // Record that it isn't there any more
+                bechange.registerOldMappedCollection(collection);
             }
         }
 
-        // Now check the owning_collection (item table) is the same as
-        // the first collection given in the CSV file entry
-        if ((change) && (!collections.get(0).equals(item.getOwningCollection().getHandle())))
+        // Process the changes
+        if (change)
         {
-            Collection owner = (Collection)HandleManager.resolveToObject(c, collections.get(0));
-            item.setOwningCollection(Collection.find(c, owner.getID()));
-            item.update();
+            // Remove old mapped collections
+            for (Collection c : bechange.getOldMappedCollections())
+            {
+                c.removeItem(item);
+            }
+
+            // Add to new owned collection
+            if (bechange.getNewOwningCollection() != null)
+            {
+                bechange.getNewOwningCollection().addItem(item);
+                item.setOwningCollection(bechange.getNewOwningCollection());
+                item.update();
+            }
+
+            // Remove from old owned collection (if still a member)
+            if (bechange.getOldOwningCollection() != null)
+            {
+                boolean found = false;
+                for (Collection c : item.getCollections())
+                {
+                    if (c.getID() == bechange.getOldOwningCollection().getID())
+                    {
+                        found = true;
+                    }
+                }
+
+                if (found)
+                {
+                    bechange.getOldOwningCollection().removeItem(item);
+                }
+            }
+
+            // Add to new mapped collections
+            for (Collection c : bechange.getNewMappedCollections())
+            {
+                c.addItem(item);
+            }
+
         }
     }
 
@@ -695,10 +757,11 @@ public class MetadataImport
             // Get the changes
             ArrayList<DCValue> adds = change.getAdds();
             ArrayList<DCValue> removes = change.getRemoves();
-            ArrayList<Collection> newCollections = change.getNewOwningCollections();
-            ArrayList<Collection> oldCollections = change.getOldOwningCollections();
+            ArrayList<Collection> newCollections = change.getNewMappedCollections();
+            ArrayList<Collection> oldCollections = change.getOldMappedCollections();
             if ((adds.size() > 0) || (removes.size() > 0) ||
-                (newCollections.size() > 0) || (oldCollections.size() > 0))
+                (newCollections.size() > 0) || (oldCollections.size() > 0) ||
+                (change.getNewOwningCollection() != null) || (change.getOldOwningCollection() != null))
             {
                 // Show the item
                 Item i = change.getItem();
@@ -727,34 +790,69 @@ public class MetadataImport
                 changeCounter++;
             }
 
-            // Show new collections
+            if (change.getNewOwningCollection() != null)
+            {
+                Collection c = change.getNewOwningCollection();
+                if (c != null)
+                {
+                    String cHandle = c.getHandle();
+                    String cName = c.getName();
+                    if (!changed)
+                    {
+                        System.out.print(" + New owning collection (" + cHandle + "): ");
+                    }
+                    else
+                    {
+                        System.out.print(" + New owning collection  (" + cHandle + "): ");
+                    }
+                    System.out.println(cName);
+                }
+
+                c = change.getOldOwningCollection();
+                if (c != null)
+                {
+                    String cHandle = c.getHandle();
+                    String cName = c.getName();
+                    if (!changed)
+                    {
+                        System.out.print(" + Old owning collection (" + cHandle + "): ");
+                    }
+                    else
+                    {
+                        System.out.print(" + Old owning collection  (" + cHandle + "): ");
+                    }
+                    System.out.println(cName);
+                }
+            }
+
+            // Show new mapped collections
             for (Collection c : newCollections)
             {
                 String cHandle = c.getHandle();
                 String cName = c.getName();
                 if (!changed)
                 {
-                    System.out.print(" + Add to collection (" + cHandle + "): ");
+                    System.out.print(" + Map to collection (" + cHandle + "): ");
                 }
                 else
                 {
-                    System.out.print(" + Added to collection  (" + cHandle + "): ");
+                    System.out.print(" + Mapped to collection  (" + cHandle + "): ");
                 }
                 System.out.println(cName);
             }
 
-            // Show old collections
+            // Show old mapped collections
             for (Collection c : oldCollections)
             {
                 String cHandle = c.getHandle();
                 String cName = c.getName();
                 if (!changed)
                 {
-                    System.out.print(" + Remove from collection (" + cHandle + "): ");
+                    System.out.print(" + Um-map from collection (" + cHandle + "): ");
                 }
                 else
                 {
-                    System.out.print(" + Removed from collection  (" + cHandle + "): ");
+                    System.out.print(" + Un-mapped from collection  (" + cHandle + "): ");
                 }
                 System.out.println(cName);
             }

dspace-api/src/main/java/org/dspace/app/launcher/ScriptLauncher.java

@@ -113,7 +113,21 @@ public class ScriptLauncher
                 {
                     // Instantiate the class
                     Class target = null;
-                    String className = step.getChild("class").getValue();
+
+                    // Is it the special case 'dsrun' where the user provides the class name?
+                    String className;
+                    if ("dsrun".equals(request))
+                    {
+                        if (args.length < 2)
+                        {
+                            System.err.println("Error in launcher.xml: Missing class name");
+                            System.exit(1);
+                        }
+                        className = args[1];
+                    }
+                    else {
+                        className = step.getChild("class").getValue();
+                    }
                     try
                     {
                         target = Class.forName(className,
@@ -136,16 +150,20 @@ public class ScriptLauncher
                     {
                         passargs = false;
                     }
-                    if ((args.length == 1) || (!passargs))
+                    if ((args.length == 1) || (("dsrun".equals(request)) && (args.length == 2)) || (!passargs))
                     {
                         useargs = new String[0];
                     }
                     else
                     {
-                        String[] argsnew = new String[useargs.length - 1];
-                        for (int i = 1; i < useargs.length; i++)
+                        // The number of arguments to ignore
+                        // If dsrun is the command, ignore the next, as it is the class name not an arg
+                        int x = 1;
+                        if ("dsrun".equals(request)) x = 2;
+                        String[] argsnew = new String[useargs.length - x];
+                        for (int i = x; i < useargs.length; i++)
                         {
-                            argsnew[i - 1] = useargs[i];
+                            argsnew[i - x] = useargs[i];
                         }
                         useargs = argsnew;
                     }
@@ -176,7 +194,7 @@ public class ScriptLauncher
                     // Establish a request related to the current session
                     // that will trigger the various request listeners
                     requestService.startRequest();
-                    
+
                     // Run the main() method
                     try
                     {
@@ -189,33 +207,33 @@ public class ScriptLauncher
                             System.out.print(" " + param);
                         }
                         System.out.println("");**/
-                        
+
                         Method main = target.getMethod("main", argTypes);
                         Object output = main.invoke(null, arguments);
 
                         // ensure we close out the request (happy request)
                         requestService.endRequest(null);
-                       
-                        
+
+
                     }
                     catch (Exception e)
                     {
                         // Failure occurred in the request so we destroy it
                         requestService.endRequest(e);
-                        
+
                     	if (kernelImpl != null)
                         {
                             kernelImpl.destroy();
                             kernelImpl = null;
                         }
-                    	
+
                         // Exceptions from the script are reported as a 'cause'
                         Throwable cause = e.getCause();
                         System.err.println("Exception: " + cause.getMessage());
                         cause.printStackTrace();
                         System.exit(1);
                     }
-                    
+
                 }
 
                 // Destroy the service kernel
@@ -224,7 +242,7 @@ public class ScriptLauncher
                     kernelImpl.destroy();
                     kernelImpl = null;
                 }
-                
+
                 System.exit(0);
             }
         }
@@ -235,7 +253,7 @@ public class ScriptLauncher
             kernelImpl.destroy();
             kernelImpl = null;
         }
-        
+
         // The command wasn't found
         System.err.println("Command not found: " + args[0]);
         display();

dspace-api/src/main/java/org/dspace/checker/DailyReportEmailer.java

@@ -89,7 +89,7 @@ public class DailyReportEmailer
      * Send the report through email.
      * 
      * @param attachment
-     *            the file conntaing the report
+     *            the file containing the report
      * @param numberOfBitstreams
      *            the number of bitstreams reported
      * 
@@ -117,7 +117,7 @@ public class DailyReportEmailer
         // create the first part of the email
         BodyPart messageBodyPart = new MimeBodyPart();
         messageBodyPart
-                .setText("This is the checksum checker report see attachement for details \n"
+                .setText("This is the checksum checker report see attachment for details \n"
                         + numberOfBitstreams
                         + " Bitstreams found with POSSIBLE issues");
         multipart.addBodyPart(messageBodyPart);
@@ -183,7 +183,7 @@ public class DailyReportEmailer
                         "c",
                         "Changed",
                         false,
-                        "Send E-mail report for all bitstrems where checksum has been changed for today");
+                        "Send E-mail report for all bitstreams where checksum has been changed for today");
         options.addOption("a", "All", false, "Send all E-mail reports");
 
         options.addOption("u", "Unchecked", false,

dspace-api/src/main/java/org/dspace/content/DCDate.java

@@ -384,7 +384,7 @@ public class DCDate
      */
     public int getYear()
     {
-        return getLocalCalendar() == null ? -1 : localCalendar.get(Calendar.YEAR);
+        return ((getLocalCalendar() == null) || (!withinGranularity(DateGran.YEAR))) ? -1 : localCalendar.get(Calendar.YEAR);
     }
                     
     /**
@@ -394,7 +394,7 @@ public class DCDate
      */
     public int getMonth()
     {
-        return getLocalCalendar() == null ? -1 : localCalendar.get(Calendar.MONTH) + 1;
+        return ((getLocalCalendar() == null) || (!withinGranularity(DateGran.MONTH))) ? -1 : localCalendar.get(Calendar.MONTH) + 1;
     }
 
     /**
@@ -404,7 +404,7 @@ public class DCDate
      */
     public int getDay()
     {
-        return getLocalCalendar() == null ? -1 : localCalendar.get(Calendar.DAY_OF_MONTH);
+        return ((getLocalCalendar() == null) || (!withinGranularity(DateGran.DAY))) ? -1 : localCalendar.get(Calendar.DAY_OF_MONTH);
     }
 
     /**
@@ -524,6 +524,52 @@ public class DCDate
 
     }
 
+    /**
+     * Test if the requested level of granularity is within that of the date.
+     *
+     * @param dg
+     *              The requested level of granularity.
+     * @return
+     *              true or false.
+     *
+     */
+    private boolean withinGranularity(DateGran dg)
+    {
+        if (granularity == DateGran.TIME)
+        {
+            if ((dg == DateGran.TIME) || (dg == DateGran.DAY) || (dg == DateGran.MONTH) || (dg == DateGran.YEAR))
+            {
+                return true;
+            }
+        }
+
+        if (granularity == DateGran.DAY)
+        {
+            if ((dg == DateGran.DAY) || (dg == DateGran.MONTH) || (dg == DateGran.YEAR))
+            {
+                return true;
+            }
+        }
+
+        if (granularity == DateGran.MONTH)
+        {
+            if ((dg == DateGran.MONTH) || (dg == DateGran.YEAR))
+            {
+                return true;
+            }
+        }
+
+        if (granularity == DateGran.YEAR)
+        {
+            if (dg == DateGran.YEAR)
+            {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     /**
      * Simple test program
      * Usage:  java org.dspace.content.DCdate [DCDate | -l yyyy [mm [dd ..]]] ]

dspace-api/src/main/java/org/dspace/content/authority/LCNameAuthority.java

@@ -262,6 +262,8 @@ public class LCNameAuthority implements ChoiceAuthority
         private int hits = -1;
         private String textValue = null;
         private String name = null;
+        private String oname = null;
+        private String bname = null;
         private String lccn = null;
         private String lastTag = null;
         private String lastCode = null;
@@ -332,6 +334,10 @@ public class LCNameAuthority implements ChoiceAuthority
                     // 100.a is the personal name
                     else if (lastTag.equals("100") && lastCode.equals("a"))
                         name = textValue;
+
+                        if (lastTag.equals("100") && lastCode.equals("d") && (name != null)) 
+                        name = name+"  "+textValue; 
+                         
                 }
             }
         }

dspace-api/src/main/java/org/dspace/content/packager/AbstractMETSIngester.java

@@ -203,7 +203,7 @@ public abstract class AbstractMETSIngester
             METSManifest manifest = null;
             wi = WorkspaceItem.create(context, collection, useTemplate);
             Item item = wi.getItem();
-            Bundle contentBundle = item.createBundle(Constants.CONTENT_BUNDLE_NAME);
+            Bundle contentBundle = null;
             Bundle mdBundle = null;
             ZipEntry ze;
             while ((ze = zip.getNextEntry()) != null)
@@ -238,6 +238,11 @@ public abstract class AbstractMETSIngester
                 }
                 else
                 {
+                	// we need to create the bundle only the first time
+                	if (contentBundle == null)
+                	{
+                		contentBundle = item.createBundle(Constants.CONTENT_BUNDLE_NAME);
+                	}
                     bs = contentBundle.createBitstream(new PackageUtils.UnclosableInputStream(zip));
                     bs.setSource(fname);
                     bs.setName(fname);

dspace-api/src/main/java/org/dspace/eperson/Group.java

@@ -479,7 +479,7 @@ public class Group extends DSpaceObject
         // However, we only do this is we are looking up the special groups
         // of the current user, as we cannot look up the special groups
         // of a user who is not logged in.
-        if ((c.getCurrentUser() != null) && (c.getCurrentUser().getID() == e.getID()))
+        if ((c.getCurrentUser() == null) || (((c.getCurrentUser() != null) && (c.getCurrentUser().getID() == e.getID()))))
         {
             Group[] specialGroups = c.getSpecialGroups();
             for(Group special : specialGroups)

dspace-api/src/main/java/org/dspace/search/Harvest.java

@@ -289,7 +289,7 @@ public class Harvest
                         itemInfo.item = Item.find(context, itemInfo.itemID);
                     }
 
-                    if ((nonAnon) || (itemInfo.item == null))
+                    if ((nonAnon) || (itemInfo.item == null) || (withdrawn && itemInfo.withdrawn))
                     {
                         infoObjects.add(itemInfo);
                         itemCounter++;

dspace-api/src/main/java/org/dspace/submit/step/DescribeStep.java

@@ -516,10 +516,14 @@ public class DescribeStep extends AbstractProcessingStep
                     + "_first");
             lasts = getRepeatedParameter(request, metadataField, metadataField
                     + "_last");
-            auths = getRepeatedParameter(request, metadataField, metadataField
+
+            if(isAuthorityControlled)
+            {
+               auths = getRepeatedParameter(request, metadataField, metadataField
                     + "_authority");
-            confs = getRepeatedParameter(request, metadataField, metadataField
+               confs = getRepeatedParameter(request, metadataField, metadataField
                     + "_confidence");
+           }
 
             // Find out if the relevant "remove" button was pressed
             // TODO: These separate remove buttons are only relevant
@@ -535,8 +539,11 @@ public class DescribeStep extends AbstractProcessingStep
 
                 firsts.remove(valToRemove);
                 lasts.remove(valToRemove);
-                auths.remove(valToRemove);
-                confs.remove(valToRemove);
+                if(isAuthorityControlled)
+                {
+                   auths.remove(valToRemove);
+                   confs.remove(valToRemove);
+                }
             }
         }
         else
@@ -684,6 +691,11 @@ public class DescribeStep extends AbstractProcessingStep
                         .substring(removeButton.length()));
 
                 vals.remove(valToRemove);
+                if(isAuthorityControlled)
+                {
+                   auths.remove(valToRemove);
+                   confs.remove(valToRemove);
+                }
             }
         }
         else

dspace-api/src/main/resources/Messages.properties

@@ -291,10 +291,14 @@ jsp.dspace-admin.metadataimport.apply                           = Apply changes
 jsp.dspace-admin.metadataimport.unknownerror                    = An unknown error has occured
 jsp.dspace-admin.metadataimport.changesforitem                  = Changes for item
 jsp.dspace-admin.metadataimport.newitem                         = New item
-jsp.dspace-admin.metadataimport.addtocollection                 = Add to collection
-jsp.dspace-admin.metadataimport.addedtocollection               = Added to collection
-jsp.dspace-admin.metadataimport.removefromcollection            = Remove from collection
-jsp.dspace-admin.metadataimport.removedfromcollection           = Removed from collection
+jsp.dspace-admin.metadataimport.addtocollection                 = Map to collection
+jsp.dspace-admin.metadataimport.addedtocollection               = Mapped to collection
+jsp.dspace-admin.metadataimport.removefromcollection            = Un-map from collection
+jsp.dspace-admin.metadataimport.removedfromcollection           = Un-mapped from collection
+jsp.dspace-admin.metadataimport.addtoownercollection            = Add to new owning collection
+jsp.dspace-admin.metadataimport.addedtoownercollection          = Added to new owning collection
+jsp.dspace-admin.metadataimport.removefromownercollection       = Remove from owning collection
+jsp.dspace-admin.metadataimport.removedfromownercollection      = Removed from owning collection
 jsp.dspace-admin.metadataimport.add                             = Add
 jsp.dspace-admin.metadataimport.added                           = Added
 jsp.dspace-admin.metadataimport.remove                          = Remove
@@ -1073,6 +1077,21 @@ jsp.tools.edit-collection.form.label11                          = Step:
 jsp.tools.edit-collection.form.label12                          = Collection Administrators:
 jsp.tools.edit-collection.form.label13                          = Item template:
 jsp.tools.edit-collection.form.label14                          = Collection's Authorizations:
+jsp.tools.edit-collection.form.label15                          = Harvesting Settings
+jsp.tools.edit-collection.form.label16                          = Content Source
+jsp.tools.edit-collection.form.label17                          = This is a standard DSpace collection
+jsp.tools.edit-collection.form.label18                          = This collection harvests its content from an external source
+jsp.tools.edit-collection.form.label19                          = OAI Provider
+jsp.tools.edit-collection.form.label20                          = OAI Set Id
+jsp.tools.edit-collection.form.label21                          = Metadata Format
+jsp.tools.edit-collection.form.label21.select.qdc	            = Qualified Dublin Core
+jsp.tools.edit-collection.form.label21.select.dc				= Dublin Core
+jsp.tools.edit-collection.form.label21.select.dim               = DSpace Intermediate Format
+jsp.tools.edit-collection.form.label22                          = Content being Harvested
+jsp.tools.edit-collection.form.label23                          = Harvest metadata only.
+jsp.tools.edit-collection.form.label24                          = Harvest metadata and references to bitstreams (requires ORE support).
+jsp.tools.edit-collection.form.label25                          = Harvest metadata and bitstreams (requires ORE support).
+jsp.tools.edit-collection.form.label26                          = Last Harvest Result
 jsp.tools.edit-collection.form.label2                           = Short Description
 jsp.tools.edit-collection.form.label3                           = Introductory text (HTML):
 jsp.tools.edit-collection.form.label4                           = Copyright text (plain text):
@@ -1216,6 +1235,7 @@ jsp.tools.itemmap-info.button.continue                          = Continue
 jsp.tools.itemmap-info.heading                                  = Item Map Info
 jsp.tools.itemmap-info.msg.added                                = Add item {0}
 jsp.tools.itemmap-info.msg.none-selected                        = No items selected, none added.
+jsp.tools.itemmap-info.msg.none-removed                        = No items selected, none removed.
 # In the following 2 messages the only parameter is an item ID
 jsp.tools.itemmap-info.msg.remove                               = Remove item {0}
 jsp.tools.itemmap-info.title                                    = Item Map Info
@@ -1234,6 +1254,7 @@ jsp.tools.move-item.collection.from.msg                         = Collection to
 jsp.tools.move-item.collection.to.msg                           = Collection to move to
 jsp.tools.move-item.item.name.msg                               = Name of the item to be moved
 jsp.tools.move-item.title                                       = Moving an item
+jsp.tools.move-item.inheritpolicies                             = Inherit default policies of destination collection
 jsp.tools.upload-bitstream.info                                 = Select the bitstream to upload
 jsp.tools.upload-bitstream.title                                = Upload Bitstream
 jsp.tools.upload-bitstream.upload                               = Upload
@@ -1393,6 +1414,8 @@ org.dspace.eperson.Subscribe.authors
 org.dspace.eperson.Subscribe.id                                                 = ID:
 org.dspace.eperson.Subscribe.new-items                                          = New Items:
 org.dspace.eperson.Subscribe.title                                              = Title:
+org.dspace.statistics.util.LocationUtils.unknown-continent                      = Unknown Continent
+org.dspace.statistics.util.LocationUtils.unknown-country                        = Unknown Country
 org.dspace.workflow.WorkflowManager.step1                                       = It requires reviewing.
 org.dspace.workflow.WorkflowManager.step2                                       = The submission must be checked before inclusion in the archive.
 org.dspace.workflow.WorkflowManager.step3                                       = The metadata needs to be checked to ensure compliance with the collection's standards, and edited if necessary.

dspace-jspui/dspace-jspui-api/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-jspui</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-api</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-api</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-api</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-api</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-api</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-api</url>
    </scm>
 
    <dependencies>

dspace-jspui/dspace-jspui-api/src/main/java/org/dspace/app/webui/jsptag/ItemTag.java

@@ -731,7 +731,19 @@ public class ItemTag extends TagSupport
         {
         	Bundle[] bundles = item.getBundles("ORIGINAL");
 
-        	if (bundles.length == 0)
+        	boolean filesExist = false;
+            
+            for (Bundle bnd : bundles)
+            {
+            	filesExist = bnd.getBitstreams().length > 0;
+            	if (filesExist)
+            	{
+            		break;
+            	}
+            }
+            
+            // if user already has uploaded at least one file
+        	if (!filesExist)
         	{
         		out.println("<p>"
         				+ LocaleSupport.getLocalizedMessage(pageContext,

dspace-jspui/dspace-jspui-api/src/main/java/org/dspace/app/webui/servlet/AuthorityChooseServlet.java

@@ -81,8 +81,8 @@ public class AuthorityChooseServlet extends DSpaceServlet {
 //        };
 //
 //        Choices result = new Choices(testValues,start,testValues.length,Choices.CF_ACCEPTED,false);
-        Writer writer = response.getWriter();
         response.setContentType("text/xml; charset=\"utf-8\"");
+        Writer writer = response.getWriter();
         // borrow xalan's serializer to let us use SAX choice menu generator
         Properties props =
            OutputPropertiesFactory.getDefaultMethodProperties(Method.XML);

dspace-jspui/dspace-jspui-api/src/main/java/org/dspace/app/webui/servlet/admin/ItemMapServlet.java

@@ -198,72 +198,6 @@ public class ItemMapServlet extends DSpaceServlet
     		// show the page
     		JSPManager.showJSP(request, response, jspPage);
     	}
-    	/*
-    	 * else if( action.equals("add") ) { int itemID =
-    	 * UIUtil.getIntParameter(request, "item_id"); String handle =
-    	 * (String)request.getParameter("handle"); boolean error = true; Item
-    	 * itemToAdd = null;
-    	 * 
-    	 * if( itemID > 0 ) { itemToAdd = Item.find(context, itemID);
-    	 * 
-    	 * if( itemToAdd != null ) error = false; } else if(handle != null &&
-    	 * !handle.equals("")) { DSpaceObject
-    	 * dso=HandleManager.resolveToObject(context, handle);
-    	 * 
-    	 * if(dso != null && dso.getType() == Constants.ITEM) { itemToAdd =
-    	 * (Item)dso; error = false; } }
-    	 * 
-    	 * //FIXME: error handling! if( !error ) { String myTitle =
-    	 * itemToAdd.getDC("title",null,Item.ANY)[0].value; String ownerName =
-    	 * itemToAdd.getOwningCollection().getMetadata("name");
-    	 *  // hook up item, but first, does it belong already? TableRowIterator
-    	 * tri = DatabaseManager.query(context, "collection2item", "SELECT
-    	 * collection2item.* FROM collection2item WHERE " + "collection_id=" +
-    	 * myCollection.getID() + " AND item_id=" + itemToAdd.getID());
-    	 * 
-    	 * if(tri.hasNext()) { request.setAttribute("message", "Item is already
-    	 * part of that collection!"); } else { // Create mapping
-    	 * myCollection.addItem( itemToAdd );
-    	 *  // set up a nice 'done' message request.setAttribute("message",
-    	 * "Item added successfully: <br> " + myTitle + " <br> From Collection:
-    	 * <br> " + ownerName);
-    	 *  }
-    	 * 
-    	 * request.setAttribute("collection", myCollection);
-    	 *  // show this page when we're done jspPage = "itemmap-info.jsp";
-    	 *  // show the page JSPManager.showJSP(request, response, jspPage); }
-    	 * else { // Display an error } } else if( action.equals("Add Entire
-    	 * Collection") ) { int targetID = UIUtil.getIntParameter(request,
-    	 * "collection2import");
-    	 * 
-    	 * Collection targetCollection = Collection.find(context, targetID);
-    	 *  // get all items from that collection and add them if not // already
-    	 * added
-    	 *  // get all items to be added ItemIterator i =
-    	 * targetCollection.getItems(); Map toAdd = new HashMap(); String
-    	 * message = "";
-    	 * 
-    	 * while( i.hasNext() ) { Item myItem = i.next();
-    	 * 
-    	 * toAdd.put(new Integer(myItem.getID()), myItem); }
-    	 *  // now see what we already have, removing dups from the 'toAdd' list
-    	 * i = myCollection.getItems();
-    	 * 
-    	 * while( i.hasNext() ) { Item myItem = i.next(); Integer myKey = new
-    	 * Integer(myItem.getID());
-    	 *  // remove works even if key isn't present toAdd.remove(myKey); }
-    	 *  // what's left in toAdd should be added Iterator addKeys =
-    	 * toAdd.keySet().iterator();
-    	 * 
-    	 * while( addKeys.hasNext() ) { Item myItem =
-    	 * (Item)toAdd.get(addKeys.next()); myCollection.addItem(myItem);
-    	 * message += " <br> Added item ID: " + myItem.getID(); }
-    	 * 
-    	 * request.setAttribute("message", message);
-    	 * request.setAttribute("collection", myCollection);
-    	 *  // show this page when we're done jspPage = "itemmap-info.jsp";
-    	 *  // show the page JSPManager.showJSP(request, response, jspPage); }
-    	 */
     	else if (action.equals("Remove"))
     	{
     		// get item IDs to remove
@@ -271,29 +205,36 @@ public class ItemMapServlet extends DSpaceServlet
     		String message = "remove";
     		LinkedList removedItems = new LinkedList();
     		
-    		for (int j = 0; j < itemIDs.length; j++)
-    		{
-    			int i = Integer.parseInt(itemIDs[j]);
-    			removedItems.add(itemIDs[j]);
+                if (itemIDs == null)
+                {
+                        message = "none-removed";
+                }
+                else
+                {
+    	 		for (int j = 0; j < itemIDs.length; j++)
+	    		{
+    				int i = Integer.parseInt(itemIDs[j]);
+	    			removedItems.add(itemIDs[j]);
     			
-    			Item myItem = Item.find(context, i);
+    				Item myItem = Item.find(context, i);
     			
-    			// make sure item doesn't belong to this collection
-    			if (!myItem.isOwningCollection(myCollection))
-    			{
-    				myCollection.removeItem(myItem);
-    				try
+    				// make sure item doesn't belong to this collection
+    				if (!myItem.isOwningCollection(myCollection))
     				{
-    					IndexBrowse ib = new IndexBrowse(context);
-    					ib.itemChanged(myItem);
-    				}
-    				catch (BrowseException e)
-    				{
-    					log.error("caught exception: ", e);
-    					throw new ServletException(e);
+    					myCollection.removeItem(myItem);
+    					try
+    					{
+    						IndexBrowse ib = new IndexBrowse(context);
+    						ib.itemChanged(myItem);
+    					}
+    					catch (BrowseException e)
+    					{
+    						log.error("caught exception: ", e);
+    						throw new ServletException(e);
+    					}
     				}
     			}
-    		}
+		}
     		
     		request.setAttribute("message", message);
     		request.setAttribute("collection", myCollection);

dspace-jspui/dspace-jspui-api/src/main/java/org/dspace/app/webui/submit/step/JSPUploadStep.java

@@ -39,11 +39,7 @@
  */
 package org.dspace.app.webui.submit.step;
 
-import java.io.BufferedInputStream;
-import java.io.File;
-import java.io.FileInputStream;
 import java.io.IOException;
-import java.io.InputStream;
 import java.sql.SQLException;
 
 import javax.servlet.ServletException;
@@ -51,15 +47,12 @@ import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
 import org.apache.log4j.Logger;
-
 import org.dspace.app.util.DCInputsReader;
 import org.dspace.app.util.DCInputsReaderException;
-import org.dspace.app.util.Util;
+import org.dspace.app.util.SubmissionInfo;
 import org.dspace.app.webui.submit.JSPStep;
 import org.dspace.app.webui.submit.JSPStepManager;
-import org.dspace.app.webui.util.FileUploadRequest;
 import org.dspace.app.webui.util.JSPManager;
-import org.dspace.app.util.SubmissionInfo;
 import org.dspace.app.webui.util.UIUtil;
 import org.dspace.authorize.AuthorizeException;
 import org.dspace.content.Bitstream;
@@ -67,7 +60,6 @@ import org.dspace.content.BitstreamFormat;
 import org.dspace.content.Bundle;
 import org.dspace.content.Collection;
 import org.dspace.content.FormatIdentifier;
-import org.dspace.content.Item;
 import org.dspace.core.ConfigurationManager;
 import org.dspace.core.Context;
 import org.dspace.core.LogManager;
@@ -217,8 +209,19 @@ public class JSPUploadStep extends JSPStep
             Bundle[] bundles = subInfo.getSubmissionItem().getItem()
                     .getBundles("ORIGINAL");
 
+            boolean fileAlreadyUploaded = false;
+            
+            for (Bundle bnd : bundles)
+            {
+            	fileAlreadyUploaded = bnd.getBitstreams().length > 0;
+            	if (fileAlreadyUploaded)
+            	{
+            		break;
+            	}
+            }
+            
             // if user already has uploaded at least one file
-            if (bundles.length > 0)
+            if (fileAlreadyUploaded)
             {
                 // return to list of uploaded files
                 showUploadFileList(context, request, response, subInfo, true,
@@ -408,8 +411,23 @@ public class JSPUploadStep extends JSPStep
     {
         Bundle[] bundles = subInfo.getSubmissionItem().getItem().getBundles(
                 "ORIGINAL");
-
-        if (justUploaded || bundles.length > 0)
+        
+        boolean fileAlreadyUploaded = false;
+        
+        if (!justUploaded)
+        {
+	        for (Bundle bnd : bundles)
+	        {
+	        	fileAlreadyUploaded = bnd.getBitstreams().length > 0;
+	        	if (fileAlreadyUploaded)
+	        	{
+	        		break;
+	        	}
+	        }
+        }
+        
+        // if user already has uploaded at least one file
+        if (justUploaded || fileAlreadyUploaded)
         {
             // The item already has files associated with it.
             showUploadFileList(context, request, response, subInfo,

dspace-jspui/dspace-jspui-api/src/main/java/org/dspace/app/webui/util/JSPManager.java

@@ -45,6 +45,7 @@ import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
+import org.apache.commons.lang.StringEscapeUtils;
 import org.apache.log4j.Logger;
 import org.dspace.authorize.AuthorizeException;
 import org.dspace.core.Context;
@@ -163,7 +164,7 @@ public class JSPManager
             HttpServletResponse response, String badID, int type)
             throws ServletException, IOException
     {
-        request.setAttribute("bad.id", badID);
+        request.setAttribute("bad.id", StringEscapeUtils.escapeHtml(badID));
         response.setStatus(HttpServletResponse.SC_NOT_FOUND);
 
         if (type != -1)

dspace-jspui/dspace-jspui-webapp/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-jspui</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-webapp</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-webapp</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui/dspace-jspui-webapp</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-webapp</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-webapp</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui/dspace-jspui-webapp</url>
    </scm>
 
    <build>

dspace-jspui/dspace-jspui-webapp/src/main/webapp/dspace-admin/metadataimport-showchanges.jsp

@@ -99,11 +99,12 @@
                 // Get the changes
                 ArrayList<DCValue> adds = change.getAdds();
                 ArrayList<DCValue> removes = change.getRemoves();
-                ArrayList<Collection> newCollections = change.getNewOwningCollections();
-                ArrayList<Collection> oldCollections = change.getOldOwningCollections();
+                ArrayList<Collection> newCollections = change.getNewMappedCollections();
+                ArrayList<Collection> oldCollections = change.getOldMappedCollections();
                 boolean first = false;
                 if ((adds.size() > 0) || (removes.size() > 0) ||
-                    (newCollections.size() > 0) || (oldCollections.size() > 0))
+                    (newCollections.size() > 0) || (oldCollections.size() > 0) ||
+                    (change.getNewOwningCollection() != null) || (change.getOldOwningCollection() != null))
                 {
                     // Show the item
                     if (!change.isNewItem())
@@ -119,6 +120,60 @@
                     first = true;
                 }
 
+                // Show new owner collection
+                if (change.getNewOwningCollection() != null)
+                {
+                    Collection c = change.getNewOwningCollection();
+                    if (c != null)
+                    {
+                        String cHandle = c.getHandle();
+                        String cName = c.getName();
+                        if (!first)
+                        {
+                            %><tr><td bgcolor="white"></td><%
+                        }
+                        else
+                        {
+                            first = false;
+                        }
+                        if (!changed)
+                        {
+                            %><td bgcolor="#4E9258" style="font-size:10pt"><fmt:message key="jsp.dspace-admin.metadataimport.addtoownercollection"/></td><td bgcolor="4E9258" style="font-size:10pt">(<%= cHandle %>): <%= cName %></td></tr><%
+                        }
+                        else
+                        {
+                            %><td bgcolor="#4E9258" style="font-size:10pt"><fmt:message key="jsp.dspace-admin.metadataimport.addedtoownercollection"/></td><td bgcolor="4E9258" style="font-size:10pt">(<%= cHandle %>): <%= cName %></td></tr><%
+                        }
+                    }
+                }
+
+                // Show old owner collection
+                if (change.getOldOwningCollection() != null)
+                {
+                    Collection c = change.getOldOwningCollection();
+                    if (c != null)
+                    {
+                        String cHandle = c.getHandle();
+                        String cName = c.getName();
+                        if (!first)
+                        {
+                            %><tr><td bgcolor="white"></td><%
+                        }
+                        else
+                        {
+                            first = false;
+                        }
+                        if (!changed)
+                        {
+                            %><td bgcolor="#4E9258" style="font-size:10pt"><fmt:message key="jsp.dspace-admin.metadataimport.removefromownercollection"/></td><td bgcolor="4E9258" style="font-size:10pt">(<%= cHandle %>): <%= cName %></td></tr><%
+                        }
+                        else
+                        {
+                            %><td bgcolor="#4E9258" style="font-size:10pt"><fmt:message key="jsp.dspace-admin.metadataimport.removedfromownercollection"/></td><td bgcolor="4E9258" style="font-size:10pt">(<%= cHandle %>): <%= cName %></td></tr><%
+                        }
+                    }
+                }
+
                 // Show new collections
                 for (Collection c : newCollections)
                 {

dspace-jspui/dspace-jspui-webapp/src/main/webapp/tools/edit-collection.jsp

@@ -396,7 +396,7 @@
      =========================================================== --%>
      
      		<tr><td>&nbsp;</td></tr>
-            <tr><td colspan="2"><center><h3>Harvesting Settings</h3></center></td></tr>
+            <tr><td colspan="2"><center><h3><fmt:message key="jsp.tools.edit-collection.form.label15"/></h3></center></td></tr>
      
      		<%--
      		oaiProviderValue = hc.getOaiSource();
@@ -411,22 +411,22 @@
 			--%>
      
      		<tr>
-                <td class="submitFormLabel">Content Source</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label16"/></td>
                 <td>
-                	<input type="radio" value="source_normal" <% if (harvestLevelValue == 0) { %> checked="checked" <% } %> name="source">This is a standard DSpace collection</input><br/>
-                	<input type="radio" value="source_harvested" <% if (harvestLevelValue > 0) { %> checked="checked" <% } %> name="source">This collection harvests its content from an external source</input><br/>
+                	<input type="radio" value="source_normal" <% if (harvestLevelValue == 0) { %> checked="checked" <% } %> name="source"><fmt:message key="jsp.tools.edit-collection.form.label17"/></input><br/>
+                	<input type="radio" value="source_harvested" <% if (harvestLevelValue > 0) { %> checked="checked" <% } %> name="source"><fmt:message key="jsp.tools.edit-collection.form.label18"/></input><br/>
                 </td>
             </tr>
             <tr>
-                <td class="submitFormLabel">OAI Provider</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label19"/></td>
                 <td><input type="text" name="oai_provider" value="<%= oaiProviderValue %>" size="50" /></td>
             </tr>
             <tr>
-                <td class="submitFormLabel">OAI Set Id</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label20"/></td>
                 <td><input type="text" name="oai_setid" value="<%= oaiSetIdValue %>" size="50" /></td>
             </tr>   
             <tr>
-                <td class="submitFormLabel">Metadata Format</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label21"/></td>
                 <td>
                 	<select name="metadata_format" >
 	                	<%
@@ -436,21 +436,19 @@
 			            while (pe.hasMoreElements())
 			            {
 			                String key = (String)pe.nextElement();
+							
+							
 			                if (key.startsWith(metaString)) {
 			                	String metadataString = ConfigurationManager.getProperty(key);
 			                	String metadataKey = key.substring(metaString.length());
-			                	String displayName;
-			
-			                	if (metadataString.indexOf(',') != -1)
-			                		displayName = metadataString.substring(metadataString.indexOf(',') + 1);
-			                	else
-			                		displayName = metadataKey + "(" + metadataString + ")";
-			                	
-			                	%>
+								String label = "jsp.tools.edit-collection.form.label21.select." + metadataKey;
+		                	
+	                	%>
 			                	<option value="<%= metadataKey %>" 
 			                	<% if(metadataKey.equalsIgnoreCase(metadataFormatValue)) { %> 
 			                	selected="selected" <% } %> >
-			                	<%= displayName %></option>
+								<fmt:message key="<%=label%>"/>
+								</option>
 			                	<% 
 			                }
 			            }
@@ -459,15 +457,15 @@
                 </td>
             </tr>
             <tr>
-                <td class="submitFormLabel">Content being Harvested</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label22"/></td>
                 <td>
-                	<input type="radio" value="1" <% if (harvestLevelValue != 2 && harvestLevelValue != 3) { %> checked="checked" <% } %> name="harvest_level">Harvest metadata only.</input><br/>
-                	<input type="radio" value="2" <% if (harvestLevelValue == 2) { %> checked="checked" <% } %> name="harvest_level">Harvest metadata and references to bitstreams (requires ORE support).</input><br/>
-                	<input type="radio" value="3" <% if (harvestLevelValue == 3) { %> checked="checked" <% } %> name="harvest_level">Harvest metadata and bitstreams (requires ORE support).</input><br/>
+                	<input type="radio" value="1" <% if (harvestLevelValue != 2 && harvestLevelValue != 3) { %> checked="checked" <% } %> name="harvest_level"><fmt:message key="jsp.tools.edit-collection.form.label23"/></input><br/>
+                	<input type="radio" value="2" <% if (harvestLevelValue == 2) { %> checked="checked" <% } %> name="harvest_level"><fmt:message key="jsp.tools.edit-collection.form.label24"/></input><br/>
+                	<input type="radio" value="3" <% if (harvestLevelValue == 3) { %> checked="checked" <% } %> name="harvest_level"><fmt:message key="jsp.tools.edit-collection.form.label25"/></input><br/>
                 </td>
             </tr>
             <tr>
-                <td class="submitFormLabel">Last Harvest Result</td>
+                <td class="submitFormLabel"><fmt:message key="jsp.tools.edit-collection.form.label26"/></td>
                 <td><%= lastHarvestMsg %></td>
             </tr> 
             <!--

dspace-jspui/dspace-jspui-webapp/src/main/webapp/tools/itemmap-info.jsp

@@ -63,7 +63,7 @@
 <%
     Collection collection = (Collection)request.getAttribute("collection");
     
-    // supported values: "none-selected", "added", "remove"
+    // supported values: "none-selected", "none-removed", "added", "remove"
     String message        = (String)request.getAttribute("message");    
     
     LinkedList processedItems = (LinkedList)request.getAttribute("processedItems");
@@ -76,6 +76,8 @@
     <p>
     <% if (message.equals("none-selected")) { %>
         <fmt:message key="jsp.tools.itemmap-info.msg.none-selected" />
+    <% } else if (message.equals("none-removed")) { %>
+        <fmt:message key="jsp.tools.itemmap-info.msg.none-removed" />
     <% } else if (message.equals("added")) { %>
             <%-- Iterate through processed items --%>
             <% for (int i=0; i<processedItems.size(); i++) { %>

dspace-jspui/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-jspui</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-jspui</url>
    </scm>
 
    <modules>

dspace-lni/dspace-lni-client/pom.xml

@@ -12,7 +12,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-lni</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -33,9 +33,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-client</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-client</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-client</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-client</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-client</url>
    </scm>
 
    <profiles>

dspace-lni/dspace-lni-core/pom.xml

@@ -10,7 +10,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-lni</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -31,9 +31,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-core</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-core</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-core</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-core</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-core</url>
    </scm>
 
    <dependencies>

dspace-lni/dspace-lni-webapp/pom.xml

@@ -12,7 +12,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-lni</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -33,9 +33,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-webapp</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-webapp</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni/dspace-lni-webapp</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-webapp</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni/dspace-lni-webapp</url>
    </scm>
 
    <build>

dspace-lni/pom.xml

@@ -12,7 +12,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -38,9 +38,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-lni</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-lni</url>
    </scm>
 
    <modules>

dspace-oai/dspace-oai-api/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-oai</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-api</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-api</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-api</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-api</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-api</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-api</url>
    </scm>
 
    <dependencies>

dspace-oai/dspace-oai-api/src/main/java/org/dspace/app/oai/DIDLCrosswalk.java

@@ -45,8 +45,6 @@ import org.apache.commons.codec.binary.Base64;
 import org.dspace.app.didl.UUIDFactory;
 import org.dspace.content.Bitstream;
 import org.dspace.content.Bundle;
-import org.dspace.content.DCDate;
-import org.dspace.content.DCValue;
 import org.dspace.content.Item;
 import org.dspace.core.ConfigurationManager;
 import org.dspace.core.Context;
@@ -72,9 +70,16 @@ public class DIDLCrosswalk extends Crosswalk
     /** default value if no oai.didl.maxresponse property is defined */
     public static int MAXRESPONSE_INLINE_BITSTREAM = 0;
     
+    /** another crosswalk that will be used to generate the metadata section */
+    private Crosswalk metadataCrosswalk;
+    
     public DIDLCrosswalk(Properties properties)
     {
     	super("urn:mpeg:mpeg21:2002:02-DIDL-NS http://standards.iso.org/ittf/PubliclyAvailableStandards/MPEG-21_schema_files/did/didl.xsd ");
+
+    	// FIXME this should be injected from the configuration... 
+    	// but it is better than duplicate the OAIDCCrosswalk code! 
+    	metadataCrosswalk = new OAIDCCrosswalk(properties);
     }
     
     
@@ -90,14 +95,7 @@ public class DIDLCrosswalk extends Crosswalk
     {
         Item item = ((HarvestedItemInfo) nativeItem).item;
        
-        Date d = ((HarvestedItemInfo) nativeItem).datestamp;
-        String ITEMDATE = new DCDate(d).toString();
-        
-        // Get all the DC
-        DCValue[] allDC = item.getDC(Item.ANY, Item.ANY, Item.ANY);
-        
         StringBuffer metadata = new StringBuffer();
-        StringBuffer metadata1 = new StringBuffer();
         String itemhandle=item.getHandle();
         String strMaxSize = ConfigurationManager.getProperty("oai.didl.maxresponse");
         int maxsize = MAXRESPONSE_INLINE_BITSTREAM;
@@ -128,57 +126,10 @@ public class DIDLCrosswalk extends Crosswalk
         metadata.append("<didl:Descriptor>")
             .append("<didl:Statement mimeType=\"application/xml; charset=utf-8\">");
 					
-        for (int i = 0; i < allDC.length; i++)
-        {
-            // Do not include description.provenance
-            boolean description = allDC[i].element.equals("description");
-            boolean provenance = allDC[i].qualifier != null &&
-                                 allDC[i].qualifier.equals("provenance");
-
-            if (!(description && provenance))
-            {
-                // Escape XML chars <, > and &
-                String value = allDC[i].value;
-
-                // First do &'s - need to be careful not to replace the
-                // & in "&amp;" again!
-                int c = -1;
-                while ((c = value.indexOf("&", c + 1)) > -1)
-                {
-                    value = value.substring(0, c) +
-                        "&amp;" +
-                        value.substring(c + 1);
-                }
-
-                while ((c = value.indexOf("<")) > -1)
-                {
-                    value = value.substring(0, c) +
-                        "&lt;" +
-                        value.substring(c + 1);
-                }
-                
-                while ((c = value.indexOf(">")) > -1)
-                {
-                    value = value.substring(0, c) +
-                        "&gt;" +
-                        value.substring(c + 1);
-                }
-
-                metadata1.append("<dc:")
-                    .append(allDC[i].element)
-                    .append(">")
-                    .append(value)
-                    .append("</dc:")
-                    .append(allDC[i].element)
-                    .append(">");
-            }
-        }
+        // delegate the metadata section to another crosswalk
+        metadata.append(metadataCrosswalk.createMetadata(nativeItem));
         
-        metadata.append("<oai_dc:dc xmlns:oai_dc=\"http://www.openarchives.org/OAI/2.0/oai_dc/\" xmlns:dc=\"http://purl.org/dc/elements/1.1/\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" xsi:schemaLocation=\"http://www.openarchives.org/OAI/2.0/oai_dc/ http://www.openarchives.org/OAI/2.0/oai_dc.xsd\">");
-        				
-        metadata.append(metadata1);
-        
-        metadata.append("</oai_dc:dc>")
+        metadata
             .append("</didl:Statement>")
             .append("</didl:Descriptor>");				
         
@@ -191,12 +142,8 @@ public class DIDLCrosswalk extends Crosswalk
         {
             Bundle[] bundles= item.getBundles("ORIGINAL");    
             
-            if (bundles.length == 0)
+            if (bundles.length != 0)
             {
-                metadata.append("<P>There are no files associated with this item.</P>");
-            }
-            else
-            {  
             	/**cycle bundles**/
                 for (int i = 0; i < bundles.length; i++)
                 { 

dspace-oai/dspace-oai-api/src/main/java/org/dspace/app/oai/DSpaceOAICatalog.java

@@ -52,16 +52,19 @@ import java.util.StringTokenizer;
 import java.util.Vector;
 
 import org.apache.log4j.Logger;
+import org.dspace.authorize.AuthorizeManager;
 import org.dspace.content.Collection;
 import org.dspace.content.Community;
 import org.dspace.content.DSpaceObject;
 import org.dspace.core.ConfigurationManager;
+import org.dspace.core.Constants;
 import org.dspace.core.Context;
 import org.dspace.core.LogManager;
 import org.dspace.core.Utils;
 import org.dspace.handle.HandleManager;
 import org.dspace.search.Harvest;
 import org.dspace.search.HarvestedItemInfo;
+import org.dspace.eperson.Group;
 
 import ORG.oclc.oai.server.catalog.AbstractCatalog;
 import ORG.oclc.oai.server.verb.BadArgumentException;
@@ -369,6 +372,28 @@ public class DSpaceOAICatalog extends AbstractCatalog
                         "id_does_not_exist"));
                 throw new IdDoesNotExistException(identifier);
             }
+            
+            boolean includeAll = ConfigurationManager.getBooleanProperty("harvest.includerestricted.oai", true);
+
+            if (!includeAll)
+            {
+                Group[] authorizedGroups = AuthorizeManager.getAuthorizedGroups(context, itemInfo.item, Constants.READ);
+                boolean authorized = false;
+                for (int i = 0; i < authorizedGroups.length; i++)
+                {
+                    if ((authorizedGroups[i].getID() == 0) && (!authorized))
+                    {
+                        authorized = true;
+                    }
+                }
+
+                if (!authorized)
+                {
+                    log.info(LogManager.getHeader(null, "oai_error",
+                            "id_not_accessible"));
+                    throw new IdDoesNotExistException(identifier);
+                }
+            }
 
             String schemaURL;
 

dspace-oai/dspace-oai-webapp/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-oai</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-webapp</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-webapp</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai/dspace-oai-webapp</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-webapp</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-webapp</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai/dspace-oai-webapp</url>
    </scm>
    
    <build>

dspace-oai/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-oai</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-oai</url>
    </scm>
    
    <modules>

dspace-stats/pom.xml

@@ -3,14 +3,14 @@
 	<parent>
 		<artifactId>dspace-parent</artifactId>
 		<groupId>org.dspace</groupId>
-		<version>1.6.1-SNAPSHOT</version>
+		<version>1.6.2</version>
 	</parent>
 	
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>org.dspace</groupId>
 	<artifactId>dspace-stats</artifactId>
 	<name>DSpace Solr Statistics Logging Client Library</name>
-	<version>1.6.1-SNAPSHOT</version>
+	<version>1.6.2</version>
 	<description>Library of Shared UsageEvent and EventConsumer Tools for Logging to Solr.</description>
 
 	<repositories>
@@ -36,9 +36,9 @@
 		releases snapshots into the snapshot repository below.
 	-->
 	<scm>
-		<connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-stats</connection>
-		<developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-stats</developerConnection>
-		<url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-stats</url>
+		<connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-stats</connection>
+		<developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-stats</developerConnection>
+		<url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-stats</url>
 	</scm>
 
 	<distributionManagement>
@@ -115,7 +115,7 @@
 		<dependency>
 			<groupId>org.dspace</groupId>
 			<artifactId>dspace-api</artifactId>
-			<version>1.6.1-SNAPSHOT</version>
+			<version>1.6.2</version>
 		</dependency>
 		<dependency>
 			<groupId>org.dspace.dependencies</groupId>

dspace-stats/src/main/java/org/dspace/statistics/Dataset.java

@@ -168,13 +168,17 @@ public class Dataset {
 
     public String[][] getMatrixFormatted(){
         DecimalFormat decimalFormat = new DecimalFormat(format);
-        String [][]strMatrix = new String[matrix.length][matrix[0].length];
-        for (int i = 0; i < matrix.length; i++) {
-            for (int j = 0; j < matrix[i].length; j++) {
-                strMatrix[i][j] = decimalFormat.format(matrix[i][j]);
+        if (matrix.length == 0) {
+            return new String[0][0];
+        } else {
+            String[][] strMatrix = new String[matrix.length][matrix[0].length];
+            for (int i = 0; i < matrix.length; i++) {
+                for (int j = 0; j < matrix[i].length; j++) {
+                    strMatrix[i][j] = decimalFormat.format(matrix[i][j]);
+                }
             }
+            return strMatrix;
         }
-        return strMatrix;
     }
 
     public void addValueToMatrix(int row, int coll, float value) {

dspace-stats/src/main/java/org/dspace/statistics/SolrLogger.java

@@ -186,7 +186,8 @@ public class SolrLogger
             }
             catch (Exception e)
             {
-    			log.error(e.getMessage());
+    			log.error("Failed DNS Lookup for IP:" + ip);
+                log.debug(e.getMessage(),e);
     		}
             
             // Save the location information if valid, save the event without

dspace-stats/src/main/java/org/dspace/statistics/util/LocationUtils.java

@@ -15,6 +15,7 @@ import java.util.Vector;
 import java.util.Arrays;
 import java.util.List;
 import java.lang.reflect.Array;
+import org.dspace.core.I18nUtil;
 
 /**
  * Mapping between Country codes, English Country names, 
@@ -799,13 +800,21 @@ public class LocationUtils {
             {"OC", "Oceania"}};
 
     public static String getCountryName(String countryCode){
-        int index = countryCodeList.indexOf(countryCode);
-        return countryNameList.get(index).toString();           
+        if (countryCode.length() > 0 && countryCodeList.contains(countryCode)) {
+            int index = countryCodeList.indexOf(countryCode);
+            return countryNameList.get(index).toString();
+        } else {
+            return I18nUtil.getMessage("org.dspace.statistics.util.LocationUtils.unknown-country");
+        }
     }
 
     public static String getContinentCode(String countryCode){
-        int index = countryCodeList.indexOf(countryCode);
-        return continentCodeList.get(index).toString();
+        if(countryCode.length() > 0 && countryCodeList.contains(countryCode)) {
+            int index = countryCodeList.indexOf(countryCode);
+            return continentCodeList.get(index).toString();
+        } else {
+            return I18nUtil.getMessage("org.dspace.statistics.util.LocationUtils.unknown-continent");
+        }
     }
 
     public static String getContinentName(String continentCode){

dspace-stats/src/main/java/org/dspace/statistics/util/StatisticsImporter.java

@@ -242,8 +242,6 @@ public class StatisticsImporter
 
                 data += ("ip addr = " + ip);
                 data += (", dns name = " + dns);
-                data += (", country = " + country);
-                data += (", city = " + city);
                 if ((dns.endsWith(".googlebot.com.")) ||
                     (dns.endsWith(".crawl.yahoo.net.")) ||
                     (dns.endsWith(".search.msn.com.")))
@@ -253,7 +251,6 @@ public class StatisticsImporter
                     searchengines++;
                     continue;
                 }
-                if (verbose) System.out.println(data);
 
                 // Get the geo information for the user
                 Location location;
@@ -264,6 +261,11 @@ public class StatisticsImporter
                     countryCode = location.countryCode;
                     longitude = location.longitude;
                     latitude = location.latitude;
+                    if(verbose) {
+                        data += (", country = " + country);
+                        data += (", city = " + city);
+                        System.out.println(data);
+                    }
                     try {
                         continent = LocationUtils.getContinentCode(countryCode);
                     } catch (Exception e) {

dspace-sword/dspace-sword-api/pom.xml

@@ -16,7 +16,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-sword</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -42,106 +42,87 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-api</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-api</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-api</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-api</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-api</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-api</url>
    </scm>
 
    <dependencies>
 
-      <!-- Leave this out for the moment, as the source is in the tree -->
-      <!--
+       <!-- Leave this out for the moment, as the source is in the tree -->
+       <!--
+          <dependency>
+          <groupId>org.dspace</groupId>
+          <artifactId>sword-common</artifactId>
+          <version>1.0.0</version>
+          </dependency>-->
+
          <dependency>
-         <groupId>org.dspace</groupId>
-         <artifactId>sword-common</artifactId>
-         <version>1.0.0</version>
-         </dependency>-->
+             <groupId>org.dspace</groupId>
+             <artifactId>dspace-api</artifactId>
+         </dependency>
 
-      <dependency>
-         <groupId>org.dspace</groupId>
-         <artifactId>dspace-api</artifactId>
-      </dependency>
+         <dependency>
+             <groupId>org.dspace</groupId>
+             <artifactId>dspace-api-lang</artifactId>
+         </dependency>
 
-      <dependency>
-         <groupId>org.dspace</groupId>
-         <artifactId>dspace-api-lang</artifactId>
-      </dependency>
-      
-      <dependency>
-         <groupId>xom</groupId>
-         <artifactId>xom</artifactId>
-         <version>1.1</version>
-      </dependency>
-
-      <dependency>
-         <groupId>jaxen</groupId>
-         <artifactId>jaxen</artifactId>
-      </dependency>
-
-      <dependency>
-         <groupId>javax.servlet</groupId>
-         <artifactId>servlet-api</artifactId>
-      </dependency>
-
-       <!-- additional dependencies for the sword-common code -->
-                       <dependency>
-                        <groupId>commons-fileupload</groupId>
-                        <artifactId>commons-fileupload</artifactId>
-                        <version>1.2</version>
-                </dependency>
-                <dependency>
-                        <groupId>commons-httpclient</groupId>
-                        <artifactId>commons-httpclient</artifactId>
-                        <version>3.0.1</version>
-                </dependency>
-                <dependency>
-                        <groupId>javax.servlet</groupId>
-                        <artifactId>servlet-api</artifactId>
-                        <version>2.4</version>
-                </dependency>
-                <dependency>
-                        <groupId>junit</groupId>
-                        <artifactId>junit</artifactId>
-                        <version>4.0</version>
-                        <scope>test</scope>
-                </dependency>
-                <dependency>
-                        <groupId>log4j</groupId>
-                        <artifactId>log4j</artifactId>
-                        <version>1.2.15</version>
-                        <exclusions>
-                                <exclusion>
-                                        <artifactId>jmxtools</artifactId>
-                                        <groupId>com.sun.jdmk</groupId>
-                                </exclusion>
-                                <exclusion>
-                                        <artifactId>jms</artifactId>
-                                        <groupId>javax.jms</groupId>
-                                </exclusion>
-                                <exclusion>
-                                        <artifactId>jmxri</artifactId>
-                                        <groupId>com.sun.jmx</groupId>
-                                </exclusion>
-                        </exclusions>
-                </dependency>
-       <!-- GUI client dependency
-                <dependency>
-                        <groupId>org.jdesktop</groupId>
-                        <artifactId>swing-worker</artifactId>
-                        <version>1.1</version>
-                </dependency>
-                -->
-                <dependency>
-                        <groupId>xom</groupId>
-                        <artifactId>xom</artifactId>
-                        <version>1.1</version>
-                </dependency>
-                <dependency>
-                        <groupId>commons-io</groupId>
-                        <artifactId>commons-io</artifactId>
-                        <version>1.3.1</version>
-                </dependency>
+         <dependency>
+             <groupId>jaxen</groupId>
+             <artifactId>jaxen</artifactId>
+         </dependency>
 
+        <!-- additional dependencies for the sword-common code -->
+        <dependency>
+             <groupId>commons-fileupload</groupId>
+             <artifactId>commons-fileupload</artifactId>
+             <version>1.2</version>
+         </dependency>
+         <dependency>
+             <groupId>commons-httpclient</groupId>
+             <artifactId>commons-httpclient</artifactId>
+             <version>3.0.1</version>
+         </dependency>
+         <dependency>
+             <groupId>javax.servlet</groupId>
+             <artifactId>servlet-api</artifactId>
+             <version>2.4</version>
+         </dependency>
+         <dependency>
+             <groupId>junit</groupId>
+             <artifactId>junit</artifactId>
+             <version>4.0</version>
+             <scope>test</scope>
+         </dependency>
+         <dependency>
+             <groupId>log4j</groupId>
+             <artifactId>log4j</artifactId>
+             <version>1.2.15</version>
+             <exclusions>
+                 <exclusion>
+                         <artifactId>jmxtools</artifactId>
+                         <groupId>com.sun.jdmk</groupId>
+                 </exclusion>
+                 <exclusion>
+                         <artifactId>jms</artifactId>
+                         <groupId>javax.jms</groupId>
+                 </exclusion>
+                 <exclusion>
+                         <artifactId>jmxri</artifactId>
+                         <groupId>com.sun.jmx</groupId>
+                 </exclusion>
+             </exclusions>
+         </dependency>
+         <dependency>
+                 <groupId>xom</groupId>
+                 <artifactId>xom</artifactId>
+                 <version>1.1</version>
+         </dependency>
+         <dependency>
+                 <groupId>commons-io</groupId>
+                 <artifactId>commons-io</artifactId>
+                 <version>1.3.1</version>
+         </dependency>
 
    </dependencies>
 

dspace-sword/dspace-sword-api/src/main/java/org/dspace/sword/DSpaceSWORDServer.java

@@ -80,7 +80,7 @@ public class DSpaceSWORDServer implements SWORDServer
 	{
 		// gah.  bloody variable scoping.
 		// set up a dummy sword context for the "finally" block
-		SWORDContext sc = new SWORDContext();
+		SWORDContext sc = null;
 
 		try
 		{
@@ -113,7 +113,10 @@ public class DSpaceSWORDServer implements SWORDServer
 		finally
 		{
 			// this is a read operation only, so there's never any need to commit the context
-			sc.abort();
+            if (sc != null)
+            {
+                sc.abort();
+            }
 		}
 	}
 	
@@ -125,7 +128,7 @@ public class DSpaceSWORDServer implements SWORDServer
 	{
 		// gah.  bloody variable scoping.
 		// set up a dummy sword context for the "finally" block
-		SWORDContext sc = new SWORDContext();
+		SWORDContext sc = null;
 
 		try
 		{
@@ -163,7 +166,10 @@ public class DSpaceSWORDServer implements SWORDServer
 		{
 			// if, for some reason, we wind up here with a not null context
 			// then abort it (the above should commit it if everything works fine)
-			sc.abort();
+		    if (sc != null)
+            {
+                sc.abort();
+            }
 		}
 	}
 
@@ -175,7 +181,7 @@ public class DSpaceSWORDServer implements SWORDServer
 	{
 		// gah.  bloody variable scoping.
 		// set up a dummy sword context for the "finally" block
-		SWORDContext sc = new SWORDContext();
+		SWORDContext sc = null;
 
 		try
 		{
@@ -208,7 +214,10 @@ public class DSpaceSWORDServer implements SWORDServer
 		finally
 		{
 			// this is a read operation only, so there's never any need to commit the context
-			sc.abort();
+		    if (sc != null)
+            {
+                sc.abort();
+            }
 		}
 	}
 }

dspace-sword/dspace-sword-api/src/main/java/org/dspace/sword/SWORDAuthenticator.java

@@ -129,7 +129,44 @@ public class SWORDAuthenticator
 			throws SWORDException, SWORDErrorException, SWORDAuthenticationException
 	{
 		Context context = this.constructContext(request.getIPAddress());
-		return this.authenticate(context, request);
+		SWORDContext sc = null;
+		try
+        {
+            sc = this.authenticate(context, request);
+        }
+        catch (SWORDException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDErrorException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDAuthenticationException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (RuntimeException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+		return sc; 
 	}
 
 	/**
@@ -146,8 +183,45 @@ public class SWORDAuthenticator
 			throws SWORDException, SWORDErrorException, SWORDAuthenticationException
 	{
 		Context context = this.constructContext(request.getIPAddress());
-		return this.authenticate(context, request);
-	}
+		SWORDContext sc = null;
+		try
+        {
+            sc = this.authenticate(context, request);
+        }
+        catch (SWORDException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDErrorException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDAuthenticationException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (RuntimeException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        return sc;
+    }
 
 	/**
 	 * Authenticate the incoming service document request.  Calls:
@@ -194,7 +268,44 @@ public class SWORDAuthenticator
 			throws SWORDException, SWORDErrorException, SWORDAuthenticationException
 	{
 		Context context = this.constructContext(deposit.getIPAddress());
-		return this.authenticate(context, deposit);
+		SWORDContext sc = null;
+		try
+		{
+		    sc = this.authenticate(context, deposit);
+		}
+        catch (SWORDException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDErrorException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (SWORDAuthenticationException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        catch (RuntimeException e)
+        {
+            if (context != null && context.isValid())
+            {
+                context.abort();
+            }
+            throw e;
+        }
+        return sc;
 	}
 
 	/**

dspace-sword/dspace-sword-webapp/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-sword</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-webapp</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-webapp</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword/dspace-sword-webapp</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-webapp</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-webapp</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword/dspace-sword-webapp</url>
    </scm>
 
    <build>
@@ -130,7 +130,7 @@
       <dependency>
          <groupId>org.dspace</groupId>
          <artifactId>dspace-sword-api</artifactId>
-         <version>1.6.1-SNAPSHOT</version>
+         <version>1.6.2</version>
       </dependency>
    </dependencies>
 

dspace-sword/pom.xml

@@ -16,7 +16,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -42,9 +42,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-sword</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-sword</url>
    </scm>
 
 

dspace-ui-shared/pom.xml

@@ -14,7 +14,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -40,9 +40,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-ui-shared</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-ui-shared</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-ui-shared</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-ui-shared</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-ui-shared</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-ui-shared</url>
    </scm>
 
    <!--

dspace-xmlui/dspace-xmlui-api/pom.xml

@@ -13,7 +13,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-xmlui</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -34,9 +34,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-api</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-api</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-api/</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-api</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-api</url>
    </scm>
 
    <dependencies>

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/administrative/metadataimport/MetadataImportConfirm.java

@@ -77,18 +77,20 @@ public class MetadataImportConfirm extends AbstractDSpaceTransformer {
 	private static final Message T_dspace_home = message("xmlui.general.dspace_home");
 	private static final Message T_submit_return = message("xmlui.general.return");
 	private static final Message T_trail = message("xmlui.administrative.metadataimport.general.trail");
-        private static final Message T_changes = message("xmlui.administrative.metadataimport.general.changes");
-        private static final Message T_new_item = message("xmlui.administrative.metadataimport.general.new_item");
-        private static final Message T_no_changes = message("xmlui.administrative.metadataimport.general.no_changes");
+    private static final Message T_changes = message("xmlui.administrative.metadataimport.general.changes");
+    private static final Message T_new_item = message("xmlui.administrative.metadataimport.general.new_item");
+    private static final Message T_no_changes = message("xmlui.administrative.metadataimport.general.no_changes");
 	private static final Message T_title = message("xmlui.administrative.metadataimport.general.title");
 	private static final Message T_head1 = message("xmlui.administrative.metadataimport.general.head1");
 
-        private static final Message T_success = message("xmlui.administrative.metadataimport.MetadataImportConfirm.success");
-        private static final Message T_changes_committed = message("xmlui.administrative.metadataimport.MetadataImportConfirm.changes_committed");
-        private static final Message T_item_addition = message("xmlui.administrative.metadataimport.MetadataImportConfirm.item_added");
-        private static final Message T_item_deletion = message("xmlui.administrative.metadataimport.MetadataImportConfirm.item_removed");
-        private static final Message T_collection_addition = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_added");
-        private static final Message T_collection_deletion = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_removed");
+    private static final Message T_success = message("xmlui.administrative.metadataimport.MetadataImportConfirm.success");
+    private static final Message T_changes_committed = message("xmlui.administrative.metadataimport.MetadataImportConfirm.changes_committed");
+    private static final Message T_item_addition = message("xmlui.administrative.metadataimport.MetadataImportConfirm.item_added");
+    private static final Message T_item_deletion = message("xmlui.administrative.metadataimport.MetadataImportConfirm.item_removed");
+    private static final Message T_collection_newowner = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_newowner");
+    private static final Message T_collection_oldowner = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_oldowner");
+    private static final Message T_collection_mapped = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_mapped");
+    private static final Message T_collection_unmapped = message("xmlui.administrative.metadataimport.MetadataImportConfirm.collection_unmapped");
 
 
 	public void addPageMeta(PageMeta pageMeta) throws WingException  
@@ -105,10 +107,12 @@ public class MetadataImportConfirm extends AbstractDSpaceTransformer {
 		// Get list of changes
 
 		Request request = ObjectModelHelper.getRequest(objectModel);
-                ArrayList<BulkEditChange> changes = null;
+        ArrayList<BulkEditChange> changes = null;
 
-                if(request.getAttribute("changes") != null)
-                        changes = ((ArrayList<BulkEditChange>)request.getAttribute("changes"));
+        if(request.getAttribute("changes") != null)
+        {
+            changes = ((ArrayList<BulkEditChange>)request.getAttribute("changes"));
+        }
 
 		// DIVISION: metadata-import
 		Division div = body.addInteractiveDivision("metadata-import",contextPath + "/admin/metadataimport", Division.METHOD_MULTIPART,"primary administrative");
@@ -129,20 +133,22 @@ public class MetadataImportConfirm extends AbstractDSpaceTransformer {
                         // Get the changes
                         ArrayList<DCValue> adds = change.getAdds();
                         ArrayList<DCValue> removes = change.getRemoves();
-                        ArrayList<Collection> newCollections = change.getNewOwningCollections();
-                        ArrayList<Collection> oldCollections = change.getOldOwningCollections();
+                        ArrayList<Collection> newCollections = change.getNewMappedCollections();
+                        ArrayList<Collection> oldCollections = change.getOldMappedCollections();
 
                         if ((adds.size() > 0) || (removes.size() > 0) ||
-                            (newCollections.size() > 0) || (oldCollections.size() > 0))
+                            (newCollections.size() > 0) || (oldCollections.size() > 0) ||
+                            (change.getNewOwningCollection() != null) || (change.getOldOwningCollection() != null))
                         {
                             Row headerrow = mdchanges.addRow(Row.ROLE_HEADER);
+
                             // Show the item
                             if (!change.isNewItem())
                             {
                                 Item i = change.getItem();
                                 Cell cell = headerrow.addCell();
                                 cell.addContent(T_changes_committed);
-                                cell.addContent(" " + i.getID() + "(" + i.getHandle() + ")");
+                                cell.addContent(" " + i.getID() + " (" + i.getHandle() + ")");
 
                             }
                             else
@@ -153,23 +159,51 @@ public class MetadataImportConfirm extends AbstractDSpaceTransformer {
                             changeCounter++;
                         }
 
-                        // Show new collections
+                        // Show new owning collection
+                        if (change.getNewOwningCollection() != null)
+                        {
+                            Collection c = change.getNewOwningCollection();
+                            if (c != null)
+                            {
+                                String cHandle = c.getHandle();
+                                String cName = c.getName();
+                                Row colrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
+                                colrow.addCellContent(T_collection_newowner);
+                                colrow.addCellContent(cHandle + " (" + cName + ")");
+                            }
+                        }
+
+                        // Show old owning collection
+                        if (change.getOldOwningCollection() != null)
+                        {
+                            Collection c = change.getOldOwningCollection();
+                            if (c != null)
+                            {
+                                String cHandle = c.getHandle();
+                                String cName = c.getName();
+                                Row colrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
+                                colrow.addCellContent(T_collection_oldowner);
+                                colrow.addCellContent(cHandle + " (" + cName + ")");
+                            }
+                        }
+
+                        // Show new mapped collections
                         for (Collection c : newCollections)
                         {
                             String cHandle = c.getHandle();
                             String cName = c.getName();
                             Row colrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
-                            colrow.addCellContent(T_collection_addition);
+                            colrow.addCellContent(T_collection_mapped);
                             colrow.addCellContent(cHandle + " (" + cName + ")");
                         }
 
-                        // Show old collections
+                        // Show old mapped collections
                         for (Collection c : oldCollections)
                         {
                             String cHandle = c.getHandle();
                             String cName = c.getName();
                             Row colrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
-                            colrow.addCellContent(T_collection_deletion);
+                            colrow.addCellContent(T_collection_unmapped);
                             colrow.addCellContent(cHandle + " (" + cName + ")");
                         }
 

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/administrative/metadataimport/MetadataImportUpload.java

@@ -78,19 +78,19 @@ public class MetadataImportUpload extends AbstractDSpaceTransformer {
 	private static final Message T_submit_return = message("xmlui.general.return");
 	private static final Message T_trail = message("xmlui.administrative.metadataimport.general.trail");
 	private static final Message T_no_changes = message("xmlui.administrative.metadataimport.general.no_changes");
-        private static final Message T_new_item = message("xmlui.administrative.metadataimport.general.new_item");
+    private static final Message T_new_item = message("xmlui.administrative.metadataimport.general.new_item");
 	private static final Message T_title = message("xmlui.administrative.metadataimport.general.title");
 	private static final Message T_head1 = message("xmlui.administrative.metadataimport.general.head1");
 
 	private static final Message T_para = message("xmlui.administrative.metadataimport.MetadataImportUpload.hint");
-        private static final Message T_submit_confirm = message("xmlui.administrative.metadataimport.MetadataImportUpload.submit_confirm");
+    private static final Message T_submit_confirm = message("xmlui.administrative.metadataimport.MetadataImportUpload.submit_confirm");
 	private static final Message T_changes_pending = message("xmlui.administrative.metadataimport.MetadataImportUpload.changes_pending");
-        private static final Message T_item_addition = message("xmlui.administrative.metadataimport.MetadataImportUpload.item_add");
-        private static final Message T_item_deletion = message("xmlui.administrative.metadataimport.MetadataImportUpload.item_remove");
-        private static final Message T_collection_addition = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_add");
-        private static final Message T_collection_deletion = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_remove");
-        
-
+    private static final Message T_item_addition = message("xmlui.administrative.metadataimport.MetadataImportUpload.item_add");
+    private static final Message T_item_deletion = message("xmlui.administrative.metadataimport.MetadataImportUpload.item_remove");
+    private static final Message T_collection_newowner = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_newowner");
+    private static final Message T_collection_oldowner = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_oldowner");
+    private static final Message T_collection_mapped = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_mapped");
+    private static final Message T_collection_unmapped = message("xmlui.administrative.metadataimport.MetadataImportUpload.collection_unmapped");
 
 	public void addPageMeta(PageMeta pageMeta) throws WingException  
 	{
@@ -106,136 +106,165 @@ public class MetadataImportUpload extends AbstractDSpaceTransformer {
 		// Get list of changes
 
 		Request request = ObjectModelHelper.getRequest(objectModel);
-                ArrayList<BulkEditChange> changes = null;
-                int num_changes = 0;
+        ArrayList<BulkEditChange> changes = null;
+        int num_changes = 0;
+
+        if(request.getAttribute("changes") != null)
+        {
+            changes = ((ArrayList<BulkEditChange>)request.getAttribute("changes"));
+            num_changes = changes.size();
+        }
 
-                if(request.getAttribute("changes") != null)
-                {
-                    changes = ((ArrayList<BulkEditChange>)request.getAttribute("changes"));
-                    num_changes = changes.size();
-                }
-                        
 
 		// DIVISION: metadata-import
 		Division div = body.addInteractiveDivision("metadata-import",contextPath + "/admin/metadataimport", Division.METHOD_MULTIPART,"primary administrative");
 		div.setHead(T_head1);
 	
-                if(num_changes > 0)
+        if(num_changes > 0)
+        {
+
+            div.addPara(T_para);
+
+            Table mdchanges = div.addTable("metadata-changes", num_changes, 2);
+
+            // Display the changes
+            int changeCounter = 0;
+            for (BulkEditChange change : changes)
+            {
+                // Get the changes
+                ArrayList<DCValue> adds = change.getAdds();
+                ArrayList<DCValue> removes = change.getRemoves();
+                ArrayList<Collection> newCollections = change.getNewMappedCollections();
+                ArrayList<Collection> oldCollections = change.getOldMappedCollections();
+
+                if ((adds.size() > 0) || (removes.size() > 0) ||
+                    (newCollections.size() > 0) || (oldCollections.size() > 0) ||
+                    (change.getNewOwningCollection() != null) || (change.getOldOwningCollection() != null))
                 {
-
-                    div.addPara(T_para);
-
-                    Table mdchanges = div.addTable("metadata-changes", num_changes, 2);
-
-                    // Display the changes
-                    int changeCounter = 0;
-                    for (BulkEditChange change : changes)
+                    Row headerrow = mdchanges.addRow(Row.ROLE_HEADER);
+                    // Show the item
+                    if (!change.isNewItem())
                     {
-                        // Get the changes
-                        ArrayList<DCValue> adds = change.getAdds();
-                        ArrayList<DCValue> removes = change.getRemoves();
-                        ArrayList<Collection> newCollections = change.getNewOwningCollections();
-                        ArrayList<Collection> oldCollections = change.getOldOwningCollections();
-                        
-                        if ((adds.size() > 0) || (removes.size() > 0) ||
-                            (newCollections.size() > 0) || (oldCollections.size() > 0))
-                        {
-                            Row headerrow = mdchanges.addRow(Row.ROLE_HEADER);
-                            // Show the item
-                            if (!change.isNewItem())
-                            {
-                                Item i = change.getItem();
-                                Cell cell = headerrow.addCell();
-                                cell.addContent(T_changes_pending);
-                                cell.addContent(" " + i.getID() + "(" + i.getHandle() + ")");
+                        Item i = change.getItem();
+                        Cell cell = headerrow.addCell();
+                        cell.addContent(T_changes_pending);
+                        cell.addContent(" " + i.getID() + " (" + i.getHandle() + ")");
 
-                            }
-                            else
-                            {
-                              headerrow.addCellContent(T_new_item);
-                            }
-                            headerrow.addCell();
-                            changeCounter++;
-                        }
-
-                        // Show new collections
-                        for (Collection c : newCollections)
-                        {
-                            String cHandle = c.getHandle();
-                            String cName = c.getName();
-                            Row colrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
-                            colrow.addCellContent(T_collection_addition);
-                            colrow.addCellContent(cHandle + " (" + cName + ")");
-                        }
-
-                        // Show old collections
-                        for (Collection c : oldCollections)
-                        {
-                            String cHandle = c.getHandle();
-                            String cName = c.getName();
-                            Row colrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
-                            colrow.addCellContent(T_collection_deletion);
-                            colrow.addCellContent(cHandle + " (" + cName + ")");
-                        }
-
-                        // Show additions
-                        for (DCValue dcv : adds)
-                        {
-                            Row mdrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
-                            String md = dcv.schema + "." + dcv.element;
-                            if (dcv.qualifier != null)
-                            {
-                                md += "." + dcv.qualifier;
-                            }
-                            if (dcv.language != null)
-                            {
-                                md += "[" + dcv.language + "]";
-                            }
-
-                            Cell cell = mdrow.addCell();
-                            cell.addContent(T_item_addition);
-                            cell.addContent(" (" + md + ")");
-                            mdrow.addCellContent(dcv.value);
-                        }
-
-                        // Show removals
-                        for (DCValue dcv : removes)
-                        {
-                            Row mdrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
-                            String md = dcv.schema + "." + dcv.element;
-                            if (dcv.qualifier != null)
-                            {
-                                md += "." + dcv.qualifier;
-                            }
-                            if (dcv.language != null)
-                            {
-                                md += "[" + dcv.language + "]";
-                            }
-
-                            Cell cell = mdrow.addCell();
-                            cell.addContent(T_item_deletion);
-                            cell.addContent(" (" + md + ")");
-                            mdrow.addCellContent(dcv.value);
-                        }
                     }
-                    Para actions = div.addPara();
-                    Button applychanges = actions.addButton("submit_confirm");
-                    applychanges.setValue(T_submit_confirm);
-                    Button cancel = actions.addButton("submit_return");
-                    cancel.setValue(T_submit_return);
+                    else
+                    {
+                      headerrow.addCellContent(T_new_item);
+                    }
+                    headerrow.addCell();
+                    changeCounter++;
                 }
-                else
+
+                // Show new owning collection
+                if (change.getNewOwningCollection() != null)
                 {
-                    Para nochanges = div.addPara();
-                    nochanges.addContent(T_no_changes);
-                    Para actions = div.addPara();
-                    Button cancel = actions.addButton("submit_return");
-                    cancel.setValue(T_submit_return);
+                    Collection c = change.getNewOwningCollection();
+                    if (c != null)
+                    {
+                        String cHandle = c.getHandle();
+                        String cName = c.getName();
+                        Row colrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
+                        colrow.addCellContent(T_collection_newowner);
+                        colrow.addCellContent(cHandle + " (" + cName + ")");
+                    }
                 }
-                
-                
-                    
-                div.addHidden("administrative-continue").setValue(knot.getId());
+
+                // Show old owning collection
+                if (change.getOldOwningCollection() != null)
+                {
+                    Collection c = change.getOldOwningCollection();
+                    if (c != null)
+                    {
+                        String cHandle = c.getHandle();
+                        String cName = c.getName();
+                        Row colrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
+                        colrow.addCellContent(T_collection_oldowner);
+                        colrow.addCellContent(cHandle + " (" + cName + ")");
+                    }
+                }
+
+                // Show new mapped collections
+                for (Collection c : newCollections)
+                {
+                    String cHandle = c.getHandle();
+                    String cName = c.getName();
+                    Row colrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
+                    colrow.addCellContent(T_collection_mapped);
+                    colrow.addCellContent(cHandle + " (" + cName + ")");
+                }
+
+                // Show old mapped collections
+                for (Collection c : oldCollections)
+                {
+                    String cHandle = c.getHandle();
+                    String cName = c.getName();
+                    Row colrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
+                    colrow.addCellContent(T_collection_unmapped);
+                    colrow.addCellContent(cHandle + " (" + cName + ")");
+                }
+
+                // Show additions
+                for (DCValue dcv : adds)
+                {
+                    Row mdrow = mdchanges.addRow("addition",Row.ROLE_DATA,"metadata-addition");
+                    String md = dcv.schema + "." + dcv.element;
+                    if (dcv.qualifier != null)
+                    {
+                        md += "." + dcv.qualifier;
+                    }
+                    if (dcv.language != null)
+                    {
+                        md += "[" + dcv.language + "]";
+                    }
+
+                    Cell cell = mdrow.addCell();
+                    cell.addContent(T_item_addition);
+                    cell.addContent(" (" + md + ")");
+                    mdrow.addCellContent(dcv.value);
+                }
+
+                // Show removals
+                for (DCValue dcv : removes)
+                {
+                    Row mdrow = mdchanges.addRow("deletion",Row.ROLE_DATA,"metadata-deletion");
+                    String md = dcv.schema + "." + dcv.element;
+                    if (dcv.qualifier != null)
+                    {
+                        md += "." + dcv.qualifier;
+                    }
+                    if (dcv.language != null)
+                    {
+                        md += "[" + dcv.language + "]";
+                    }
+
+                    Cell cell = mdrow.addCell();
+                    cell.addContent(T_item_deletion);
+                    cell.addContent(" (" + md + ")");
+                    mdrow.addCellContent(dcv.value);
+                }
+            }
+            Para actions = div.addPara();
+            Button applychanges = actions.addButton("submit_confirm");
+            applychanges.setValue(T_submit_confirm);
+            Button cancel = actions.addButton("submit_return");
+            cancel.setValue(T_submit_return);
+        }
+        else
+        {
+            Para nochanges = div.addPara();
+            nochanges.addContent(T_no_changes);
+            Para actions = div.addPara();
+            Button cancel = actions.addButton("submit_return");
+            cancel.setValue(T_submit_return);
+        }
+
+
+
+        div.addHidden("administrative-continue").setValue(knot.getId());
 	}
 
 }

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/artifactbrowser/CollectionViewer.java

@@ -124,7 +124,7 @@ public class CollectionViewer extends AbstractDSpaceTransformer implements Cache
         message("xmlui.ArtifactBrowser.CollectionViewer.head_recent_submissions");
     
     /** How many recent submissions to include in the page */
-    private static final int RECENT_SUBMISISONS = 5;
+    private static final int RECENT_SUBMISSIONS = 5;
 
     /** The cache of recently submitted items */
     private java.util.List<BrowseItem> recentSubmissionItems;
@@ -355,9 +355,10 @@ public class CollectionViewer extends AbstractDSpaceTransformer implements Cache
             return recentSubmissionItems;
         
         String source = ConfigurationManager.getProperty("recent.submissions.sort-option");
+        int numRecentSubmissions = ConfigurationManager.getIntProperty("recent.submissions.count", RECENT_SUBMISSIONS);
         BrowserScope scope = new BrowserScope(context);
         scope.setCollection(collection);
-        scope.setResultsPerPage(RECENT_SUBMISISONS);
+        scope.setResultsPerPage(numRecentSubmissions);
         
         // FIXME Exception Handling
         try

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/artifactbrowser/CommunityViewer.java

@@ -125,7 +125,7 @@ public class CommunityViewer extends AbstractDSpaceTransformer implements Cachea
         message("xmlui.ArtifactBrowser.CommunityViewer.head_recent_submissions");
     
     /** How many recent submissions to list */
-    private static final int RECENT_SUBMISISONS = 5;
+    private static final int RECENT_SUBMISSIONS = 5;
 
     /** The cache of recently submitted items */
     private java.util.List<BrowseItem> recentSubmittedItems;
@@ -397,9 +397,10 @@ public class CommunityViewer extends AbstractDSpaceTransformer implements Cachea
             return recentSubmittedItems;
 
         String source = ConfigurationManager.getProperty("recent.submissions.sort-option");
+        int numRecentSubmissions = ConfigurationManager.getIntProperty("recent.submissions.count", RECENT_SUBMISSIONS);
         BrowserScope scope = new BrowserScope(context);
         scope.setCommunity(community);
-        scope.setResultsPerPage(RECENT_SUBMISISONS);
+        scope.setResultsPerPage(numRecentSubmissions);
         
         // FIXME Exception Handling
         try

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/artifactbrowser/ConfigurableBrowse.java

@@ -128,6 +128,8 @@ public class ConfigurableBrowse extends AbstractDSpaceTransformer implements
 
     private final static Message T_order = message("xmlui.ArtifactBrowser.ConfigurableBrowse.general.order");
 
+    private final static Message T_no_results= message("xmlui.ArtifactBrowser.ConfigurableBrowse.general.no_results");
+
     private final static Message T_rpp = message("xmlui.ArtifactBrowser.ConfigurableBrowse.general.rpp");
 
     private final static Message T_etal = message("xmlui.ArtifactBrowser.ConfigurableBrowse.general.etal");
@@ -282,57 +284,65 @@ public class ConfigurableBrowse extends AbstractDSpaceTransformer implements
         // This div will hold the browsing results
         Division results = div.addDivision("browse-by-" + type + "-results", "primary");
 
-        // Add the pagination
-        //results.setSimplePagination(itemsTotal, firstItemIndex, lastItemIndex, previousPage, nextPage)
-        results.setSimplePagination(info.getTotal(), browseInfo.getOverallPosition() + 1,
-                browseInfo.getOverallPosition() + browseInfo.getResultCount(), getPreviousPageURL(
-                        params, info), getNextPageURL(params, info));
-
-        // Reference all the browsed items
-        ReferenceSet referenceSet = results.addReferenceSet("browse-by-" + type,
-                ReferenceSet.TYPE_SUMMARY_LIST, type, null);
-
-        // Are we browsing items, or unique metadata?
-        if (isItemBrowse(info))
+        // If there are items to browse, add the pagination
+        int itemsTotal = info.getTotal();
+        if (itemsTotal > 0) 
         {
-            // Add the items to the browse results
-            for (BrowseItem item : (java.util.List<BrowseItem>) info.getResults())
+            //results.setSimplePagination(itemsTotal, firstItemIndex, lastItemIndex, previousPage, nextPage)
+            results.setSimplePagination(itemsTotal, browseInfo.getOverallPosition() + 1,
+                    browseInfo.getOverallPosition() + browseInfo.getResultCount(), getPreviousPageURL(
+                            params, info), getNextPageURL(params, info));
+
+            // Reference all the browsed items
+            ReferenceSet referenceSet = results.addReferenceSet("browse-by-" + type,
+                    ReferenceSet.TYPE_SUMMARY_LIST, type, null);
+
+            // Are we browsing items, or unique metadata?
+            if (isItemBrowse(info))
             {
-                referenceSet.addReference(item);
+                // Add the items to the browse results
+                for (BrowseItem item : (java.util.List<BrowseItem>) info.getResults())
+                {
+                    referenceSet.addReference(item);
+                }
+            }
+            else    // browsing a list of unique metadata entries
+            {
+                // Create a table for the results
+                Table singleTable = results.addTable("browse-by-" + type + "-results",
+                        browseInfo.getResultCount() + 1, 1);
+            
+                // Add the column heading
+                singleTable.addRow(Row.ROLE_HEADER).addCell().addContent(
+                        message("xmlui.ArtifactBrowser.ConfigurableBrowse." + type + ".column_heading"));
+
+                // Iterate each result
+                for (String[] singleEntry : browseInfo.getStringResults())
+                {
+                    // Create a Map of the query parameters for the link
+                    Map<String, String> queryParams = new HashMap<String, String>();
+                    queryParams.put(BrowseParams.TYPE, URLEncode(type));
+                    if (singleEntry[1] != null)
+                    {
+                        queryParams.put(BrowseParams.FILTER_VALUE[1], URLEncode(
+                            singleEntry[1]));
+                    }
+                    else
+                    {
+                        queryParams.put(BrowseParams.FILTER_VALUE[0], URLEncode(
+                            singleEntry[0]));
+                    }
+
+                    // Create an entry in the table, and a linked entry
+                    Cell cell = singleTable.addRow().addCell();
+                    cell.addXref(super.generateURL(BROWSE_URL_BASE, queryParams),
+                          singleEntry[0]);
+                }  
             }
         }
-        else    // browsing a list of unique metadata entries
+        else 
         {
-            // Create a table for the results
-            Table singleTable = results.addTable("browse-by-" + type + "-results",
-                    browseInfo.getResultCount() + 1, 1);
-            
-            // Add the column heading
-            singleTable.addRow(Row.ROLE_HEADER).addCell().addContent(
-                    message("xmlui.ArtifactBrowser.ConfigurableBrowse." + type + ".column_heading"));
-
-            // Iterate each result
-            for (String[] singleEntry : browseInfo.getStringResults())
-            {
-                // Create a Map of the query parameters for the link
-                Map<String, String> queryParams = new HashMap<String, String>();
-                queryParams.put(BrowseParams.TYPE, URLEncode(type));
-                if (singleEntry[1] != null)
-                {
-                    queryParams.put(BrowseParams.FILTER_VALUE[1], URLEncode(
-                        singleEntry[1]));
-                }
-                else
-                {
-                    queryParams.put(BrowseParams.FILTER_VALUE[0], URLEncode(
-                        singleEntry[0]));
-                }
-
-                // Create an entry in the table, and a linked entry
-                Cell cell = singleTable.addRow().addCell();
-                cell.addXref(super.generateURL(BROWSE_URL_BASE, queryParams),
-                      singleEntry[0]);
-            }
+            results.addPara(T_no_results);
         }
     }
 

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/aspect/submission/submit/DescribeStep.java

@@ -447,7 +447,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         fullName.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        fullName.addError(dcInput.getWarning());
+                    } else {
                         fullName.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         fullName.enableAddOperation();
                 if ((dcInput.isRepeatable() || dcValues.length > 1)  && !readonly)
@@ -543,7 +547,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         fullDate.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        fullDate.addError(dcInput.getWarning());
+                    } else {
                         fullDate.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         fullDate.enableAddOperation();
                 if ((dcInput.isRepeatable() || dcValues.length > 1) && !readonly)
@@ -632,7 +640,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         fullSeries.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        fullSeries.addError(dcInput.getWarning());
+                    } else {
                         fullSeries.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         fullSeries.enableAddOperation();
                 if ((dcInput.isRepeatable() || dcValues.length > 1) && !readonly)
@@ -698,7 +710,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         qualdrop.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        qualdrop.addError(dcInput.getWarning());
+                    } else {
                         qualdrop.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         qualdrop.enableAddOperation();
                 // Update delete based upon the filtered values.
@@ -776,7 +792,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         textArea.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        textArea.addError(dcInput.getWarning());
+                    } else {
                         textArea.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         textArea.enableAddOperation();
                 if ((dcInput.isRepeatable() || dcValues.length > 1) && !readonly)
@@ -845,7 +865,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         select.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        select.addError(dcInput.getWarning());
+                    } else {
                         select.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() || dcValues.length > 1)
                 {
                         // Use the multiple functionality from the HTML
@@ -900,7 +924,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         select.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        select.addError(dcInput.getWarning());
+                    } else {
                         select.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() || dcValues.length > 1)
                 {
                         // Use the multiple functionality from the HTML
@@ -974,8 +1002,12 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         listField.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        listField.addError(dcInput.getWarning());
+                    } else {
                         listField.addError(T_required_field);
-                
+                    }
+
         
                 //Setup each of the possible options
                 java.util.List<String> pairs = dcInput.getPairs();
@@ -1050,7 +1082,11 @@ public class DescribeStep extends AbstractSubmissionStep
                 if (dcInput.isRequired())
                         text.setRequired();
                 if (isFieldInError(fieldName))
+                    if (dcInput.getWarning() != null && dcInput.getWarning().length() > 0) {
+                        text.addError(dcInput.getWarning());
+                    } else {
                         text.addError(T_required_field);
+                    }
                 if (dcInput.isRepeatable() && !readonly)
                         text.enableAddOperation();
                 if ((dcInput.isRepeatable() || dcValues.length > 1) && !readonly)

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/cocoon/DSpaceMETSGenerator.java

@@ -184,32 +184,38 @@ public class DSpaceMETSGenerator extends AbstractGenerator
          	if (parts.length == 2)
          	{
          		String type = parts[0];
-         		int id = Integer.valueOf(parts[1]);
-         		
-         		if ("item".equals(type))
-         		{
-         			Item item = Item.find(context,id);
-         			if (item != null)
-                                        adapter = new ItemAdapter(context,item,contextPath);
-         		}
-         		else if ("collection".equals(type))
-         		{
-         			Collection collection = Collection.find(context,id);
-         			if (collection != null)
-         				adapter = new ContainerAdapter(context, collection,contextPath);
-         		}
-         		else if ("community".equals(type))
-         		{
-         			Community community = Community.find(context,id);
-         			if (community != null)
-         				adapter = new ContainerAdapter(context, community,contextPath);
-         		}
-         		else if ("repository".equals(type))
-     			{
-         			if (ConfigurationManager.getProperty("handle.prefix").equals(String.valueOf(id)))
-                        adapter = new RepositoryAdapter(context,contextPath);
-     			}
-         		
+                       String strid = parts[1];
+         		int id = 0;
+
+                        // Handle prefixes must be treated as strings
+                        // all non-repository types need integer IDs
+                        if ("repository".equals(type))
+                        {
+                                if (ConfigurationManager.getProperty("handle.prefix").equals(strid))
+                                        adapter = new RepositoryAdapter(context,contextPath);
+                        }
+                        else
+                        {
+                               id = Integer.valueOf(parts[1]); 
+         			if ("item".equals(type))
+         			{
+         				Item item = Item.find(context,id);
+         				if (item != null)
+                                       	        adapter = new ItemAdapter(context,item,contextPath);
+         			}
+         			else if ("collection".equals(type))
+         			{
+         				Collection collection = Collection.find(context,id);
+         				if (collection != null)
+         					adapter = new ContainerAdapter(context, collection,contextPath);
+         			}
+         			else if ("community".equals(type))
+         			{
+         				Community community = Community.find(context,id);
+         				if (community != null)
+         					adapter = new ContainerAdapter(context, community,contextPath);
+         			}
+			}
          	}
          }
 		 return adapter;

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/utils/AuthenticationUtil.java

@@ -564,7 +564,11 @@ public class AuthenticationUtil
         	
         	// Return the path for which this request belongs too. Only urls
         	// for this path may be resumed.
-        	return interruptedRequest.getServletPath();
+        	if (interruptedRequest.getServletPath() == null || interruptedRequest.getServletPath().length() == 0) {
+                return interruptedRequest.getActualPath();
+            } else {
+        	    return interruptedRequest.getServletPath();
+            }
         }
         
         // No request was interrupted.

dspace-xmlui/dspace-xmlui-api/src/main/java/org/dspace/app/xmlui/utils/RequestInfo.java

@@ -117,6 +117,14 @@ public class RequestInfo
     {
     	return this.servletPath;
     }
+
+    /**
+     * Return the servlet path that this request is for.
+     */
+    public String getActualPath()
+    {
+    	return this.pathInfo + ((queryString == null || queryString.length() == 0) ? "" : "?"+queryString);
+    }
     
     /**
      * Wrap an incoming request to make it look like the request that the

dspace-xmlui/dspace-xmlui-webapp/pom.xml

@@ -13,7 +13,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-xmlui</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -34,9 +34,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-webapp</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-webapp</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-webapp</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-webapp</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-webapp</url>
    </scm>
 
    <build>

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/WEB-INF/spring/applicationContext.xml

@@ -3,8 +3,8 @@
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:util="http://www.springframework.org/schema/util"
     xmlns:configurator="http://cocoon.apache.org/schema/configurator"
     xmlns:avalon="http://cocoon.apache.org/schema/avalon" xmlns:servlet="http://cocoon.apache.org/schema/servlet"
-    xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-2.0.xsd
-                           http://www.springframework.org/schema/util http://www.springframework.org/schema/util/spring-util-2.0.xsd
+    xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-2.5.xsd
+                           http://www.springframework.org/schema/util http://www.springframework.org/schema/util/spring-util-2.5.xsd
                            http://cocoon.apache.org/schema/configurator http://cocoon.apache.org/schema/configurator/cocoon-configurator-1.0.1.xsd
                            http://cocoon.apache.org/schema/avalon http://cocoon.apache.org/schema/avalon/cocoon-avalon-1.0.xsd
                            http://cocoon.apache.org/schema/servlet http://cocoon.apache.org/schema/servlet/cocoon-servlet-1.0.xsd">
@@ -17,7 +17,7 @@
     <!-- Activate Avalon Bridge -->
     <avalon:bridge />
 
-    <bean id="dspace.org.myApplication" class="org.apache.cocoon.servlet.SitemapServlet">
+    <bean id="dspace.org.myApplication" class="org.apache.cocoon.sitemap.SitemapServlet">
         <servlet:context mount-path="" context-path="" />
     </bean>
 

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/i18n/messages.xml

@@ -168,6 +168,7 @@
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.choose_year">(Choose year)</message>	
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.jump_year">Or type in a year: </message>
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.jump_year_help">Browse for items that are from the given year.</message>
+        <message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.no_results">Sorry, there are no results for this browse.</message>
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.sort_by"> Sort by: </message>
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.order"> Order: </message>
 	<message key="xmlui.ArtifactBrowser.ConfigurableBrowse.general.rpp"> Results: </message><!-- /Page -->
@@ -1277,6 +1278,8 @@
 	<message key="xmlui.administrative.item.MoveItemForm.collection_help">Select the collection you wish to move this item to.</message>
 	<message key="xmlui.administrative.item.MoveItemForm.collection_default">Select a collection...</message>
 	<message key="xmlui.administrative.item.MoveItemForm.submit_move">Move</message>
+    <message key="xmlui.administrative.item.MoveItemForm.inherit_policies">Inherit policies</message>
+    <message key="xmlui.administrative.item.MoveItemForm.inherit_policies_help">Inherit the default policies of the destination collection</message>
 
 	<!-- org.dspace.app.xmlui.administrative.item.EditBitstreamForm -->
 	<message key="xmlui.administrative.item.EditBitstreamForm.title">Edit Bitstream</message>
@@ -1383,14 +1386,18 @@
         <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.changes_committed">Changes applied to item</message>
         <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.item_added">Added: </message>
         <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.item_removed">Removed: </message>
-        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_added">Added to collection</message>
-        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_removed">Removed from collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_newowner">Added to owning collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_oldowner">Removed from owning collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_mapped">Mapped to collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportConfirm.collection_unmapped">Unmapped from collection</message>
         
         <!-- org.dspace.app.xmlui.administrative.metadataimport.MetadataImportUpload -->
         <message key="xmlui.administrative.metadataimport.MetadataImportUpload.item_add">Add: </message>
         <message key="xmlui.administrative.metadataimport.MetadataImportUpload.item_remove">Remove: </message>
-        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_add">Add to collection</message>
-        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_remove">Remove from collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_newowner">Add to owning collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_oldowner">Remove from owning collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_mapped">Map to collection</message>
+        <message key="xmlui.administrative.metadataimport.MetadataImportUpload.collection_unmapped">Un-map from collection</message>
         <message key="xmlui.administrative.metadataimport.MetadataImportUpload.changes_pending">Changes pending for item</message>
         <message key="xmlui.administrative.metadataimport.MetadataImportUpload.submit_confirm">Apply changes</message>
         <message key="xmlui.administrative.metadataimport.MetadataImportUpload.hint">Pending changes are listed below for review</message>

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/themes/Kubrick/Kubrick.xsl

@@ -800,24 +800,40 @@
                         </span>
                         <xsl:text>, </xsl:text>
                     </xsl:if>
+                    
                     <span class="date">
-                        <!--
-		    	<xsl:value-of select="substring(dim:field[@element='date' and @qualifier='issued']/node(),1,10)"/>
-			-->
-			<xsl:call-template name="month-name">
-				<xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>					
-			</xsl:call-template>
-			<xsl:text> </xsl:text>
-			<xsl:call-template name="day-in-month">
-				<xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>
-			</xsl:call-template>
-			<xsl:text>, </xsl:text>
-			<xsl:call-template name="year">
-				<xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>
-			</xsl:call-template>				
+
+                        <xsl:variable name="month-issued" select= "substring(dim:field[@element='date' and @qualifier='issued']/node(),6,2)"/>
+                        <xsl:variable name="day-issued" select= "substring(dim:field[@element='date' and @qualifier='issued']/node(),9,2)"/>
+
+                        <xsl:if test="string-length($month-issued) > 0">
+
+                            <xsl:call-template name="month-name">
+                                <xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>
+                            </xsl:call-template>
+
+                            <xsl:text> </xsl:text>
+
+                            <xsl:if test="string-length($day-issued) > 0">
+
+                                <xsl:call-template name="day-in-month">
+                                    <xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>
+                                </xsl:call-template>
+
+                            </xsl:if>
+
+                            <xsl:text>, </xsl:text>
+
+                        </xsl:if>
+
+                        <xsl:call-template name="year">
+                            <xsl:with-param name="date-time" select = "dim:field[@element='date' and @qualifier='issued']/node()"/>
+                        </xsl:call-template>
+
                     </span>
                     <xsl:text>)</xsl:text>
                 </span>
+
             </div>
         </div>
     </xsl:template>

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/themes/dri2xhtml/DIM-Handler.xsl

@@ -829,7 +829,7 @@
         	</div>
         </xsl:if>
         
-        <xsl:if test="string-length(dim:field[@element='rights'][not(@qualifier)])&gt;0 or string-length(dim:field[@element='rights'][@qualifier='license'])&gt;0">
+        <xsl:if test="string-length(dim:field[@element='rights'][not(@qualifier)])&gt;0">
         	<div class="detail-view-rights-and-license">
 		        <xsl:if test="string-length(dim:field[@element='rights'][not(@qualifier)])&gt;0">
 		            <p class="copyright-text">

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/themes/dri2xhtml/General-Handler.xsl

@@ -206,7 +206,7 @@
                 </xsl:choose>                        
             </td>
 	    <!-- Display the contents of 'Description' as long as at least one bitstream contains a description -->
-	    <xsl:if test="$context/mets:fileSec/mets:fileGrp/mets:file/mets:FLocat/@xlink:label != ''">
+	    <xsl:if test="$context/mets:fileSec/mets:fileGrp[@USE='CONTENT']/mets:file/mets:FLocat/@xlink:label != ''">
 	        <td>
 	            <xsl:value-of select="mets:FLocat[@LOCTYPE='URL']/@xlink:label"/>
 	        </td>

dspace-xmlui/dspace-xmlui-webapp/src/main/webapp/themes/dri2xhtml/MODS-Handler.xsl

@@ -552,9 +552,9 @@
                 <xsl:value-of select="mods:note[1]"/>
             </p>
         </xsl:if>
-        <xsl:if test="string-length(mods:accessCondition[@type='useAndReproducation'])&gt;0">
+        <xsl:if test="string-length(mods:accessCondition[@type='useAndReproduction'])&gt;0">
             <p class="copyright-text">
-                <xsl:value-of select="mods:accessCondition[@type='useAndReproducation']"/>
+                <xsl:value-of select="mods:accessCondition[@type='useAndReproduction']"/>
             </p>
         </xsl:if>
         <xsl:if test="string-length(mods:extension/dim:field[@mdschema='dc' and @element='rights.license'])&gt;0">
@@ -583,9 +583,9 @@
                 <xsl:value-of select="mods:note[1]"/>
             </p>
         </xsl:if>
-        <xsl:if test="string-length(mods:accessCondition[@type='useAndReproducation'])&gt;0">
+        <xsl:if test="string-length(mods:accessCondition[@type='useAndReproduction'])&gt;0">
             <p class="copyright-text">
-                <xsl:value-of select="mods:accessCondition[@type='useAndReproducation']"/>
+                <xsl:value-of select="mods:accessCondition[@type='useAndReproduction']"/>
             </p>
         </xsl:if>
     </xsl:template>   

dspace-xmlui/dspace-xmlui-wing/pom.xml

@@ -10,7 +10,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-xmlui</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -31,9 +31,9 @@
    </repositories>
 
    <scm>
-      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-wing</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-wing</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui/dspace-xmlui-wing</url>
+      <connection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-wing</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui/dspace-xmlui-wing</url>
    </scm>
 
    <dependencies>
@@ -67,15 +67,21 @@
       </dependency>
       
       <dependency>
-      	<groupId>org.apache.cocoon</groupId>
-      	<artifactId>cocoon-servlet-service-impl</artifactId>
-      	<version>1.2.0</version>
+          <groupId>org.apache.cocoon</groupId>
+          <artifactId>cocoon-servlet-service-components</artifactId>
+          <version>1.0.0</version>
+          <exclusions>
+              <exclusion>
+                  <groupId>org.apache.cocoon</groupId>
+      	          <artifactId>cocoon-servlet-service-impl</artifactId>
+              </exclusion>
+          </exclusions>
       </dependency>
       
       <dependency>
-      	<groupId>org.apache.cocoon</groupId>
-      	<artifactId>cocoon-linkrewriter-impl</artifactId>
-      	<version>1.0.0</version>
+      	<groupId>org.dspace.dependencies.cocoon</groupId>
+      	<artifactId>dspace-cocoon-servlet-service-impl</artifactId>
+      	<version>1.0.1</version>
       </dependency>
       
       <dependency>

dspace-xmlui/pom.xml

@@ -12,7 +12,7 @@
    <parent>
       <groupId>org.dspace</groupId>
       <artifactId>dspace-parent</artifactId>
-      <version>1.6.1-SNAPSHOT</version>
+      <version>1.6.2</version>
    </parent>
 
    <repositories>
@@ -38,9 +38,9 @@
       into the snapshot repository below.
    -->
    <scm>
-      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui</connection>
-      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui</developerConnection>
-      <url>http://scm.dspace.org/svn/repo/dspace/trunk/dspace-xmlui</url>
+      <connection>scm:svn:http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui</connection>
+      <developerConnection>scm:svn:https://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui</developerConnection>
+      <url>http://scm.dspace.org/svn/repo/dspace/tags/dspace-1.6.2/dspace-xmlui</url>
    </scm>
    
    <modules>

dspace/CHANGES

@@ -1,6 +1,98 @@
-1.6.0
+1.6.2
 =====
 
+(Tim Donohue, Kim Shepherd, Keith Gilbertson, Stuart Lewis)
+- [DS-584] start-handle-server script broken - Error in launcher.xml: Invalid class name
+
+(Stuart Lewis)
+- [DS-607] Invalid identifiers not escaped
+- [DS-608] Batch metadata import missing item headers
+
+(Ivan Masar, Claudia Jurgen)
+- [DS-567] Czech localization of 1.6.0
+- [DS-595] Czech localization of 1.6.1
+
+1.6.1
+=====
+
+(Javier Garrido)
+- [DS-582] Basque translation (of jspui for DSpace 1.5.2)
+
+(Mark Diggory)
+- [DS-571] Upgrade DSpace Services to next release 2.0.1
+
+
+(Dale Poulter)
+- [DS-557] LC Authority Names - Lookup Feature - names w/o dates
+
+(Paulo Jobim)
+- [DS-471] Accessing site-level 'mets.xml' in XMLUI doesn't work properly for handle prefixes with periods (e.g. 2010.1)
+
+(Yin Yin Latt)
+- [DS-565] Empty description column in Itemview Page - General-Handler.xsl
+- [DS-566] Side bar menu dropping when there is license text in collection - DIM-Handler.xsl
+
+(Robin Taylor)
+- [DS-497] Date month and day get default values when user returns to describe form
+- [DS-501] Change kubrick.xsl to check for the presence of a day and month before displaying them
+
+(Keiji Suzuki)
+- [DS-537] Malformed Japanese option values in the authority lookup window
+
+(Peter Dietz)
+- [DS-509] SOLR returns ArrayIndexOutOfBounds with non-existent country code OR when there is no view/download data
+- [DS-542] verbose output for stats-log-importer displays spurious city/country from previous committed entry
+
+(Kim Shepherd)
+- [DS-547] Value for Recent Submissions is not workin in the XMLUI
+- [DS-527] Withdrawn items not shown as deleted in OAI
+- [DS-548] Removing repeatable values in DescribeStep does not properly test for authority control
+- [DS-544] Removal of mapped items can lead to NPE
+
+(Claudia Jurgen)
+- [DS-558] Error in update sequence script 1.5 to 1.6 Oracle
+- [DS-551] Export directories dspace.cfg and build.xml out of sync
+- [DS-543] Harvest not internationalized
+
+(Serhij Dubyk)
+- [DS-500] Ukrainian for DSpace 1.6.0
+
+(Andrea Bollini)
+ - [DS-513] Connection leak in SWORD authentication process 
+ - [DS-581] DIDL doesn't respect the hidden fields and the oai_dc metadata section is different than the simple oai_dc implementation
+ - [DS-580] DIDL format include HTML element if the item has no files
+ - [DS-574] DSpaceMETSIngester creates empty original bundle
+ - [DS-573] NPE resuming submission for item with an empty bundle original
+ 
+(Caryn Neiswender)
+ - [DS-518] Duplicate listing of dependencies in dspace-sword/pom.xml
+
+(Keith Gilbertson)
+ - [DS-123] xmlui browse in empty collection displays "Now showing items 1-0" of 0 - incorrect numbering
+ - [DS-508] Attachment spelled as attachement in DailyReportEmailer
+ - [DS-539] Misspelled attribute in MODS/METS output
+
+(Stuart Lewis)
+ - [DS-242] Special groups shown for logged in user rather than for user being examined
+ - [DS-506] embargo-lifter command missing from launcher.xml
+ - [DS-516] DSRUN does not start Service Manager
+ - [DS-526] ItemUpdate - script update
+ - [DS-572] Batch metadata editor fails to notice change of item's owning collection
+
+(Mark Wood)
+ - [DS-469] DCDate.displayDate(false,*) displays only year
+
+ (Ivan Masar)
+ - [DS-515] Czech localization of 1.5.2
+
+(Ben Bosman)
+ - [DS-579] Required fields in submissions display wrong error message
+ - [DS-538] restricted items are being returned in OAI GetRecord method while using harvest.includerestricted.oai
+ - [DS-493] Url in browser is incorrect after login
+
+1.6.0 final
+===========
+
 (Alexey Maslov)
  - [DS-289] OAI-PMH + OAI-ORE harvesting support
 

dspace/bin/checker

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.checker.ChecksumChecker $@
+$BINDIR/dspace checker "$@"

dspace/bin/cleanup

@@ -47,7 +47,7 @@ BINDIR=`dirname $0`
 
 echo "Cleaning the asset store"
 
-$BINDIR/dsrun org.dspace.storage.bitstore.Cleanup "$@"
+$BINDIR/dspace cleanup "$@"
 
 echo "Cleanup completed"
 

dspace/bin/create-administrator

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.administer.CreateAdministrator "$@"
+$BINDIR/dspace create-administrator "$@"

dspace/bin/dspace

@@ -39,10 +39,27 @@
 # DAMAGE.
 #
 ###########################################################################
+#
+# This is a simple shell script for running a command-line DSpace tool.
+# It sets the CLASSPATH appropriately before invoking Java.
 
-# Script for running the DSpace launcher.
-
-# Get the DSPACE/bin directory
 BINDIR=`dirname $0`
+DSPACEDIR=`cd "$BINDIR/.." ; pwd`
 
-$BINDIR/dsrun org.dspace.app.launcher.ScriptLauncher $@
+# Get the JARs in $DSPACEDIR/jsp/WEB-INF/lib, separated by ':'
+JARS=`echo $DSPACEDIR/lib/*.jar | sed 's/ /\:/g'`
+
+# Class path for DSpace will be:
+#   Any existing classpath
+#   The JARs (WEB-INF/lib/*.jar)
+#   The WEB-INF/classes directory
+FULLPATH=$CLASSPATH:$JARS:$DSPACEDIR/config
+
+#Allow user to specify java options through JAVA_OPTS variable
+if [ "$JAVA_OPTS" = "" ]; then
+  #Default Java to use 256MB of memory
+  JAVA_OPTS=-Xmx256m
+fi
+
+# Now invoke Java
+java $JAVA_OPTS -classpath $FULLPATH org.dspace.app.launcher.ScriptLauncher "$@"

dspace/bin/dspace.bat

@@ -38,4 +38,37 @@
 @REM #
 @REM ###########################################################################
 
-@call dsrun.bat org.dspace.app.launcher.ScriptLauncher %*
+@echo off
+
+set CURRENT_DIR=%cd%
+
+REM Guess DSpace directory: CD to directory script is in; CD to parent
+chdir /D "%~p0"
+chdir ..
+
+REM Check we can find dspace.cfg.  Quit with an error if not.
+if exist "config\dspace.cfg" goto okExec
+echo Cannot find %cd%\config\dspace.cfg
+goto end
+
+:okExec
+echo Using DSpace installation in: %cd%
+
+REM Build a CLASSPATH
+set DSPACE_CLASSPATH=%CLASSPATH%;config
+for %%f in (lib\*.jar) DO CALL bin\buildpath.bat %%f
+
+REM If JAVA_OPTS specified, use those options
+REM Otherwise, default Java to using 256MB of memory
+if "%JAVA_OPTS%"=="" set JAVA_OPTS=-Xmx256m
+
+REM Execute Java
+java %JAVA_OPTS% -classpath "%DSPACE_CLASSPATH%" org.dspace.app.launcher.ScriptLauncher %*
+
+REM Clean up DSPACE_CLASSPATH variable
+set DSPACE_CLASSPATH=
+
+:end
+
+REM Back to original dir
+chdir /D %CURRENT_DIR%

dspace/bin/dsprop

@@ -42,15 +42,7 @@
 # This is a simple shell script for running retrieving the value of a dspace
 # property 
 
-USAGE="dsprop <property.name>"
-
-# Check we have at least one command-line argument
-if [ "$#" != "1" ]; then
-    echo $USAGE
-    exit 1
-fi
-
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.core.ConfigurationManager -property "$1"
+$BINDIR/dspace dsprop -property "$1"

dspace/bin/dsrun

@@ -43,7 +43,7 @@
 # This is a simple shell script for running a command-line DSpace tool.
 # It sets the CLASSPATH appropriately before invoking Java.
 
-USAGE="dsrun <classname> [arg1 [arg2 ...]]"
+USAGE="dspace dsrun <classname> [arg1 [arg2 ...]]"
 
 # Check we have at least one command-line argument
 if [ "$#" = "0" ]; then
@@ -51,24 +51,5 @@ if [ "$#" = "0" ]; then
     exit 1
 fi
 
-# Assume we're in the bin subdirectory of the DSpace installation directory
 BINDIR=`dirname $0`
-DSPACEDIR=`cd "$BINDIR/.." ; pwd`
-
-# Get the JARs in $DSPACEDIR/jsp/WEB-INF/lib, separated by ':'
-JARS=`echo $DSPACEDIR/lib/*.jar | sed 's/ /\:/g'`
-
-# Class path for DSpace will be:
-#   Any existing classpath
-#   The JARs (WEB-INF/lib/*.jar)
-#   The WEB-INF/classes directory
-FULLPATH=$CLASSPATH:$JARS:$DSPACEDIR/config
-
-#Allow user to specify java options through JAVA_OPTS variable
-if [ "$JAVA_OPTS" = "" ]; then
-   #Default Java to use 256MB of memory
-   JAVA_OPTS=-Xmx256m
-fi
-
-# Now invoke Java
-java $JAVA_OPTS -classpath $FULLPATH "$@"
+$BINDIR/dspace dsrun "$@"

dspace/bin/dsrun.bat

@@ -43,49 +43,4 @@
 REM This is a simple shell script for running a command-line DSpace tool.
 REM sets the CLASSPATH appropriately before invoking Java.
 
-REM Remember startup dir
-
-set CURRENT_DIR=%cd%
-
-
-REM Guess DSpace directory: CD to directory script is in; CD to parent
-
-chdir /D "%~p0"
-chdir ..
-
-REM Check we can find dspace.cfg.  Quit with an error if not.
-
-if exist "config\dspace.cfg" goto okExec
-echo Cannot find %cd%\config\dspace.cfg
-goto end
-
-
-:okExec
-
-echo Using DSpace installation in: %cd%
-
-REM Build a CLASSPATH
-
-set DSPACE_CLASSPATH=%CLASSPATH%;config
-for %%f in (lib\*.jar) DO CALL bin\buildpath.bat %%f
-
-REM If JAVA_OPTS specified, use those options
-REM Otherwise, default Java to using 256MB of memory
-
-if "%JAVA_OPTS%"=="" set JAVA_OPTS=-Xmx256m
-
-REM Execute Java
-
-java %JAVA_OPTS% -classpath "%DSPACE_CLASSPATH%" %*
-
-
-REM Clean up DSPACE_CLASSPATH variable
-
-set DSPACE_CLASSPATH=
-
-
-:end
-
-REM Back to original dir
-
-chdir /D %CURRENT_DIR%
+@call dspace.bat dsrun %*

dspace/bin/export

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.itemexport.ItemExport $@
+$BINDIR/dspace export "$@"

dspace/bin/filter-media

@@ -46,4 +46,4 @@
 BINDIR=`dirname $0`
 
 echo "Applying Media Filters"
-$BINDIR/dsrun org.dspace.app.mediafilter.MediaFilterManager $@
+$BINDIR/dspace filter-media "$@"

dspace/bin/generate-sitemaps

@@ -46,4 +46,4 @@
 BINDIR=`dirname $0`
 
 echo "Generating sitemaps"
-$BINDIR/dsrun org.dspace.app.sitemap.GenerateSitemaps $@
+$BINDIR/dspace generate-sitemaps "$@"

dspace/bin/import

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.itemimport.ItemImport $@
+$BINDIR/dspace import "$@"

dspace/bin/index-init

@@ -45,9 +45,5 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-echo "Creating browse index"
-$BINDIR/dsrun org.dspace.browse.IndexBrowse -f -r
-$BINDIR/dsrun org.dspace.browse.ItemCounter
-
-echo "Creating search index"
-$BINDIR/dsrun org.dspace.search.DSIndexer -b
+echo "Creating browse and search indexes"
+$BINDIR/dspace index-init

dspace/bin/index-update

@@ -45,9 +45,5 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-echo "Creating browse index"
-$BINDIR/dsrun org.dspace.browse.IndexBrowse -i
-$BINDIR/dsrun org.dspace.browse.ItemCounter
-
-echo "Creating search index"
-$BINDIR/dsrun org.dspace.search.DSIndexer
+echo "Creating browse and search indexes"
+$BINDIR/dspace index-update

dspace/bin/itemcounter

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.browse.ItemCounter $@
+$BINDIR/dspace itemcounter "$@"

dspace/bin/metadata-export

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.bulkedit.MetadataExport $@
+$BINDIR/dspace metadata-export "$@"

dspace/bin/metadata-import

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.bulkedit.MetadataImport $@
+$BINDIR/dspace metadata-import "$@"

dspace/bin/packager

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.packager.Packager $@
+$BINDIR/dspace packager "$@"

dspace/bin/start-handle-server

@@ -8,7 +8,7 @@
 #
 # Date: $Date$
 #
-# Copyright (c) 2002-2009, The DSpace Foundation.  All rights reserved.
+# Copyright (c) 2002-2010, The DSpace Foundation.  All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
@@ -45,6 +45,22 @@
 
 # Assume we're in the bin subdirectory of the DSpace installation directory
 BINDIR=`dirname $0`
+DSPACEDIR=`cd "$BINDIR/.." ; pwd`
+
+# Get the JARs in $DSPACEDIR/jsp/WEB-INF/lib, separated by ':'
+JARS=`echo $DSPACEDIR/lib/*.jar | sed 's/ /\:/g'`
+
+# Class path for DSpace will be:
+#   Any existing classpath
+#   The JARs (WEB-INF/lib/*.jar)
+#   The WEB-INF/classes directory
+FULLPATH=$CLASSPATH:$JARS:$DSPACEDIR/config
+
+#Allow user to specify java options through JAVA_OPTS variable
+if [ "$JAVA_OPTS" = "" ]; then
+ #Default Java to use 256MB of memory
+ JAVA_OPTS=-Xmx256m
+fi
 
 # Read parameters from DSpace config
 handledir=`$BINDIR/dsrun org.dspace.core.ConfigurationManager -property handle.dir`
@@ -56,4 +72,4 @@ rm -f $handledir/txns/lock
 # Start the Handle server, with a special log4j properties file.
 # We cannot simply write to the same logs, since log4j
 # does not support more than one JVM writing to the same rolling log.
-nohup $BINDIR/dsrun -Ddspace.log.init.disable=true -Dlog4j.configuration=log4j-handle-plugin.properties net.handle.server.Main $handledir </dev/null >> $logdir/handle-server.log 2>&1 &
+nohup java $JAVA_OPTS -classpath $FULLPATH -Ddspace.log.init.disable=true -Dlog4j.configuration=log4j-handle-plugin.properties net.handle.server.Main $handledir </dev/null >> $logdir/handle-server.log 2>&1 &

dspace/bin/stat-general

@@ -46,4 +46,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-general
+$BINDIR/dspace stat-general

dspace/bin/stat-initial

@@ -48,4 +48,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-initial
+$BINDIR/dspace stat-initial

dspace/bin/stat-monthly

@@ -47,4 +47,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-monthly
+$BINDIR/dspace stat-monthly

dspace/bin/stat-report-general

@@ -46,4 +46,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-report-general
+$BINDIR/dspace stat-report-general

dspace/bin/stat-report-initial

@@ -48,4 +48,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-report-initial
+$BINDIR/dspace stat-report-initial

dspace/bin/stat-report-monthly

@@ -47,4 +47,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.app.statistics.CreateStatReport -r stat-report-monthly
+$BINDIR/dspace stat-report-monthly

dspace/bin/structure-builder

@@ -45,4 +45,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.administer.StructBuilder $@
+$BINDIR/dspace structure-builder "$@"

dspace/bin/sub-daily

@@ -46,4 +46,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.eperson.Subscribe $@
+$BINDIR/dspace sub-daily "$@"

dspace/bin/update-handle-prefix

@@ -46,4 +46,4 @@
 # Get the DSPACE/bin directory
 BINDIR=`dirname $0`
 
-$BINDIR/dsrun org.dspace.handle.UpdateHandlePrefix $@
+$BINDIR/dspace update-handle-prefix "$@"

dspace/config/crosswalks/mods.properties

@@ -57,7 +57,7 @@ dc.relation.isversionof = <mods:relatedItem type="otherVersion">%s</mods:related
 dc.relation.replaces = <mods:relatedItem type="preceding">%s</mods:relatedItem> | text()
 dc.relation.requires = <mods:note type="requires">%s</mods:note> | text()
 dc.relation.uri = <mods:relatedItem><mods:location><mods:url>%s</mods:url></mods:location></mods:relatedItem> | mods:location/mods:url/text()
-dc.rights = <mods:accessCondition type="useAndReproducation">%s</mods:accessCondition> | text()
+dc.rights = <mods:accessCondition type="useAndReproduction">%s</mods:accessCondition> | text()
 dc.rights.uri = <mods:accessCondition xlink:simpleLink="%s">%s</mods:accessCondition> | @xlink:simpleLink|text()
 dc.source = <mods:relatedItem type="original">%s</mods:relatedItem> | text()
 dc.source.uri = <mods:relatedItem type="original" xlink:simpleLink="%s">%s</mods:relatedItem> | @xlink:simpleLink|text()

dspace/config/dc2mods.cfg

@@ -56,7 +56,7 @@ relation.requires = <mods:note type="requires">%s</mods:note>
 relation.uri = <mods:relatedItem><mods:location><mods:url>%s</mods:url></mods:location></mods:relatedItem>
 relation = <mods:relatedItem><mods:title>%s</mods:title></mods:relatedItem>
 rights.uri = <mods:accessCondition xlink:simpleLink="%s">%s</mods:accessCondition>
-rights = <mods:accessCondition type="useAndReproducation">%s</mods:accessCondition>
+rights = <mods:accessCondition type="useAndReproduction">%s</mods:accessCondition>
 source.uri = <mods:relatedItem type="original" xlink:simpleLink="%s">%s</mods:relatedItem>
 source = <mods:relatedItem type="original">%s</mods:relatedItem>
 subject.classification = <mods:classification>%s</mods:classification>

dspace/config/dspace.cfg

@@ -1959,6 +1959,8 @@ plugin.named.org.dspace.sword.SWORDIngester = \
 
 # Crosswalk settings; the {name} value must correspond to a declated ingestion crosswalk
 # harvester.oai.metadataformats.{name} = {namespace},{optional display name}
+# The display name is only used in the xmlui for the jspui there are entries in the
+# Messages.properties in the form jsp.tools.edit-collection.form.label21.select.{name}
 harvester.oai.metadataformats.dc = http://www.openarchives.org/OAI/2.0/oai_dc/, Simple Dublin Core
 harvester.oai.metadataformats.qdc = http://purl.org/dc/terms/, Qualified Dublin Core
 harvester.oai.metadataformats.dim = http://www.dspace.org/xmlns/dspace/dim, DSpace Intermediate Metadata
@@ -2034,7 +2036,12 @@ harvester.unknownSchema = fail
 #---------------------------------------------------------------#
 
 ##### Usage Logging #####
-solr.log.server = ${dspace.baseUrl}/solr/statistics
+# set this to be the port you run the dspace "solr" webapp
+# on, by default, we are assuming a test configuration with
+# tomcat still running on port 8080
+solr.log.server = http://localhost:8080/solr/statistics
+
+# The location for the Geo Database retrieved on update/installation
 solr.dbfile = ${dspace.dir}/config/GeoLiteCity.dat
 
 # If enabled the statistics system will look for an X-Forward header

dspace/config/launcher.xml

@@ -49,6 +49,22 @@
         </step>
     </command>
 
+    <command>
+        <name>dsrun</name>
+        <description>Run a class directly</description>
+        <step>
+            <class>dsrun</class>
+        </step>
+    </command>
+
+    <command>
+        <name>embargo-lifter</name>
+        <description>Embargo manager tool used to check, list and lift embargoes</description>
+        <step>
+            <class>org.dspace.embargo.EmbargoManager</class>
+        </step>
+    </command>
+
     <command>
         <name>export</name>
         <description>Export items or collections</description>
@@ -141,7 +157,7 @@
         <name>itemupdate</name>
         <description>Item update tool for altering metadata and bitstream content in items</description>
         <step>
-            <class>org.dspace.app.itemimport.ItemUpdate</class>
+            <class>org.dspace.app.itemupdate.ItemUpdate</class>
         </step>
     </command>
 
@@ -277,6 +293,14 @@
         </step>
     </command>
 
+    <command>
+        <name>sub-daily</name>
+        <description>Send daily subscription notices</description>
+        <step>
+            <class>org.dspace.eperson.Subscribe</class>
+        </step>
+    </command>
+    
     <command>
         <name>test-database</name>
         <description>Test the DSpace database connection is OK</description>
@@ -293,14 +317,6 @@
         </step>
     </command>
 
-    <command>
-        <name>sub-daily</name>
-        <description>Send daily subscription notices</description>
-        <step>
-            <class>org.dspace.eperson.Subscribe</class>
-        </step>
-    </command>
-
     <command>
         <name>update-handle-prefix</name>
         <description>Update handle records and metadata when moving from one handle to another</description>

dspace/docs/docbook/DRISchemaReference.xml

@@ -25,7 +25,7 @@
       <section remap="h3">
         <title>Themes</title>
         <para>A Theme is a collection of XSL stylesheets and supporting files like images, CSS styles, translations, and help documents. The XSL stylesheets are applied to the DRI Document to covert it into a readable format and give it structure and basic visual formatting in that format. The supporting files are used to provide the page with a specific look and feel, insert images and other media, translate the content, and perform other tasks. The currently used output format is XHTML and the supporting files are generally limited to CSS, images, and JavaScript. More output formats, like PDF or SVG, may be added in the future.</para>
-        <para>A DSpace installation running Manakin may have several Themes associated with it. When applied to a page, a Theme determines most of the pageís look and feel. Different themes can be applied to different sets of DSpace pages allowing for both variety of styles between sets of pages and consistency within those sets. The xmlui.xconf configuration file determines which Themes are applied to which DSpace pages (see the <link linkend="docbook-configure.html-xmlui-configure">Configuration and Customization chapter</link> for more information on installing and configuring themes). Themes may be configured to apply to all pages of specific type, like browse-by-title, to all pages of a one particular community or collection or sets of communities and collections, and to any mix of the two. They can also be configured to apply to a singe arbitrary page or handle.</para>
+        <para>A DSpace installation running Manakin may have several Themes associated with it. When applied to a page, a Theme determines most of the pageís look and feel. Different themes can be applied to different sets of DSpace pages allowing for both variety of styles between sets of pages and consistency within those sets. The xmlui.xconf configuration file determines which Themes are applied to which DSpace pages (see the <link linkend="docbook-xmlui.html">Chapter 7. Manakin [XMLUI] Configuration and Customization</link> for more information on installing and configuring themes). Themes may be configured to apply to all pages of specific type, like browse-by-title, to all pages of a one particular community or collection or sets of communities and collections, and to any mix of the two. They can also be configured to apply to a singe arbitrary page or handle.</para>
       </section>
       <section remap="h3">
         <title>Aspect Chains</title>

dspace/docs/docbook/architecture.xml

@@ -1,5 +1,5 @@
 <?xml version='1.0' encoding='UTF-8'?>
-<chapter remap="h1">
+  <chapter remap="h1">
     <title><anchor id="docbook-architecture.html"/>DSpace System Documentation: Architecture</title>
     <section remap="h2">
       <title><anchor id="docbook-architecture.html-overview" xreflabel="Overview"/>Overview</title>

dspace/docs/docbook/book.xml

@@ -22,19 +22,19 @@
 <book>
 	<bookinfo>
 		<title>DSpace Manual</title>
-		<subtitle>Release 1.6.0</subtitle>
+		<subtitle>Release 1.6.2</subtitle>
 		<author>
-			<surname>The DuraSpace Foundation</surname>
+			<surname>DuraSpace</surname>
 			<affiliation>
 				<address><email>webmaster@dspace.org</email></address>
 			</affiliation>
 		</author>
-		<edition>1.6.0</edition>
-		<pubdate>2010-03-02  11:45:00 EST</pubdate>
+		<edition>1.6.2</edition>
+		<pubdate>2010-06-13 08:00:00 EST</pubdate>
 		<copyright>
 			<year>2002-2010</year>
 			<holder>
-				<ulink url="http://www.duraspace.org/">The DuraSpace Foundation</ulink>
+				<ulink url="http://www.duraspace.org/">DuraSpace</ulink>
 			</holder>
 		</copyright>
 		<legalnotice>
@@ -51,8 +51,8 @@
 		</legalnotice>
 		<revhistory>
 			<revision>
-				<revnumber>Version 1.6.0 </revnumber>
-				<date>This text generated: 2010-03-02  11:45:00 EST EST</date>
+				<revnumber>Version 1.6.2 </revnumber>
+				<date>This text generated: 2010-06-13 08:00:00 EST EST</date>
 				<authorinitials>JAT</authorinitials>
 			</revision>
 		</revhistory>
@@ -63,8 +63,8 @@
 	<preface>
 		<title>Preface</title>
 		<para/>
-		<para>Welcome to Release 1.6.0. The committers have volunteered many hours to fix, re-write and contribute new software code for this release. New features have been added and documentation has been updated.</para>
-		<para>Some of the new features include (not an exhaustive list):</para>
+		<para>Welcome to Release 1.6.2. This is a "fix" release. The committers have volunteered many hours to fix, re-write and contribute new software code for this release. Documentation has been updated. No new features have been added to this release.</para>
+		<para>The following is a list of the new features included (not an exhaustive list) for release 1.6.0 and are also contained in 1.6.2:</para>
 		<itemizedlist>
 			<listitem>
 				<para>SOLR Statistics.</para>
@@ -91,10 +91,11 @@
 				<para>Delegated Administration</para>
 			</listitem>
 		</itemizedlist>
-		<para>The following people have contributed to this release of DSpace: @mire, BioMed Central, Andrea Bollini, Ben Bosman, Flavio Botelho, Peter Dietz, Mark Diggory, Tim Donohue, Enovation Solutions, Ladd Hanson, Leonie Hayes, Claudia Juergen, Keller, Fabio Kepler, Yin Yin Latt, Stuart Lewis, Bram Luyten, Van Ly, Alexey Maslov, Gabriela Mircea, Hardik Mishra, Venessa Newton-Wade, OhioLINK, Ekaterina Pechekhonova, Toni Prieto, Richard Rodgers, Ricardo Saraiva, Andreas Schwander, Kim Shepherd, Elin Stangeland, Larry Stone, Graham Triggs, Jeffrey Trimble, Kevin Van de velde, Steve Williams and Mark Wood. Many of them could not do this work without the support (release time and financial) of their institutions that they are associated with. We offer thanks to those institutions for supporting their staff to take time to contribute to the DSpace project.</para>
-		<para>Documentation for this release was overhauled by Jeffrey Trimble, the Documentation Gardener, with many other contributing suggestions, changes, edits, and additions. Special thanks to Bram Luyten for pulling his hair out for re-ordering Chapter 5 to match the order of the <literal>dspace.cfg</literal> file and for the Documentation Gardener who is still picking his own hair up from that re-ordering.</para>
-		<para>Stuart Lewis is the Release Co-ordinator and kept all of the committers inline and fixed things when broken.  Many thanks Stuart.</para>
-		<para>Additional thanks to Tim Donohue from DuraSpace Foundation on keeping all of us focused on the work at hand, and calming us when we got excited and for the general support for the DSpace project.</para>
+		<para>The following people have contributed to this release of DSpace: @mire, BioMed Central, Andrea Bollini, Ben Bosman, Flavio Botelho, Peter Dietz, Mark Diggory, Tim Donohue, Serhij Dubyk, Enovation Solutions, Javier Garrido, Keith Gilbertson, Ladd Hanson, Leonie Hayes, Paulo Jobim, Claudia Juergen, Keller, Fabio Kepler, Yin Yin Latt, Stuart Lewis, Bram Luyten, Van Ly, Ivan Masar, Alexey Maslov, Gabriela Mircea, Hardik Mishra, Caryn, Neiswender, Venessa Newton-Wade, OhioLINK, Ekaterina Pechekhonova, Dale Poulter, Toni Prieto, Richard Rodgers, Ricardo Saraiva, Andreas Schwander, Kim Shepherd, Elin Stangeland, Larry Stone, Keiji Suzuki, Robin Taylor, Graham Triggs, Jeffrey Trimble, Kevin Van de velde, Steve Williams and Mark Wood. Many of them could not do this work without the support (release time and financial) of their institutions that they are associated with. We offer thanks to those institutions for supporting their staff to take time to contribute to the DSpace project.</para>
+		<para>We apologize to any committer left off this list. DSpace has such a large, arctive development community that we sometimes loose track of all our contributors. Our ongoing list of all known people/institutions that contribute to DSpace can be found on our DSpace Contributors Wiki page (http://wiki.dspace.org/confluence/display/DSPACE/DSpaceContributors). Acknowledgements to those left off will be made in future releases.</para>
+		<para>The Documentation Gardener for this release was Jeffrey Trimble with input from everyone. All typos are his fault. </para>
+		<para>Kim Shepherd is the Release Co-ordinator of this release.</para>
+		<para>Additional thanks to Tim Donohue from DuraSpace on keeping all of us focused on the work at hand, and calming us when we got excited and for the general support for the DSpace project.</para>
 	</preface>
 	<toc/>
 	&art01;