svn commit: r760815 - in /ofbiz/trunk: applications/accounting/widget/ applications/content/widget/ applications/party/widget/partymgr/ specialpurpose/ecommerce/widget/ specialpurpose/mypage/widget/

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

svn commit: r760815 - in /ofbiz/trunk: applications/accounting/widget/ applications/content/widget/ applications/party/widget/partymgr/ specialpurpose/ecommerce/widget/ specialpurpose/mypage/widget/

jleroux@apache.org
Author: jleroux
Date: Wed Apr  1 08:41:00 2009
New Revision: 760815

URL: http://svn.apache.org/viewvc?rev=760815&view=rev
Log:
Securing URLs, link element in screens and menus : 13th set + tabs to 4 spaces

Modified:
    ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml
    ofbiz/trunk/applications/accounting/widget/GlScreens.xml
    ofbiz/trunk/applications/content/widget/CommonScreens.xml
    ofbiz/trunk/applications/party/widget/partymgr/PartyMenus.xml
    ofbiz/trunk/specialpurpose/ecommerce/widget/ForumScreens.xml
    ofbiz/trunk/specialpurpose/mypage/widget/MyPageMenus.xml

Modified: ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml (original)
+++ ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml Wed Apr  1 08:41:00 2009
@@ -266,7 +266,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementPromoAppl?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.AccountingNewAgreementPromoAppl}" style="buttontext"/>
+                            <link target="EditAgreementPromoAppl" text="${uiLabelMap.AccountingNewAgreementPromoAppl}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -305,7 +308,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementPromoAppl?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.AccountingNewAgreementPromoAppl}" style="buttontext"/>
+                            <link target="EditAgreementPromoAppl" text="${uiLabelMap.AccountingNewAgreementPromoAppl}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -345,7 +351,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemTerm?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.AccountingNewAgreementItemTerm}" style="buttontext"/>
+                            <link target="EditAgreementItemTerm" text="${uiLabelMap.AccountingNewAgreementItemTerm}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -385,7 +394,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemTerm?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.AccountingNewAgreementItemTerm}" style="buttontext"/>
+                            <link target="EditAgreementItemTerm" text="${uiLabelMap.AccountingNewAgreementItemTerm}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -425,8 +437,14 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemProduct?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.ProductNewProduct}" style="buttontext"/>
-                            <link target="ListAgreementItemProductsReport?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
+                            <link target="EditAgreementItemProduct" text="${uiLabelMap.ProductNewProduct}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
+                            <link target="ListAgreementItemProductsReport" text="${uiLabelMap.CommonPrint}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -532,7 +550,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemProduct?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.ProductNewProduct}" style="buttontext"/>
+                            <link target="EditAgreementItemProduct" text="${uiLabelMap.ProductNewProduct}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -572,8 +593,14 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemSupplierProduct?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.ProductNewProduct}" style="buttontext"/>
-                            <link target="ListAgreementItemSupplierProductsReport?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonPrint}" style="buttontext"/>
+                            <link target="EditAgreementItemSupplierProduct" text="${uiLabelMap.ProductNewProduct}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
+                            <link target="ListAgreementItemSupplierProductsReport" text="${uiLabelMap.CommonPrint}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -686,7 +713,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemSupplierProduct?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.ProductNewProduct}" style="buttontext"/>
+                            <link target="EditAgreementItemSupplierProduct" text="${uiLabelMap.ProductNewProduct}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -737,7 +767,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemParty?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonNew}" style="buttontext"/>
+                            <link target="EditAgreementItemParty" text="${uiLabelMap.CommonNew}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -777,7 +810,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementItemParty?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonNew}" style="buttontext"/>
+                            <link target="EditAgreementItemParty" text="${uiLabelMap.CommonNew}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -817,7 +853,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementGeographicalApplic?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonNew}" style="buttontext"/>
+                            <link target="EditAgreementGeographicalApplic" text="${uiLabelMap.CommonNew}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">
@@ -857,7 +896,10 @@
                 <decorator-screen name="CommonAgreementDecorator" location="${parameters.agreementDecoratorLocation}">
                     <decorator-section name="body">
                         <container>
-                            <link target="EditAgreementGeographicalApplic?agreementId=${agreement.agreementId}&amp;agreementItemSeqId=${agreementItem.agreementItemSeqId}" text="${uiLabelMap.CommonNew}" style="buttontext"/>
+                            <link target="EditAgreementGeographicalApplic" text="${uiLabelMap.CommonNew}" style="buttontext">
+                            <parameter param-name="agreementId" from-field="agreement.agreementId"/>
+                            <parameter param-name="agreementItemSeqId" from-field="agreementItem.agreementItemSeqId"/>
+                        </link>
                         </container>
                         <container style="screenlet">
                             <container style="screenlet-title-bar">

Modified: ofbiz/trunk/applications/accounting/widget/GlScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/widget/GlScreens.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/widget/GlScreens.xml (original)
+++ ofbiz/trunk/applications/accounting/widget/GlScreens.xml Wed Apr  1 08:41:00 2009
@@ -326,7 +326,11 @@
                                                     <parameter param-name="acctgTransId" from-field="acctgTransId"/>
                                                     <parameter param-name="organizationPartyId" from-field="organizationPartyId"/>
                                                 </link>
-                                                <link text="${uiLabelMap.AccountingVerifyTransaction}" style="buttontext" target="postAcctgTrans?acctgTransId=${acctgTransId}&amp;organizationPartyId=${organizationPartyId}&amp;verifyOnly=Y"/>
+                                                <link text="${uiLabelMap.AccountingVerifyTransaction}" style="buttontext" target="postAcctgTrans">
+                                                    <parameter param-name="acctgTransId"/>
+                                                    <parameter param-name="organizationPartyId"/>
+                                                    <parameter param-name="verifyOnly" value="Y"/>
+                                                </link>
                                                 <link text="${uiLabelMap.AccountingPostTransaction}" style="buttontext" target="postAcctgTrans">
                                                     <parameter param-name="acctgTransId" from-field="acctgTransId"/>
                                                     <parameter param-name="organizationPartyId" from-field="organizationPartyId"/>

Modified: ofbiz/trunk/applications/content/widget/CommonScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/widget/CommonScreens.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/applications/content/widget/CommonScreens.xml (original)
+++ ofbiz/trunk/applications/content/widget/CommonScreens.xml Wed Apr  1 08:41:00 2009
@@ -24,7 +24,7 @@
     <screen name="main-decorator">
         <section>
             <actions>
-                <!-- base/top/specific map first, then more common map added for shared labels -->
+                <!-- base/top/specific map first, then more common map added for shared labels -->
                 <property-map resource="ContentUiLabels" map-name="uiLabelMap" global="true"/>
                 <property-map resource="CommonUiLabels" map-name="uiLabelMap" global="true"/>
                 <property-map resource="WorkEffortUiLabels" map-name="uiLabelMap" global="true"/>
@@ -33,10 +33,10 @@
                 <set field="layoutSettings.companyName" from-field="uiLabelMap.ContentCompanyName" global="true"/>
                 <set field="layoutSettings.companySubtitle" from-field="uiLabelMap.ContentCompanySubtitle" global="true"/>
                 <set field="layoutSettings.styleSheets[]" value="/content/images/contentForum.css" global="true"/>
-                <set field="layoutSettings.javaScripts[]" value="/images/selectall.js" global="true"/>                
+                <set field="layoutSettings.javaScripts[]" value="/images/selectall.js" global="true"/>
                 <set field="activeApp" value="contentmgr" global="true"/>
                 <set field="applicationMenuName" value="ContentAppBar" global="true"/>
-                <set field="applicationMenuLocation" value="component://content/widget/content/ContentMenus.xml" global="true"/>
+                <set field="applicationMenuLocation" value="component://content/widget/content/ContentMenus.xml" global="true"/>
                 <set field="applicationTitle" value="${uiLabelMap.ContentContentManagerApplication}"/>
             </actions>
             <widgets>
@@ -111,7 +111,7 @@
             </widgets>
         </section>
     </screen>
-    
+
     <screen name="contentDecorator">
         <section>
             <actions>
@@ -125,7 +125,7 @@
                                 <if-has-permission permission="CONTENTMGR" action="UPDATE"/>
                             </condition>
                             <widgets>
-                                <include-menu name="contentTopMenu" location="component://content/widget/content/ContentMenus.xml"/>  
+                                <include-menu name="contentTopMenu" location="component://content/widget/content/ContentMenus.xml"/>
                                 <decorator-section-include name="body"/>
                             </widgets>
                             <fail-widgets>
@@ -137,8 +137,8 @@
             </widgets>
         </section>
     </screen>
-    
-    
+
+
     <screen name="commonDataResourceDecorator">
         <section>
             <actions>
@@ -177,7 +177,7 @@
             </widgets>
         </section>
     </screen>
-    
+
     <screen name="commonCompDocDecorator">
         <section>
             <actions>
@@ -194,7 +194,7 @@
             </widgets>
         </section>
     </screen>
-    
+
     <screen name="commonContentSetupDecorator">
         <section>
             <actions>
@@ -283,7 +283,7 @@
             </widgets>
         </section>
     </screen>
-    
+
     <screen name="commonLayoutDecorator">
         <section>
             <actions>
@@ -344,20 +344,30 @@
                     <widgets>
                         <container style="responseSelected">
                             <label text="${rsp.contentName} - ${rsp.description} [${rsp.contentId}]" style="responseheader" />
-                            <link  text="${uiLabelMap.PartyReply}" target="addForumThreadMessage?forumGroupId=${parameters.forumGroupId}&amp;forumId=${parameters.forumId}&amp;forumMessageIdTo=${rsp.contentId}&amp;contentAssocTypeId=RESPONSE" style="buttontext"/>
+                            <link  text="${uiLabelMap.PartyReply}" target="addForumThreadMessage" style="buttontext">
+                                <parameter param-name="forumGroupId" from-field="parameters.forumGroupId"/>
+                                <parameter param-name="forumId" from-field="parameters.forumId"/>
+                                <parameter param-name="forumMessageIdTo" from-field="rsp.contentId"/>
+                                <parameter param-name="contentAssocTypeId" value="RESPONSE" />
+                            </link>
                             <container style="responsetext">
                                 <include-form name="EditForumThreadMessage" location="component://content/widget/forum/ForumForms.xml"/>
                             </container>
-                        </container>    
+                        </container>
                     </widgets>
                     <fail-widgets>
                         <container>
                             <label text="${rsp.contentName} - ${rsp.description} [${rsp.contentId}]" style="responseheader" />
-                            <link  text="${uiLabelMap.PartyReply}" target="addForumThreadMessage?forumGroupId=${parameters.forumGroupId}&amp;forumId=${parameters.forumId}&amp;forumMessageIdTo=${rsp.contentId}&amp;contentAssocTypeId=RESPONSE" style="buttontext"/>
+                            <link  text="${uiLabelMap.PartyReply}" target="addForumThreadMessage" style="buttontext">
+                                <parameter param-name="forumGroupId" from-field="parameters.forumGroupId"/>
+                                <parameter param-name="forumId" from-field="parameters.forumId"/>
+                                <parameter param-name="forumMessageIdTo" from-field="rsp.contentId"/>
+                                <parameter param-name="contentAssocTypeId" value="RESPONSE" />
+                            </link>
                             <container style="responsetext">
                                 <include-form name="EditForumThreadMessage" location="component://content/widget/forum/ForumForms.xml"/>
                             </container>
-                        </container>    
+                        </container>
                     </fail-widgets>
                 </section>
             </widgets>

Modified: ofbiz/trunk/applications/party/widget/partymgr/PartyMenus.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/widget/partymgr/PartyMenus.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/widget/partymgr/PartyMenus.xml (original)
+++ ofbiz/trunk/applications/party/widget/partymgr/PartyMenus.xml Wed Apr  1 08:41:00 2009
@@ -157,7 +157,14 @@
             <condition>
                 <if-has-permission permission="ORDERMGR" action="_VIEW"/>
             </condition>
-            <link target="/ordermgr/control/searchorders?lookupFlag=Y&amp;hideFields=Y&amp;partyId=${partyId}&amp;externalLoginKey=${externalLoginKey}&amp;viewIndex=1&amp;viewSize=20" url-mode="inter-app"/>
+            <link target="/ordermgr/control/searchorders" url-mode="inter-app">
+                <parameter param-name="lookupFlag" value="Y"/>
+                <parameter param-name="hideFields" value="Y"/>
+                <parameter param-name="partyId" from-field="partyId"/>
+                <parameter param-name="externalLoginKey" from-field="externalLoginKey"/>
+                <parameter param-name="viewIndex" value="1"/>
+                <parameter param-name="viewSize" value="20"/>
+            </link>
         </menu-item>
         <menu-item name="newQuote" title="${uiLabelMap.OrderNewQuote}">
             <condition>
@@ -349,7 +356,12 @@
                     <if-has-permission permission="PARTYMGR_CME-EMAIL_UPDATE"/>
                 </and>
             </condition>
-            <link target="setCommunicationEventRoleStatus?communicationEventId=${communicationEvent.communicationEventId}&amp;partyId=${myCommunicationEventRole.partyId&amp;roleType=${myCommunicationEventRole.roleTypeId}&amp;statusId=COM_ROLE_COMPLETED"/>
+            <link target="setCommunicationEventRoleStatus">
+                <parameter param-name="communicationEventId" from-field="communicationEvent.communicationEventId"/>
+                <parameter param-name="partyId" from-field="myCommunicationEventRole.partyId"/>
+                <parameter param-name="roleType" from-field="myCommunicationEventRole.roleTypeId"/>
+                <parameter param-name="statusId" value="COM_ROLE_COMPLETED""/>
+            </link>
         </menu-item>
         <menu-item name="delete" title="${uiLabelMap.CommonDelete}">
             <condition>

Modified: ofbiz/trunk/specialpurpose/ecommerce/widget/ForumScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/widget/ForumScreens.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/widget/ForumScreens.xml (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/widget/ForumScreens.xml Wed Apr  1 08:41:00 2009
@@ -549,14 +549,24 @@
                         <container style="responseSelected">
                             <label text="${rsp.contentName}" style="responseheader" />
                             <label text=" - ${rsp.description} [${rsp.contentId}]" style="responsetext" />
-                            <link  text="${uiLabelMap.CommonView}" target="ViewForumMessage?forumId=${parameters.forumId}&amp;threadContentId=${parameters.threadContentId}&amp;responseContentId=${rsp.contentId}&amp;contentAssocTypeId=RESPONSE" style="buttontext"/>
+                            <link  text="${uiLabelMap.CommonView}" target="ViewForumMessage" style="buttontext">
+                                <parameter param-name="forumId" from-field="parameters.forumId"/>
+                                <parameter param-name="threadContentId" from-field="parameters.threadContentId"/>
+                                <parameter param-name="responseContentId" from-field="rsp.contentId"/>
+                                <parameter param-name="contentAssocTypeId" value="RESPONSE"/>
+                            </link>
                         </container>
                     </widgets>
                     <fail-widgets>
                         <container>
                             <label text="${rsp.contentName}" style="responseheader" />
                             <label text=" - ${rsp.description} [${rsp.contentId}]" style="responsetext" />
-                            <link  text="${uiLabelMap.CommonView}" target="ViewForumMessage?forumId=${parameters.forumId}&amp;threadContentId=${parameters.threadContentId}&amp;responseContentId=${rsp.contentId}&amp;contentAssocTypeId=RESPONSE" style="buttontext"/>
+                            <link  text="${uiLabelMap.CommonView}" target="ViewForumMessage" style="buttontext">
+                                <parameter param-name="forumId" from-field="parameters.forumId"/>
+                                <parameter param-name="threadContentId" from-field="parameters.threadContentId"/>
+                                <parameter param-name="responseContentId" from-field="rsp.contentId"/>
+                                <parameter param-name="contentAssocTypeId" value="RESPONSE"/>
+                            </link>
                         </container>
                     </fail-widgets>
                 </section>

Modified: ofbiz/trunk/specialpurpose/mypage/widget/MyPageMenus.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/mypage/widget/MyPageMenus.xml?rev=760815&r1=760814&r2=760815&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/mypage/widget/MyPageMenus.xml (original)
+++ ofbiz/trunk/specialpurpose/mypage/widget/MyPageMenus.xml Wed Apr  1 08:41:00 2009
@@ -207,7 +207,12 @@
             <condition>
                 <if-has-permission permission="PARTYMGR_CME-EMAIL_CREATE"/>
             </condition>
-            <link target="NewCommunicationEvent?communicationEventTypeId=EMAIL_COMMUNICATION&amp;donePage=${parameters.donePage}&amp;partyIdFrom=${userLogin.partyId}&amp;statusId=COM_PENDING"/>
+            <link target="NewCommunicationEvent">
+                <parameter param-name="communicationEventTypeId" value="EMAIL_COMMUNICATION"/>
+                <parameter param-name="donePage" from-field="parameters.donePage"/>
+                <parameter param-name="partyIdFrom" from-field="userLogin.partyId"/>
+                <parameter param-name="statusId" value="COM_PENDING"/>
+            </link>
         </menu-item>
         <menu-item name="newInternalNote" title="${uiLabelMap.PartyNewInternalNote}">
             <condition>
@@ -263,7 +268,12 @@
                     <if-has-permission permission="PARTYMGR_CME-EMAIL_CREATE"/>
                 </and>
             </condition>
-            <link target="setCommunicationEventRoleStatus?communicationEventId=${parameters.communicationEventId}&amp;partyId=${userLogin.partyId}&amp;roleTypeId=${communicationEventRole.roleTypeId}&amp;statusId=COM_ROLE_COMPLETED"/>
+            <link target="setCommunicationEventRoleStatus">
+                <parameter param-name="communicationEventId" from-field="parameters.communicationEventId"/>
+                <parameter param-name="partyId" from-field="userLogin.partyId"/>
+                <parameter param-name="roleTypeId" from-field="communicationEventRole.roleTypeId"/>
+                <parameter param-name="statusId" value="COM_ROLE_COMPLETED"/>
+            </link>
         </menu-item>
         <menu-item name="send" title="${uiLabelMap.CommonSend}">
             <condition>
@@ -282,7 +292,13 @@
                     <if-has-permission permission="PARTYMGR_CME-EMAIL_DELETE"/>
                 </and>
             </condition>
-            <link target="RemoveCommunicationEventRole?communicationEventId=${parameters.communicationEventId}&amp;roleTypeId=${communicationEventRole.roleTypeId}&amp;partyId=${userLogin.partyId}&amp;deleteCommEventIfLast=Y&amp;delContentDataResource=Y"/>
+            <link target="RemoveCommunicationEventRole">
+                <parameter param-name="communicationEventId" from-field="parameters.communicationEventId"/>
+                <parameter param-name="roleTypeId" from-field="communicationEventRole.roleTypeId"/>
+                <parameter param-name="partyId" from-field="userLogin.partyId"/>
+                <parameter param-name="deleteCommEventIfLast" value="Y"/>
+                <parameter param-name="delContentDataResource" value="Y"/>
+            </link>
         </menu-item>
     </menu>
     <menu name="RequestTabBar" extends="CommonTabBarMenu" extends-resource="component://common/widget/CommonMenus.xml">