[jira] Created: (OFBIZ-2272) Secure URLs exceptions

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

[jira] Created: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
Secure URLs exceptions
----------------------

                 Key: OFBIZ-2272
                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
             Project: OFBiz
          Issue Type: Improvement
          Components: ALL COMPONENTS
    Affects Versions: Release Branch 9.3, SVN trunk
            Reporter: Jacques Le Roux
             Fix For: Release Branch 9.3, SVN trunk


Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience(?) but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
{code}
Index: applications/content/widget/cms/CMSScreens.xml
===================================================================
--- applications/content/widget/cms/CMSScreens.xml (revision 761874)
+++ applications/content/widget/cms/CMSScreens.xml (working copy)
@@ -171,7 +171,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                         <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
                                     </widgets>
@@ -217,7 +219,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                     </widgets>
                                 </section>
Index: applications/product/widget/facility/FacilityScreens.xml
===================================================================
--- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
+++ applications/product/widget/facility/FacilityScreens.xml (working copy)
@@ -431,8 +431,12 @@
                                     <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                     </link>-->
-                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                                     <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                         <parameter param-name="action" value="Y"/>
Index: applications/product/widget/facility/ReportScreens.xml
===================================================================
--- applications/product/widget/facility/ReportScreens.xml (revision 761874)
+++ applications/product/widget/facility/ReportScreens.xml (working copy)
@@ -54,8 +54,12 @@
                             <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
                         </container>
                         <container>
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
===================================================================
--- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
+++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
@@ -59,8 +59,12 @@
                             <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                 <parameter param-name="facilityId" from-field="facilityId"/>
                             </link>-->
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
 
                         <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
Index: specialpurpose/mypage/widget/MyPageMenus.xml
===================================================================
--- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
+++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
@@ -126,7 +126,7 @@
         </menu-item>
         <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
             <condition><not><if-empty field="userLogin"/></not></condition>
-            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
+            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
         </menu-item>
         <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
             <condition>
{code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux updated OFBIZ-2272:
-----------------------------------

    Description:
Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
{code}
Index: applications/content/widget/cms/CMSScreens.xml
===================================================================
--- applications/content/widget/cms/CMSScreens.xml (revision 761874)
+++ applications/content/widget/cms/CMSScreens.xml (working copy)
@@ -171,7 +171,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                         <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
                                     </widgets>
@@ -217,7 +219,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                     </widgets>
                                 </section>
Index: applications/product/widget/facility/FacilityScreens.xml
===================================================================
--- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
+++ applications/product/widget/facility/FacilityScreens.xml (working copy)
@@ -431,8 +431,12 @@
                                     <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                     </link>-->
-                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                                     <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                         <parameter param-name="action" value="Y"/>
Index: applications/product/widget/facility/ReportScreens.xml
===================================================================
--- applications/product/widget/facility/ReportScreens.xml (revision 761874)
+++ applications/product/widget/facility/ReportScreens.xml (working copy)
@@ -54,8 +54,12 @@
                             <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
                         </container>
                         <container>
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
===================================================================
--- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
+++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
@@ -59,8 +59,12 @@
                             <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                 <parameter param-name="facilityId" from-field="facilityId"/>
                             </link>-->
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
 
                         <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
Index: specialpurpose/mypage/widget/MyPageMenus.xml
===================================================================
--- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
+++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
@@ -126,7 +126,7 @@
         </menu-item>
         <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
             <condition><not><if-empty field="userLogin"/></not></condition>
-            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
+            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
         </menu-item>
         <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
             <condition>
{code}

  was:
Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience(?) but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
{code}
Index: applications/content/widget/cms/CMSScreens.xml
===================================================================
--- applications/content/widget/cms/CMSScreens.xml (revision 761874)
+++ applications/content/widget/cms/CMSScreens.xml (working copy)
@@ -171,7 +171,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                         <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
                                     </widgets>
@@ -217,7 +219,9 @@
                             <container style="screenlet-body">
                                 <section>
                                     <widgets>    
-                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
+                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
+                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
+                                    </link>
                                         <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
                                     </widgets>
                                 </section>
Index: applications/product/widget/facility/FacilityScreens.xml
===================================================================
--- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
+++ applications/product/widget/facility/FacilityScreens.xml (working copy)
@@ -431,8 +431,12 @@
                                     <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                     </link>-->
-                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                                     <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
                                         <parameter param-name="facilityId" from-field="facilityId"/>
                                         <parameter param-name="action" value="Y"/>
Index: applications/product/widget/facility/ReportScreens.xml
===================================================================
--- applications/product/widget/facility/ReportScreens.xml (revision 761874)
+++ applications/product/widget/facility/ReportScreens.xml (working copy)
@@ -54,8 +54,12 @@
                             <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
                         </container>
                         <container>
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
===================================================================
--- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
+++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
@@ -59,8 +59,12 @@
                             <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
                                 <parameter param-name="facilityId" from-field="facilityId"/>
                             </link>-->
-                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
-                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
+                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
+                                    </link>
+                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
+                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
+                                    </link>
                         </container>
 
                         <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
Index: specialpurpose/mypage/widget/MyPageMenus.xml
===================================================================
--- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
+++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
@@ -126,7 +126,7 @@
         </menu-item>
         <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
             <condition><not><if-empty field="userLogin"/></not></condition>
-            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
+            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
         </menu-item>
         <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
             <condition>
{code}


> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.3, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.3, SVN trunk
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux updated OFBIZ-2272:
-----------------------------------

    Attachment: URLs secure exceptions.patch

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.3, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.3, SVN trunk
>
>         Attachments: URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12695855#action_12695855 ]

Jacques Le Roux commented on OFBIZ-2272:
----------------------------------------

Here is the David's comment about the/CMSScreens.xml case

{quote}
From: "David E Jones" <[hidden email]>

 This is kind of a bad pattern since it puts presentation-specific code  in the data preparation, ie the CMSContentFindQueryString
 variable is  a presentation-specific parameter string that is prepared in a the  data preparation code. While not a terrible thing
 to do, it does  reduce reusability and complicates things like this.

 The best solution is probably to find the code that maintains the  "CMSContentFindQueryString" field, and then put the necessary
 parameters in this link. It may be that the field is use to keep  context for a chain of pages, which is a bit of a mess and while
 I've  seen that sort of thing done in different places it's a bit error- prone and there may be better approaches.

 I guess in other words, find out of the parameters are necessary and  include what you can and then the rest... well we can either
 tear it  apart and rewrite the stuff, or we might get lucky and whoever wrote  will hop in... ;)

 -David
{quote}

The [thread is here|http://markmail.org/message/qaz5peog4ci45qmn]

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.3, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.3, SVN trunk
>
>         Attachments: URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12695856#action_12695856 ]

Jacques Le Roux commented on OFBIZ-2272:
----------------------------------------

I found this one also
{code}
<hyperlink target="${my}TimeSheetStatusToComplete" description="${statusId.equals(&quot;TIMESHEET_IN_PROCESS&quot;)?complete:&quot;&quot;}" also-hidden="false">
{code}

But I think it's ok

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.3, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.3, SVN trunk
>
>         Attachments: URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux updated OFBIZ-2272:
-----------------------------------

    Attachment: param-name formules screens menus.txt

For those interested here are the regexp I used for this secure URLs campaign.

I think we should do something for the form also (2 last regexps)

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.3, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.3, SVN trunk
>
>         Attachments: param-name formules screens menus.txt, URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux updated OFBIZ-2272:
-----------------------------------

    Issue Type: Sub-task  (was: Improvement)
        Parent: OFBIZ-1525

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.04, SVN trunk
>
>         Attachments: param-name formules screens menus.txt, URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Issue Comment Edited: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12695864#action_12695864 ]

Jacques Le Roux edited comment on OFBIZ-2272 at 12/5/09 1:43 PM:
-----------------------------------------------------------------

For those interested here are the regexp I used for this secure URLs campaign.

I think we should do something for the form also (2 last regexps)

Forgot to say that I had opened OFBIZ-2449 for that

Edit ********** added OFBIZ-2449 **************



      was (Author: jacques.le.roux):
    For those interested here are the regexp I used for this secure URLs campaign.

I think we should do something for the form also (2 last regexps)
 

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Jacques Le Roux
>             Fix For: Release Branch 9.04, SVN trunk
>
>         Attachments: param-name formules screens menus.txt, URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Closed: (OFBIZ-2272) Secure URLs exceptions

Nicolas Malin (Jira)
In reply to this post by Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux closed OFBIZ-2272.
----------------------------------

      Assignee: Jacques Le Roux
    Resolution: Not A Problem

I completly forgot this one. There are no problems here, it's only a bunch of find and view, no actions: no worries

> Secure URLs exceptions
> ----------------------
>
>                 Key: OFBIZ-2272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2272
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL COMPONENTS
>    Affects Versions: Release Branch 09.04, SVN trunk
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>             Fix For: Release Branch 09.04, SVN trunk
>
>         Attachments: param-name formules screens menus.txt, URLs secure exceptions.patch
>
>
> Here are theexceptions I crossed during the recent secure URLs action. I put it here for convenience but I attached also the patch. Beware this patch is only for demontration (to show problems, not solutions) and should *not* be applied of course.
> {code}
> Index: applications/content/widget/cms/CMSScreens.xml
> ===================================================================
> --- applications/content/widget/cms/CMSScreens.xml (revision 761874)
> +++ applications/content/widget/cms/CMSScreens.xml (working copy)
> @@ -171,7 +171,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                          <content content-id="${contentId}" enable-edit-name="notfound"  edit-request="EditAddSubContent?MASTER_contentId=${MASTER_contentId}&amp;MASTER_caContentIdTo=${MASTER_caContentIdTo}&amp;MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&amp;MASTER_caFromDate=${MASTER_caFromDate}&amp;MASTER_drDataResourceId=${MASTER_drDataResourceId}&amp;caContentIdTo=${caContentIdTo}"/>
>                                      </widgets>
> @@ -217,7 +219,9 @@
>                              <container style="screenlet-body">
>                                  <section>
>                                      <widgets>    
> -                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}" style="tabletext"/>
> +                                        <link  text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext">
> +                                        <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&amp;${CMSContentFindQueryString}"/>
> +                                    </link>
>                                          <include-form name="EditAddSubContentStuff" location="component://content/webapp/content/cms/CMSForms.xml" />
>                                      </widgets>
>                                  </section>
> Index: applications/product/widget/facility/FacilityScreens.xml
> ===================================================================
> --- applications/product/widget/facility/FacilityScreens.xml (revision 761874)
> +++ applications/product/widget/facility/FacilityScreens.xml (working copy)
> @@ -431,8 +431,12 @@
>                                      <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                      </link>-->
> -                                    <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                                    <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                                    <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                                    <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                                      <link target="InventoryItemTotals" text="${uiLabelMap.ProductInventoryItemTotals}" style="buttontext">
>                                          <parameter param-name="facilityId" from-field="facilityId"/>
>                                          <parameter param-name="action" value="Y"/>
> Index: applications/product/widget/facility/ReportScreens.xml
> ===================================================================
> --- applications/product/widget/facility/ReportScreens.xml (revision 761874)
> +++ applications/product/widget/facility/ReportScreens.xml (working copy)
> @@ -54,8 +54,12 @@
>                              <label style="h1" text="${uiLabelMap.ProductInventoryByProduct} ${uiLabelMap.CommonFor} ${facility.facilityName}"/>
>                          </container>
>                          <container>
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>                          <container style="screenlet">
>                              <container style="screenlet-title-bar">
> Index: specialpurpose/assetmaint/widget/FacilityScreens.xml
> ===================================================================
> --- specialpurpose/assetmaint/widget/FacilityScreens.xml (revision 761874)
> +++ specialpurpose/assetmaint/widget/FacilityScreens.xml (working copy)
> @@ -59,8 +59,12 @@
>                              <!--<link target="ViewFacilityInventoryByProductSimple" text="${uiLabelMap.CommonPrint}" style="buttontext">
>                                  <parameter param-name="facilityId" from-field="facilityId"/>
>                              </link>-->
> -                            <link target="ViewFacilityInventoryByProductReport?${searchParameterString}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
> -                            <link target="ViewFacilityInventoryByProductExport?${searchParameterString}" text="${uiLabelMap.CommonExport}" style="buttontext"/>
> +                            <link target="ViewFacilityInventoryByProductReport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonPrint}"/>
> +                                    </link>
> +                            <link target="ViewFacilityInventoryByProductExport" style="buttontext">
> +                                        <parameter param-name="${searchParameterString}" text" value=""${uiLabelMap.CommonExport}"/>
> +                                    </link>
>                          </container>
>  
>                          <include-form name="FindFacilityInventoryByProduct" location="component://product/webapp/facility/facility/FacilityForms.xml"/>
> Index: specialpurpose/mypage/widget/MyPageMenus.xml
> ===================================================================
> --- specialpurpose/mypage/widget/MyPageMenus.xml (revision 761874)
> +++ specialpurpose/mypage/widget/MyPageMenus.xml (working copy)
> @@ -126,7 +126,7 @@
>          </menu-item>
>          <menu-item name="help" title="${uiLabelMap.CommonHelp}" align-style="opposed">
>              <condition><not><if-empty field="userLogin"/></not></condition>
> -            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page?decorator=printable" url-mode="plain" target-window="new"/>
> +            <link target="http://docs.ofbiz.org/display/OFBENDUSER/My+Page" url-mode="plain" target-window="new"/>
>          </menu-item>
>          <menu-item name="preferences" title="${uiLabelMap.CommonPreferences}" align-style="opposed">
>              <condition>
> {code}

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.