This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new fb3a59f3ee Improved: PARTY - Move from hard-code menu location to 
parameterized (OFBIZ-12945) (#730)
fb3a59f3ee is described below

commit fb3a59f3ee3f455cd4b1ca354e9da263765eaf17
Author: Pierre Smits <pierre.sm...@somonar.com>
AuthorDate: Sat Mar 16 15:05:38 2024 +0100

    Improved: PARTY - Move from hard-code menu location to parameterized 
(OFBIZ-12945) (#730)
    
    Move the menu location in the various party screens from hard-coded to 
parameterized.
    
    modified:
    - CommonScreens.xml
    - CommunicationEventScreens.xml
    - PartyClassificationScreens.xml
    - PartyForms.xml
    - PartyInvitationScreens.xml
    - PartyScreens.xml
    - ProfileScreens.xml
    - VisitScreens.xml
---
 applications/party/widget/partymgr/CommonScreens.xml | 10 +++++-----
 .../widget/partymgr/CommunicationEventScreens.xml    | 20 ++++++++++----------
 .../widget/partymgr/PartyClassificationScreens.xml   |  8 ++++----
 applications/party/widget/partymgr/PartyForms.xml    |  2 +-
 .../party/widget/partymgr/PartyInvitationScreens.xml |  8 ++++----
 applications/party/widget/partymgr/PartyScreens.xml  |  8 ++++----
 .../party/widget/partymgr/ProfileScreens.xml         | 12 ++++++------
 applications/party/widget/partymgr/VisitScreens.xml  |  4 ++--
 8 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/applications/party/widget/partymgr/CommonScreens.xml 
b/applications/party/widget/partymgr/CommonScreens.xml
index 44318d7d6e..b292e4ece0 100644
--- a/applications/party/widget/partymgr/CommonScreens.xml
+++ b/applications/party/widget/partymgr/CommonScreens.xml
@@ -48,7 +48,7 @@ under the License.
                 <set field="layoutSettings.styleSheets[]" 
value="/partymgr/static/partymgr.css" global="true"/>
                 <set field="activeApp" value="partymgr" global="true"/>
                 <set field="applicationMenuName" value="PartyAppBar" 
global="true"/>
-                <set field="applicationMenuLocation" 
value="component://party/widget/partymgr/PartyMenus.xml" global="true"/>
+                <set field="applicationMenuLocation" 
value="${parameters.mainMenuLocation}" global="true"/>
                 <set field="helpAnchor" from-field="helpAnchor" 
default-value="_party"/>
                 <set field="applicationTitle" 
from-field="uiLabelMap.PartyManagerApplication" global="true"/>
             </actions>
@@ -68,7 +68,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                         <section>
                             <condition>
                                 <and>
@@ -77,7 +77,7 @@ under the License.
                                 </and>
                             </condition>
                             <widgets>
-                                <include-menu name="ProfileTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="ProfileTabBar" 
location="${parameters.mainMenuLocation}"/>
                             </widgets>
                         </section>
                     </decorator-section>
@@ -91,7 +91,7 @@ under the License.
                                 <section>
                                     <condition><not><if-empty 
field="partyId"/></not></condition>
                                     <widgets>
-                                        <include-menu name="ProfileSubTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                        <include-menu name="ProfileSubTabBar" 
location="${parameters.mainMenuLocation}"/>
                                         <container style="clear"/>
                                         <container style="page-title">
                                             <section>
@@ -156,7 +156,7 @@ under the License.
             <widgets>
                 <decorator-screen name="ShortcutDecorator" 
location="component://common/widget/CommonScreens.xml">
                     <decorator-section name="body">
-                        <include-menu name="PartyShortcutAppBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="PartyShortcutAppBar" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                 </decorator-screen>
             </widgets>
diff --git a/applications/party/widget/partymgr/CommunicationEventScreens.xml 
b/applications/party/widget/partymgr/CommunicationEventScreens.xml
index 1d24800ff7..320ecac5b4 100644
--- a/applications/party/widget/partymgr/CommunicationEventScreens.xml
+++ b/applications/party/widget/partymgr/CommunicationEventScreens.xml
@@ -29,14 +29,14 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                         <section>
                             <condition>
                                 <if-has-permission permission="PARTYMGR" 
action="_VIEW"/>
                             </condition>
                             <widgets>
-                                <include-menu name="CommEventTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
-                                <include-menu name="CommSubTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="CommEventTabBar" 
location="${parameters.mainMenuLocation}"/>
+                                <include-menu name="CommSubTabBar" 
location="${parameters.mainMenuLocation}"/>
                             </widgets>
                         </section>
                     </decorator-section>
@@ -66,8 +66,8 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                                <include-menu name="CommEventTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
-                                <include-menu name="CommSubTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="CommEventTabBar" 
location="${parameters.mainMenuLocation}"/>
+                                <include-menu name="CommSubTabBar" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <section>
@@ -111,10 +111,10 @@ under the License.
                                         <not><if-empty field="partyId"/></not>
                                     </condition>
                                     <widgets>
-                                        <include-menu name="ProfileTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                        <include-menu name="ProfileTabBar" 
location="${parameters.mainMenuLocation}"/>
                                     </widgets>
                                 </section>
-                                <include-menu name="CommFindTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="CommFindTabBar" 
location="${parameters.mainMenuLocation}"/>
                                 <container><label style="h1" 
text="${uiLabelMap.PartyPendingCommunicationEvents}"/></container>
                                 <container>
                                     <link style="smallSubmit" 
target="ViewCommunicationEvent" text="${uiLabelMap.PartyNewCommunication}">
@@ -803,7 +803,7 @@ under the License.
                                         <not><if-empty field="partyId"/></not>
                                     </condition>
                                     <widgets>
-                                        <include-menu name="ProfileTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                        <include-menu name="ProfileTabBar" 
location="${parameters.mainMenuLocation}"/>
                                     </widgets>
                                 </section>
                                 <container style="h1"><label 
text="${uiLabelMap.PageTitleEditCommContent}"/></container>
@@ -877,7 +877,7 @@ under the License.
             <widgets>
                 <screenlet title="${uiLabelMap.PartyCommunicationsOfParty}: 
${partyName.firstName} ${partyName.middleName} ${partyName.lastName} 
${partyName.groupName} [${partyId}] "
                     navigation-menu-name="CommunicationsMenu" 
navigation-form-name="ListPartyCommEvents" name="myComms">
-                    <include-menu name="CommunicationsMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                    <include-menu name="CommunicationsMenu" 
location="${parameters.mainMenuLocation}"/>
                     <section><!-- List my communication events -->
                         <condition>
                             <or>
@@ -901,7 +901,7 @@ under the License.
                             <set field="partyIdFrom" 
from-field="parameters.partyId" default-value="${userLogin.partyId}"/>
                         </actions>
                         <widgets>
-                            <include-menu name="MyCommSubTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                            <include-menu name="MyCommSubTabBar" 
location="${parameters.mainMenuLocation}"/>
                             <include-screen name="commOverview"/>
                         </widgets>
                     </section>
diff --git a/applications/party/widget/partymgr/PartyClassificationScreens.xml 
b/applications/party/widget/partymgr/PartyClassificationScreens.xml
index c2e38dd798..43cf768117 100644
--- a/applications/party/widget/partymgr/PartyClassificationScreens.xml
+++ b/applications/party/widget/partymgr/PartyClassificationScreens.xml
@@ -60,13 +60,13 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                         <section>
                             <condition>
                                 <if-service-permission 
service-name="partyBasePermissionCheck" main-action="VIEW"/>
                             </condition>
                             <widgets>
-                                <include-menu name="PartyClassificationTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="PartyClassificationTabBar" 
location="${parameters.mainMenuLocation}"/>
                             </widgets>
                         </section>
                     </decorator-section>
@@ -105,7 +105,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <screenlet 
title="${uiLabelMap.PartyClassificationGroups}">
@@ -139,7 +139,7 @@ under the License.
                                 <if-service-permission 
service-name="partyBasePermissionCheck" main-action="VIEW"/>
                             </condition>
                             <widgets>
-                                <include-menu name="PartyClassificationTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="PartyClassificationTabBar" 
location="${parameters.mainMenuLocation}"/>
                             </widgets>
                         </section>
                     </decorator-section>
diff --git a/applications/party/widget/partymgr/PartyForms.xml 
b/applications/party/widget/partymgr/PartyForms.xml
index 9bc3d43ce5..40feb9b16e 100644
--- a/applications/party/widget/partymgr/PartyForms.xml
+++ b/applications/party/widget/partymgr/PartyForms.xml
@@ -163,7 +163,7 @@ under the License.
         <field name="mainRole" title="${uiLabelMap.PartyMainRole}"><display 
description="${mainRole.description}"/></field>
         <field name="createdDate" sort-field="true"><display 
type="date"/></field>
         <field name="lastModifiedDate" sort-field="true"><display 
type="date"/></field>
-        <field name="partyAction" title=" "><include-menu 
name="ListPartyAction" 
location="component://party/widget/partymgr/PartyMenus.xml"/></field>
+        <field name="partyAction" title=" "><include-menu 
name="ListPartyAction" location="${parameters.mainMenuLocation}"/></field>
         <sort-order>
              <last-field name="createdDate"/>
              <last-field name="lastModifiedDate"/>
diff --git a/applications/party/widget/partymgr/PartyInvitationScreens.xml 
b/applications/party/widget/partymgr/PartyInvitationScreens.xml
index f72151b202..6ff8fb4676 100644
--- a/applications/party/widget/partymgr/PartyInvitationScreens.xml
+++ b/applications/party/widget/partymgr/PartyInvitationScreens.xml
@@ -30,7 +30,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                         <section>
                             <condition>
                                 <and>
@@ -39,7 +39,7 @@ under the License.
                                 </and>
                             </condition>
                             <widgets>
-                                <include-menu name="PartyInvitationTabBar" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                <include-menu name="PartyInvitationTabBar" 
location="${parameters.mainMenuLocation}"/>
                             </widgets>
                         </section>
                     </decorator-section>
@@ -82,7 +82,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <section>
@@ -123,7 +123,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <section>
diff --git a/applications/party/widget/partymgr/PartyScreens.xml 
b/applications/party/widget/partymgr/PartyScreens.xml
index 7d39873af6..ffeca925b2 100644
--- a/applications/party/widget/partymgr/PartyScreens.xml
+++ b/applications/party/widget/partymgr/PartyScreens.xml
@@ -42,7 +42,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <platform-specific><html><html-template 
multi-block="true" 
location="component://common-theme/template/includes/SetMultipleSelectJs.ftl"/></html></platform-specific>
@@ -54,7 +54,7 @@ under the License.
                             <widgets>
                                 <screenlet id="FindPartyPanel" 
title="${uiLabelMap.PartyFindParty}" navigation-menu-name="CreateNewParty"
                                            navigation-form-name="LookupParty" 
save-collapsed="false" collapsible="true" 
initially-collapsed="${initialyCollapsed}">
-                                    <include-menu name="CreateNewParty" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                    <include-menu name="CreateNewParty" 
location="${parameters.mainMenuLocation}"/>
                                     <include-form name="LookupParty" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                                 </screenlet>
                                 <screenlet 
title="${uiLabelMap.CommonSearchResults}" navigation-form-name="ListParty">
@@ -1314,7 +1314,7 @@ under the License.
                              <include-grid name="ListPartyIdentification" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                         </screenlet>
                         <screenlet 
title="${uiLabelMap.PartyPartyIdentification}" 
navigation-menu-name="NewPartyIdentification" 
id="PartyIdentificationCreationPanel">
-                             <include-menu name="NewPartyIdentification" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                             <include-menu name="NewPartyIdentification" 
location="${parameters.mainMenuLocation}"/>
                              <include-form name="EditPartyIdentification" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                          </screenlet>
                     </decorator-section>
@@ -1395,7 +1395,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <screenlet title="${uiLabelMap.PartyParty} 
${uiLabelMap.CommonImportExport} ID Name, single role (employee, customer, 
supplier) and contactmechs">
diff --git a/applications/party/widget/partymgr/ProfileScreens.xml 
b/applications/party/widget/partymgr/ProfileScreens.xml
index 97b7a684f9..fb02390f51 100644
--- a/applications/party/widget/partymgr/ProfileScreens.xml
+++ b/applications/party/widget/partymgr/ProfileScreens.xml
@@ -63,7 +63,7 @@
                             </condition>
                             <widgets>
                                 <screenlet 
title="${uiLabelMap.PartyPersonalInformation}" 
navigation-menu-name="PersonUpdate" id="PartyPersonalInformationPanel">
-                                           <include-menu name="PersonUpdate" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                           <include-menu name="PersonUpdate" 
location="${parameters.mainMenuLocation}"/>
                                     <include-form name="ViewPartyPerson" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                                     <section>
                                         <condition>
@@ -102,7 +102,7 @@
                                </condition>
                             <widgets>
                                 <screenlet 
title="${uiLabelMap.PartyPartyGroupInformation}" 
navigation-menu-name="GroupUpdate" id="PartyGroupInformationPanel">
-                                    <include-menu name="GroupUpdate" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                                    <include-menu name="GroupUpdate" 
location="${parameters.mainMenuLocation}"/>
                                     <include-form name="ViewPartyGroup" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                                     <section>
                                         <condition>
@@ -160,7 +160,7 @@
             </actions>
             <widgets>
                 <screenlet title="${uiLabelMap.PartyPartyIdentifications}" 
navigation-menu-name="NewPartyIdentification" id="PartyIdentificationPanel">
-                    <include-menu name="NewPartyIdentification" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                    <include-menu name="NewPartyIdentification" 
location="${parameters.mainMenuLocation}"/>
                     <include-form name="ListPartyIdentification" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                 </screenlet>
             </widgets>
@@ -493,7 +493,7 @@
             </actions>
             <widgets>
                 <screenlet title="${uiLabelMap.PartyShipperAccount}" 
navigation-menu-name="AddShipper">
-                    <include-menu name="AddShipper" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                    <include-menu name="AddShipper" 
location="${parameters.mainMenuLocation}"/>
                     <include-form name="ListCarrierAccounts" 
location="component://party/widget/partymgr/PartyForms.xml"/>
                 </screenlet>
             </widgets>
@@ -528,7 +528,7 @@
                     </condition>
                     <widgets>
                         <screenlet 
title="${uiLabelMap.PartyListRelatedContacts}" 
navigation-menu-name="AddRelAccountContacts">
-                            <include-menu name="AddRelAccountContacts" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                            <include-menu name="AddRelAccountContacts" 
location="${parameters.mainMenuLocation}"/>
                             <section>
                                 <condition>
                                     <not><if-empty 
field="parameters.editPartyRel"/></not>
@@ -563,7 +563,7 @@
                     </condition>
                     <widgets>                        
                         <screenlet 
title="${uiLabelMap.PartyListRelatedAccounts}" 
navigation-menu-name="AddRelContactAccounts">  
-                            <include-menu name="AddRelContactAccounts" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                            <include-menu name="AddRelContactAccounts" 
location="${parameters.mainMenuLocation}"/>
                             <section>
                                 <condition>
                                     <not><if-empty 
field="parameters.editPartyRel"/></not>
diff --git a/applications/party/widget/partymgr/VisitScreens.xml 
b/applications/party/widget/partymgr/VisitScreens.xml
index b946d992b2..389696f90f 100644
--- a/applications/party/widget/partymgr/VisitScreens.xml
+++ b/applications/party/widget/partymgr/VisitScreens.xml
@@ -31,7 +31,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <section>
@@ -68,7 +68,7 @@ under the License.
             <widgets>
                 <decorator-screen name="main-decorator" 
location="${parameters.mainDecoratorLocation}">
                     <decorator-section name="pre-body">
-                        <include-menu name="MainActionMenu" 
location="component://party/widget/partymgr/PartyMenus.xml"/>
+                        <include-menu name="MainActionMenu" 
location="${parameters.mainMenuLocation}"/>
                     </decorator-section>
                     <decorator-section name="body">
                         <include-screen name="LoggedInUsersScreen" />

Reply via email to