[open-ils-commits] r18716 - branches/rel_2_0/Open-ILS/web/opac/common/js (phasefx)

[svn at svn.open-ils.org]

Author: phasefx
Date: 2010-11-12 14:49:34 -0500 (Fri, 12 Nov 2010)
New Revision: 18716

Modified:
   branches/rel_2_0/Open-ILS/web/opac/common/js/opac_utils.js
Log:
ensure that we only fetch this setting once per page

Modified: branches/rel_2_0/Open-ILS/web/opac/common/js/opac_utils.js
===================================================================
--- branches/rel_2_0/Open-ILS/web/opac/common/js/opac_utils.js	2010-11-12 19:43:27 UTC (rev 18715)
+++ branches/rel_2_0/Open-ILS/web/opac/common/js/opac_utils.js	2010-11-12 19:49:34 UTC (rev 18716)
@@ -827,17 +827,34 @@
 	//setTimeout( 'buildOrgSelector(G.ui.common.org_tree, orgTreeSelector);', 10 );
 	setTimeout( 'buildOrgSelector(G.ui.common.org_tree, orgTreeSelector);', 1 );
 }
-	
+
+var checkOrgHiding_cached = false;
+var checkOrgHiding_cached_context_org;
+var checkOrgHiding_cached_depth;
 function checkOrgHiding() {
-	var context_org = getOrigLocation() || globalOrgTree.id();
-	var depth = fetchOrgSettingDefault( context_org, 'opac.org_unit_hiding.depth');
-	if (isXUL()) {
-		return false; // disable org hiding for staff client
-	}
-	if ( findOrgDepth( context_org ) < depth ) {
-		return false; // disable org hiding if Original Location doesn't make sense with setting depth (avoids disjointed org selectors)
-	}
-	return { 'org' : findOrgUnit(context_org), 'depth' : depth };
+    if (isXUL()) {
+        return false; // disable org hiding for staff client
+    }
+    var context_org = getOrigLocation() || globalOrgTree.id();
+    var depth;
+    if (checkOrgHiding_cached) {
+        if (checkOrgHiding_cached_context_org != context_org) {
+            checkOrgHiding_cached_context_org = context_org;
+            checkOrgHiding_cached_depth = undefined;
+            checkOrgHiding_cached = false;
+        } else {
+            depth = checkOrgHiding_cached_depth;
+        }
+    } else {
+        depth = fetchOrgSettingDefault( context_org, 'opac.org_unit_hiding.depth');
+        checkOrgHiding_cached_depth = depth;
+        checkOrgHiding_cached_context_org = context_org;
+        checkOrgHiding_cached = true;
+    }
+    if ( findOrgDepth( context_org ) < depth ) {
+        return false; // disable org hiding if Original Location doesn't make sense with setting depth (avoids disjointed org selectors)
+    }
+    return { 'org' : findOrgUnit(context_org), 'depth' : depth };
 }
 
 var orgTreeSelector;