You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2012/08/17 16:08:24 UTC

svn commit: r1374273 - /ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java

Author: jacopoc
Date: Fri Aug 17 14:08:23 2012
New Revision: 1374273

URL: http://svn.apache.org/viewvc?rev=1374273&view=rev
Log:
Minor code cleanup: removed empty lines and reused a variable rather than getting it from a GenericValue.

Modified:
    ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java

Modified: ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java?rev=1374273&r1=1374272&r2=1374273&view=diff
==============================================================================
--- ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java (original)
+++ ofbiz/trunk/applications/content/src/org/ofbiz/content/search/SearchWorker.java Fri Aug 17 14:08:23 2012
@@ -59,7 +59,6 @@ public class SearchWorker {
     public static final String module = SearchWorker.class.getName();
 
     public static Map<String, Object> indexTree(LocalDispatcher dispatcher, Delegator delegator, String siteId, Map<String, Object> context, String path) throws Exception {
-
         Map<String, Object> results = FastMap.newInstance();
         GenericValue content = delegator.makeValue("Content", UtilMisc.toMap("contentId", siteId));
         if (Debug.infoOn()) Debug.logInfo("in indexTree, siteId:" + siteId + " content:" + content, module);
@@ -75,11 +74,8 @@ public class SearchWorker {
                     for (GenericValue subContent : subContentList) {
                         contentIdList.add(subContent.getString("contentId"));
                     }
-
                     indexContentList(contentIdList, delegator, dispatcher, context);
-
-                    String subSiteId = siteContent.getString("contentId");
-                    indexTree(dispatcher, delegator, subSiteId, context, path);
+                    indexTree(dispatcher, delegator, siteContentId, context, path);
                 } else {
                     List<String> badIndexList = UtilGenerics.checkList(context.get("badIndexList"));
                     badIndexList.add(siteContentId + " had no sub-entities.");