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}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> </widgets> @@ -217,7 +219,9 @@ <container style="screenlet-body"> <section> <widgets> - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
[ 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}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> </widgets> @@ -217,7 +219,9 @@ <container style="screenlet-body"> <section> <widgets> - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> </widgets> @@ -217,7 +219,9 @@ <container style="screenlet-body"> <section> <widgets> - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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("TIMESHEET_IN_PROCESS")?complete:""}" 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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
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}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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}&MASTER_caContentIdTo=${MASTER_caContentIdTo}&MASTER_caContentAssocTypeId=${MASTER_caContentAssocTypeId}&MASTER_caFromDate=${MASTER_caFromDate}&MASTER_drDataResourceId=${MASTER_drDataResourceId}&caContentIdTo=${caContentIdTo}"/> > </widgets> > @@ -217,7 +219,9 @@ > <container style="screenlet-body"> > <section> > <widgets> > - <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind?VIEW_INDEX=${CMSContentFindViewIndex}&${CMSContentFindQueryString}" style="tabletext"/> > + <link text="${uiLabelMap.ContentGoToFind}" target="CMSContentFind" style="tabletext"> > + <parameter param-name="VIEW_INDEX" value="${CMSContentFindViewIndex}&${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. |
Free forum by Nabble | Edit this page |