Author: jleroux
Date: Thu Jul 12 14:54:14 2012
New Revision: 1360711

URL: http://svn.apache.org/viewvc?rev=1360711&view=rev
Log:
A patch from Olivier Heintz "Help links not working after lookup.js 
refactoring, needs to change lookup-popu2 to lookup-popu1" 
https://issues.apache.org/jira/browse/OFBIZ-4964

Help button is not working, currently it's a lookup-popup2 which is called and 
this method has not the same number of parameters than the call.
In lookup.js, I have see that lookup-popup1 has the correct number of 
parameters, so in the join patch I have change the lookup-popup2 to 
lookup-popup1

Modified:
    ofbiz/trunk/themes/bizznesstime/includes/header.ftl
    ofbiz/trunk/themes/bluelight/includes/header.ftl
    ofbiz/trunk/themes/droppingcrumbs/includes/header.ftl
    ofbiz/trunk/themes/flatgrey/includes/header.ftl
    ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl

Modified: ofbiz/trunk/themes/bizznesstime/includes/header.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/themes/bizznesstime/includes/header.ftl?rev=1360711&r1=1360710&r2=1360711&view=diff
==============================================================================
--- ofbiz/trunk/themes/bizznesstime/includes/header.ftl (original)
+++ ofbiz/trunk/themes/bizznesstime/includes/header.ftl Thu Jul 12 14:54:14 2012
@@ -148,7 +148,7 @@ under the License.
             <#--if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists 
&& helpTopic?exists-->
             <#if parameters.componentName?exists && 
requestAttributes._CURRENT_VIEW_?exists && helpTopic?exists>
               <#include "component://common/webcommon/includes/helplink.ftl" />
-              <span><a 
href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);">${uiLabelMap.CommonHelp}</a></span>
+              <span><a 
href="javascript:lookup_popup1('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);">${uiLabelMap.CommonHelp}</a></span>
            </#if>
            <#if layoutSettings.middleTopMessage1?has_content && 
layoutSettings.middleTopMessage1 != " ">
              <span id="last-system-msg">

Modified: ofbiz/trunk/themes/bluelight/includes/header.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/themes/bluelight/includes/header.ftl?rev=1360711&r1=1360710&r2=1360711&view=diff
==============================================================================
--- ofbiz/trunk/themes/bluelight/includes/header.ftl (original)
+++ ofbiz/trunk/themes/bluelight/includes/header.ftl Thu Jul 12 14:54:14 2012
@@ -179,7 +179,7 @@ under the License.
             <#--if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists 
&& helpTopic?exists-->
             <#if parameters.componentName?exists && 
requestAttributes._CURRENT_VIEW_?exists && helpTopic?exists>
               <#include "component://common/webcommon/includes/helplink.ftl" />
-              <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
+              <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup1('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
             </#if>
             <#if userLogin?exists>
               <#if (userPreferences.COMPACT_HEADER)?default("N") == "Y">

Modified: ofbiz/trunk/themes/droppingcrumbs/includes/header.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/themes/droppingcrumbs/includes/header.ftl?rev=1360711&r1=1360710&r2=1360711&view=diff
==============================================================================
--- ofbiz/trunk/themes/droppingcrumbs/includes/header.ftl (original)
+++ ofbiz/trunk/themes/droppingcrumbs/includes/header.ftl Thu Jul 12 14:54:14 
2012
@@ -172,7 +172,7 @@ under the License.
             <#--if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists 
&& helpTopic?exists-->
             <#if parameters.componentName?exists && 
requestAttributes._CURRENT_VIEW_?exists && helpTopic?exists>
               <#include "component://common/webcommon/includes/helplink.ftl" />
-              <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
+              <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup1('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
             </#if>
             <#if userLogin?exists>
               <#if (userPreferences.COMPACT_HEADER)?default("N") == "Y">

Modified: ofbiz/trunk/themes/flatgrey/includes/header.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/themes/flatgrey/includes/header.ftl?rev=1360711&r1=1360710&r2=1360711&view=diff
==============================================================================
--- ofbiz/trunk/themes/flatgrey/includes/header.ftl (original)
+++ ofbiz/trunk/themes/flatgrey/includes/header.ftl Thu Jul 12 14:54:14 2012
@@ -150,7 +150,7 @@ under the License.
           <#---if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists 
&& helpTopic?exists-->
           <#if parameters.componentName?exists && 
requestAttributes._CURRENT_VIEW_?exists && helpTopic?exists>
             <#include "component://common/webcommon/includes/helplink.ftl" />
-            <li><a <#if pageAvail?has_content>class="alert"</#if> 
href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);">${uiLabelMap.CommonHelp}</a></li>
+            <li><a <#if pageAvail?has_content>class="alert"</#if> 
href="javascript:lookup_popup1('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);">${uiLabelMap.CommonHelp}</a></li>
           </#if>
           </ul>
       </li>

Modified: ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl?rev=1360711&r1=1360710&r2=1360711&view=diff
==============================================================================
--- ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl (original)
+++ ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl Thu Jul 12 14:54:14 
2012
@@ -75,7 +75,7 @@ under the License.
       <#--if webSiteId?exists && requestAttributes._CURRENT_VIEW_?exists && 
helpTopic?exists-->
       <#if parameters.componentName?exists && 
requestAttributes._CURRENT_VIEW_?exists && helpTopic?exists>
         <#include "component://common/webcommon/includes/helplink.ftl" />
-        <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup2('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
+        <li><a class="help-link <#if pageAvail?has_content> alert</#if>" 
href="javascript:lookup_popup1('showHelp?helpTopic=${helpTopic}&amp;portalPageId=${parameters.portalPageId?if_exists}','help'
 ,500,500);" title="${uiLabelMap.CommonHelp}"></a></li>
       </#if>
       <li><a 
href="<@ofbizUrl>logout</@ofbizUrl>">${uiLabelMap.CommonLogout}</a></li>
       <li><a 
href="<@ofbizUrl>ListVisualThemes</@ofbizUrl>">${uiLabelMap.CommonVisualThemes}</a></li>


Reply via email to