From fe3bd563bcf33215df5794d44af46dd5da244c80 Mon Sep 17 00:00:00 2001
From: rjy7 <rjy7@440791e1-c2bf-4c1e-ad7c-caf3f8b0ebe8>
Date: Mon, 21 Feb 2011 18:13:26 +0000
Subject: [PATCH] Merge r7497 from nihvivo-rel-1.2-maint

---
 .../freemarker/FreemarkerHttpServlet.java           |  3 ---
 .../freemarker/body/partials/browse-classgroups.ftl | 13 +++----------
 .../body/partials/menupage/menupage-scripts.ftl     | 13 ++++---------
 3 files changed, 7 insertions(+), 22 deletions(-)

diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java
index 272dcdb1c..b7e733705 100644
--- a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java
+++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java
@@ -407,9 +407,6 @@ public class FreemarkerHttpServlet extends VitroHttpServlet {
 
         // Let the page template know which page it's processing.
         map.put("currentServlet", normalizeServletName(vreq.getServletPath().replaceFirst("/", "")));
-        
-        // Allow template to send domain name to JavaScript (needed for AJAX calls)
-        map.put("requestedPage", vreq.getRequestURL().toString());
 
         // In template: ${now?date}, ${now?datetime}, ${now?time}
         map.put("now", new Date());
diff --git a/webapp/web/templates/freemarker/body/partials/browse-classgroups.ftl b/webapp/web/templates/freemarker/body/partials/browse-classgroups.ftl
index d8ab5ecf1..d04d9e5ae 100644
--- a/webapp/web/templates/freemarker/body/partials/browse-classgroups.ftl
+++ b/webapp/web/templates/freemarker/body/partials/browse-classgroups.ftl
@@ -63,18 +63,11 @@ ${stylesheets.add("/css/browseClassGroups.css")}
             </section> <!-- #browse-classes -->
         </section> <!-- #browse -->
         
-        <#----------------------------------------------------------------------------------
-        requestedPage is currently provided by FreemarkerHttpServlet. Should this be moved
-        to PageController? Maybe we should have Java provide the domain name directly
-        instead of the full URL of the requested page? Chintan was also asking for a
-        template variable with the domain name for an AJAX request with visualizations.
-        ------------------------------------------------------------------------------------>
-        <#assign domainName = requestedPage?substring(0, requestedPage?index_of("/", 7)) />
-
+        <#-- For v1.3: The controller should pass in the dataservice url. -->
         <script type="text/javascript">
             var browseData = {
-                baseUrl: '${domainName + urls.base}',
-                dataServiceUrl: '${domainName + urls.base}/dataservice?getVClassesForVClassGroup=1&classgroupUri=',
+                baseUrl: '${urls.base}',
+                dataServiceUrl: '${urls.base}/dataservice?getVClassesForVClassGroup=1&classgroupUri=',
                 defaultBrowseClassGroupUri: '${firstPopulatedClassGroup.uri!}',
                 defaultBrowseClassGroupCount: '${firstPopulatedClassGroup.individualCount!}'
             };
diff --git a/webapp/web/templates/freemarker/body/partials/menupage/menupage-scripts.ftl b/webapp/web/templates/freemarker/body/partials/menupage/menupage-scripts.ftl
index 56966186a..8e56d6947 100644
--- a/webapp/web/templates/freemarker/body/partials/menupage/menupage-scripts.ftl
+++ b/webapp/web/templates/freemarker/body/partials/menupage/menupage-scripts.ftl
@@ -2,13 +2,7 @@
 
 <#-- Template to setup and call scripts for menupages -->
 
-<#----------------------------------------------------------------------------------
-requestedPage is currently provided by FreemarkerHttpServlet. Should this be moved
-to PageController? Maybe we should have Java provide the domain name directly
-instead of the full URL of the requested page? Chintan was also asking for a
-template variable with the domain name for an AJAX request with visualizations.
------------------------------------------------------------------------------------->
-<#assign domainName = requestedPage?substring(0, requestedPage?index_of("/", 7)) />
+
 
 <#list vClassGroup as vClass>
     <#if (vClass.entityCount > 0)>
@@ -21,10 +15,11 @@ template variable with the domain name for an AJAX request with visualizations.
     </#if>
 </#list>
 
+<#-- For v1.3: The controller should pass in the dataservice url. -->
 <script type="text/javascript">
     var menupageData = {
-        baseUrl: '${domainName + urls.base}',
-        dataServiceUrl: '${domainName + urls.base}/dataservice?getLuceneIndividualsByVClass=1&vclassId=',
+        baseUrl: '${urls.base}',
+        dataServiceUrl: '${urls.base}/dataservice?getLuceneIndividualsByVClass=1&vclassId=',
         defaultBrowseVClassUri: '${firstNonEmptyVClass}'
     };
 </script>