Changeset 23050

Timestamp:

2010-10-06T15:01:25+13:00 (14 years ago)

Author:

kjdon

Message:

code was assuming CLx and removing CL then parsing the integer to get the position. But now just use the position in the list to be the position and assume that the coll config and build config lists are the same. then we can get classifier info for eg oai classifier

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/AbstractBrowse.java (modified) (2 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/AbstractBrowse.java

@@ -38 +38 @@
 /** Partially implements a generic classifier service
  *
- * @author <a href="mailto:[email protected]">Katherine Don</a>
  */
 public abstract class AbstractBrowse
@@ -268 +267 @@
         //                         name);
             //now use the position to get the node - CL1 
-            int position = Integer.parseInt(name.substring(2));
+        // assumes the same number of classifiers in collectionCOnfig as in buildConfig
+        //            int position = Integer.parseInt(name.substring(2));
+        
         Element node_extra = null;
             NodeList cls = config_browse.getElementsByTagName(GSXML.CLASSIFIER_ELEM);
-            if (position >0 && position <= cls.getLength()) {
-            node_extra  = (Element) cls.item((position -1)); 
+            //if (position >0 && position <= cls.getLength()) {
+        //    node_extra  = (Element) cls.item((position -1)); 
+        //}
+        if (i<cls.getLength()) {
+          node_extra = (Element) cls.item(i);
         }