Arthur Bogaart pushed to branch bugfix/ESSENTIALS-1017 at cms-community / 
hippo-essentials


Commits:
93502748 by Arthur Bogaart at 2016-08-12T13:48:30+02:00
ESSENTIALS-1017 Indent with four spaces

- - - - -
e41622ba by Arthur Bogaart at 2016-08-12T13:49:25+02:00
ESSENTIALS-1017 Indent with four spaces

- - - - -
6e3b48d3 by Arthur Bogaart at 2016-08-12T13:57:05+02:00
ESSENTIALS-1017 Align googlemaps JSP with Freemarker template

- - - - -


2 changed files:

- plugins/google-maps/src/main/resources/freemarker/essentials-googlemaps.ftl
- plugins/google-maps/src/main/resources/jsp/essentials-googlemaps.jsp


Changes:

=====================================
plugins/google-maps/src/main/resources/freemarker/essentials-googlemaps.ftl
=====================================
--- 
a/plugins/google-maps/src/main/resources/freemarker/essentials-googlemaps.ftl
+++ 
b/plugins/google-maps/src/main/resources/freemarker/essentials-googlemaps.ftl
@@ -6,13 +6,13 @@
 <div id="map-canvas-${componentId}" style="width: ${cparam.width}px; height: 
${cparam.height}px;"></div>
 
 <@hst.headContribution category="htmlBodyEnd">
-  <script type="text/javascript">
-      if (!window.HEGM) {
-          window.HEGM = [];
-      }
-      window.HEGM.push(function() {
-          initGoogleMap('map-canvas-${componentId}', '${cparam.address?html}', 
${cparam.longitude}, ${cparam.latitude}, ${cparam.zoomFactor}, 
'${cparam.mapType}');
-      });
+    <script type="text/javascript">
+        if (!window.HEGM) {
+            window.HEGM = [];
+        }
+        window.HEGM.push(function() {
+            initGoogleMap('map-canvas-${componentId}', 
'${cparam.address?html}', ${cparam.longitude}, ${cparam.latitude}, 
${cparam.zoomFactor}, '${cparam.mapType}');
+        });
     </script>
 </@hst.headContribution>
 


=====================================
plugins/google-maps/src/main/resources/jsp/essentials-googlemaps.jsp
=====================================
--- a/plugins/google-maps/src/main/resources/jsp/essentials-googlemaps.jsp
+++ b/plugins/google-maps/src/main/resources/jsp/essentials-googlemaps.jsp
@@ -1,34 +1,42 @@
 <%@ include file="/WEB-INF/jsp/include/imports.jsp" %>
 
+<%--@elvariable id="componentId" type="java.lang.String"--%>
 <%--@elvariable id="cparam" 
type="org.onehippo.cms7.essentials.components.info.EssentialsGoogleMapsComponentInfo"--%>
-<c:choose>
-  <c:when test="${not empty requestScope.cparam.apiKey}">
-    <c:set 
var="mapsUrl">https://maps.googleapis.com/maps/api/js?key=${fn:escapeXml(requestScope.cparam.apiKey)}</c:set>
-  </c:when>
-  <c:otherwise>
-    <c:set var="mapsUrl">https://maps.googleapis.com/maps/api/js</c:set>
-  </c:otherwise>
-</c:choose>
 
-<hst:headContribution keyHint="api" category="htmlHead">
-  <script type="text/javascript" src="${mapsUrl}"></script>
+<div id="map-canvas-${componentId}" style="width: 
${requestScope.cparam.width}px; height: ${requestScope.cparam.height}px;"></div>
+
+<hst:headContribution category="htmlBodyEnd">
+    <script type="text/javascript">
+        if (!window.HEGM) {
+            window.HEGM = [];
+        }
+        window.HEGM.push(function() {
+            initGoogleMap('map-canvas-${componentId}', 
'${fn:escapeXml(requestScope.cparam.address)}', 
${requestScope.cparam.longitude}, ${requestScope.cparam.latitude}, 
${requestScope.cparam.zoomFactor}, '${requestScope.cparam.mapType}');
+        });
+    </script>
 </hst:headContribution>
 
-<hst:headContribution keyHint="maps" category="htmlHead">
-  <hst:webfile path="/js/essentials-google-maps.js" var="customMapsJSUrl" />
-  <script type="text/javascript" src="${customMapsJSUrl}"></script>
+<hst:headContribution keyHint="essentials-google-maps" category="htmlBodyEnd">
+    <hst:webfile path="/js/essentials-google-maps.js" 
var="essentialsGoogleMapsJs" />
+    <script type="text/javascript" src="${essentialsGoogleMapsJs}"></script>
 </hst:headContribution>
 
-<hst:headContribution keyHint="initializeGoogleMaps" category="htmlBodyEnd">
-  <script type="text/javascript">
-    initializeGoogleMaps("${fn:escapeXml(requestScope.cparam.address)}", 
${requestScope.cparam.longitude}, ${requestScope.cparam.latitude}, 
${requestScope.cparam.zoomFactor}, "${requestScope.cparam.mapType}");
-  </script>
+<hst:headContribution keyHint="google-maps-api" category="htmlBodyEnd">
+    <c:choose>
+        <c:when test="${not empty requestScope.cparam.apiKey}">
+            <c:set 
var="mapsUrl">https://maps.googleapis.com/maps/api/js?key=${fn:escapeXml(requestScope.cparam.apiKey)}&callback=initGoogleMaps</c:set>
+        </c:when>
+        <c:otherwise>
+            <c:set 
var="mapsUrl">https://maps.googleapis.com/maps/api/js?callback=initGoogleMaps</c:set>
+        </c:otherwise>
+    </c:choose>
+    <script type="text/javascript" src="${mapsUrl}" async="async" 
defer="defer"></script>
 </hst:headContribution>
 
-<c:if test="${requestScope.cmsrequest}">
-  <script type="text/javascript">
-    initializeGoogleMaps("${fn:escapeXml(requestScope.cparam.address)}", 
${requestScope.cparam.longitude}, ${requestScope.cparam.latitude}, 
${requestScope.cparam.zoomFactor}, "${requestScope.cparam.mapType}");
-  </script>
+<c:if test="${editMode}">
+    <script type="text/javascript">
+        if (window.initGoogleMap) {
+            initGoogleMap('map-canvas-${componentId}', 
'${fn:escapeXml(requestScope.cparam.address)}', 
${requestScope.cparam.longitude}, ${requestScope.cparam.latitude}, 
${requestScope.cparam.zoomFactor}, '${requestScope.cparam.mapType}');
+        }
+    </script>
 </c:if>
-
-<div id="map-canvas" style="width: ${requestScope.cparam.width}px; height: 
${requestScope.cparam.height}px;"></div>
\ No newline at end of file



View it on GitLab: 
https://code.onehippo.org/cms-community/hippo-essentials/compare/9aaf9e9d639b15171b6b3935ed681eedd961e551...6e3b48d355599989490b1fcc2fc9d7dba4f5bb8e
_______________________________________________
Hippocms-svn mailing list
Hippocms-svn@lists.onehippo.org
https://lists.onehippo.org/mailman/listinfo/hippocms-svn

Reply via email to