From 46ddb556d94db185184880f8085e21f83d329a10 Mon Sep 17 00:00:00 2001
From: Philip Durbin <philip_durbin@harvard.edu>
Date: Fri, 20 Jun 2014 16:22:50 -0400
Subject: [PATCH] swap order of methods within indexAll #4062

---
 .../iq/dataverse/IndexServiceBean.java        | 33 +++++++++----------
 1 file changed, 16 insertions(+), 17 deletions(-)

diff --git a/src/main/java/edu/harvard/iq/dataverse/IndexServiceBean.java b/src/main/java/edu/harvard/iq/dataverse/IndexServiceBean.java
index cf6c01cd5d6..92077ce0a0e 100644
--- a/src/main/java/edu/harvard/iq/dataverse/IndexServiceBean.java
+++ b/src/main/java/edu/harvard/iq/dataverse/IndexServiceBean.java
@@ -77,23 +77,6 @@ public String indexAll() {
             return ex.toString();
         }
 
-        List<Dataverse> dataverses = dataverseService.findAll();
-        int dataverseIndexCount = 0;
-        for (Dataverse dataverse : dataverses) {
-            logger.info("indexing dataverse " + dataverseIndexCount + " of " + dataverses.size() + ": " + indexDataverse(dataverse));
-            dataverseIndexCount++;
-        }
-
-        int datasetIndexCount = 0;
-        List<Dataset> datasets = datasetService.findAll();
-        for (Dataset dataset : datasets) {
-            datasetIndexCount++;
-            logger.info("indexing dataset " + datasetIndexCount + " of " + datasets.size() + ": " + indexDataset(dataset));
-        }
-//        logger.info("advanced search fields: " + advancedSearchFields);
-//        logger.info("not advanced search fields: " + notAdvancedSearchFields);
-        logger.info("done iterating through all datasets");
-
         /**
          * @todo: replace hard-coded groups with real groups
          */
@@ -114,6 +97,22 @@ public String indexAll() {
             userIndexCount++;
             logger.info("indexing user " + userIndexCount + " of several: " + indexUser(user));
         }
+        List<Dataverse> dataverses = dataverseService.findAll();
+        int dataverseIndexCount = 0;
+        for (Dataverse dataverse : dataverses) {
+            logger.info("indexing dataverse " + dataverseIndexCount + " of " + dataverses.size() + ": " + indexDataverse(dataverse));
+            dataverseIndexCount++;
+        }
+
+        int datasetIndexCount = 0;
+        List<Dataset> datasets = datasetService.findAll();
+        for (Dataset dataset : datasets) {
+            datasetIndexCount++;
+            logger.info("indexing dataset " + datasetIndexCount + " of " + datasets.size() + ": " + indexDataset(dataset));
+        }
+//        logger.info("advanced search fields: " + advancedSearchFields);
+//        logger.info("not advanced search fields: " + notAdvancedSearchFields);
+        logger.info("done iterating through all datasets");
 
         return dataverseIndexCount + " dataverses, " + datasetIndexCount + " datasets, " + groupIndexCount + " groups, and " + userIndexCount + " users indexed\n";
     }