[ofbiz-framework] branch release18.12 updated: Fixed: MessagingException in sendShipmentScheduledNotification service (OFBIZ-11984)

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

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


The following commit(s) were added to refs/heads/release18.12 by this push:
     new 9b9bc10  Fixed: MessagingException in sendShipmentScheduledNotification service (OFBIZ-11984)
9b9bc10 is described below

commit 9b9bc1009252b141748f228d1ffd35199dd21142
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Wed Aug 26 19:51:47 2020 +0200

    Fixed: MessagingException in sendShipmentScheduledNotification service (OFBIZ-11984)
   
    This MessagingException issue is with sendFrom,
    please refer the the code blocks from minilang and groovy versions below:
   
    <!-- find email address for currently logged in user, set as sendFrom -->
    <entity-and entity-name="PartyAndContactMech" list="curUserPartyAndContactMechs">
        <field-map field-name="partyId" from-field="userLogin.partyId"/>
        <field-map field-name="contactMechTypeId" value="EMAIL_ADDRESS"/>
    </entity-and>
   
    <first-from-list list="curUserPartyAndContactMechs" entry="curUserPartyAndContactMech"/>
    <string-append field="sendEmailMap.sendFrom" string="${curUserPartyAndContactMech.infoString}" prefix=","/>
   
    There should not be a prefix, there is always only 1 sender
    The sendOrderDeliveryScheduleNotification service was also affected
   
    Thanks: Mridul Pathak for analyse
---
 applications/order/minilang/order/OrderDeliveryServices.xml          | 2 +-
 applications/product/minilang/shipment/shipment/ShipmentServices.xml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/applications/order/minilang/order/OrderDeliveryServices.xml b/applications/order/minilang/order/OrderDeliveryServices.xml
index c2e04f1..6341086 100644
--- a/applications/order/minilang/order/OrderDeliveryServices.xml
+++ b/applications/order/minilang/order/OrderDeliveryServices.xml
@@ -88,7 +88,7 @@ under the License.
         <set field="curUserPcmFindMap.contactMechTypeId" value="EMAIL_ADDRESS"/>
         <find-by-and entity-name="PartyAndContactMech" map="curUserPcmFindMap" list="curUserPartyAndContactMechs"/>
         <first-from-list list="curUserPartyAndContactMechs" entry="curUserPartyAndContactMech"/>
-        <string-append field="sendEmailMap.sendFrom" string="${curUserPartyAndContactMech.infoString}" prefix=","/>
+        <string-append field="sendEmailMap.sendFrom" string="${curUserPartyAndContactMech.infoString}"/>
 
         <!-- find email addresses of all parties in SHIPMENT_CLERK roleTypeId, set as sendTo -->
         <set value="SHIPMENT_CLERK" field="shipmentClerkFindMap.roleTypeId"/>
diff --git a/applications/product/minilang/shipment/shipment/ShipmentServices.xml b/applications/product/minilang/shipment/shipment/ShipmentServices.xml
index ade61c6..ea10ef3 100644
--- a/applications/product/minilang/shipment/shipment/ShipmentServices.xml
+++ b/applications/product/minilang/shipment/shipment/ShipmentServices.xml
@@ -740,7 +740,7 @@ under the License.
         </entity-and>
 
         <first-from-list list="curUserPartyAndContactMechs" entry="curUserPartyAndContactMech"/>
-        <string-append field="sendEmailMap.sendFrom" string="${curUserPartyAndContactMech.infoString}" prefix=","/>
+        <string-append field="sendEmailMap.sendFrom" string="${curUserPartyAndContactMech.infoString}"/>
 
         <!-- find email addresses of partyIdFrom, set as sendTo -->
         <set from-field="shipment.partyIdFrom" field="sendToPartyIdMap.${shipment.partyIdFrom}"/>