svn commit: r1806289 - /ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1806289 - /ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl

Deepak Dixit-5
Author: deepak
Date: Sat Aug 26 12:06:57 2017
New Revision: 1806289

URL: http://svn.apache.org/viewvc?rev=1806289&view=rev
Log:
Fixed: HTML encoded path of image is showing on product detail page, applied slightly modified patch from jira issue.(OFBIZ-9654)
Thanks Devanshu Vyas for your contribution.
This was mistakenly committed at r#1796379, I don;t know how plugins was committed as I trigger svn ci only for ofbiz-framework.

Modified:
    ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl

Modified: ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl?rev=1806289&r1=1806288&r2=1806289&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl (original)
+++ ofbiz/ofbiz-plugins/trunk/ecommerce/template/catalog/ProductDetail.ftl Sat Aug 26 12:06:57 2017
@@ -367,9 +367,8 @@ $(function(){
           <#assign productLargeImageUrl = firstLargeImage />
         </#if>
         <#if productLargeImageUrl?string?has_content>
-        ${productLargeImageUrl!}===
           <a href="javascript:popupDetail();">
-            <img id="detailImage" src="<@ofbizContentUrl>${contentPathPrefix!}${productLargeImageUrl?html}</@ofbizContentUrl>"
+            <img id="detailImage" src="<@ofbizContentUrl>${contentPathPrefix!}${productLargeImageUrl!}</@ofbizContentUrl>"
                 name="mainImage" vspace="5" hspace="5" class="cssImgXLarge" alt=""/>
           </a>
           <input type="hidden" id="originalImage" name="originalImage"
@@ -397,14 +396,13 @@ $(function(){
       <#-- Product image/name/price -->
       <div id="detailImageBox">
         <#assign productLargeImageUrl = productContentWrapper.get("LARGE_IMAGE_URL", "url")! />
-          <label>${productLargeImageUrl}</label>
         <#-- remove the next two lines to always display the virtual image first (virtual images must exist) -->
         <#if firstLargeImage?has_content>
           <#assign productLargeImageUrl = firstLargeImage />
         </#if>
         <#if productLargeImageUrl?string?has_content>
           <a href="javascript:popupDetail();">
-            <img id="detailImage" src="<@ofbizContentUrl>${contentPathPrefix!}${productLargeImageUrl?html!}</@ofbizContentUrl>"
+            <img id="detailImage" src="<@ofbizContentUrl>${contentPathPrefix!}${productLargeImageUrl!}</@ofbizContentUrl>"
                 name="mainImage" vspace="5" hspace="5" class="cssImgXLarge" alt=""/>
           </a>
           <input type="hidden" id="originalImage" name="originalImage"