Always make item removal update the browse tables (may have entries in 'admin' tables).

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

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

@@ -1697,9 +1697,11 @@ public class Item extends DSpaceObject
          ** and the DB transaction will fail.
          ** Any fix would involve too much work on Browse code that
          ** is likely to be replaced soon anyway.   --lcs, Aug 2006
+         **
+         ** NB Do not check to see if the item is archived - withdrawn /
+         ** non-archived items may still be tracked in some browse tables
+         ** for administrative purposes, and these need to be removed.
          **/
-        if (isArchived())
-        {
 //        	 FIXME: there is an exception handling problem here
         try
         {
@@ -1713,8 +1715,6 @@ public class Item extends DSpaceObject
             throw new SQLException(e.getMessage());
         }
 
-        }
-
         // Delete the Dublin Core
         removeMetadataFromDatabase();