You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mb...@apache.org on 2015/05/20 23:03:58 UTC

svn commit: r1680675 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java

Author: mbrohl
Date: Wed May 20 21:03:58 2015
New Revision: 1680675

URL: http://svn.apache.org/r1680675
Log:
Applied slightly modified patch from jira issue OFBIZ-6238: Give the order notification email attachment the orderId as filename.

Thanks Martin Becker for reporting the issue and providing the patch.

Modified:
    ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java

Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1680675&r1=1680674&r2=1680675&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Wed May 20 21:03:58 2015
@@ -2611,8 +2611,11 @@ public class OrderServices {
                 bodyScreenLocation = ProductStoreWorker.getDefaultProductStoreEmailScreenLocation(emailType);
             }
             sendMap.put("bodyScreenUri", bodyScreenLocation);
+
             String xslfoAttachScreenLocation = productStoreEmail.getString("xslfoAttachScreenLocation");
             sendMap.put("xslfoAttachScreenLocation", xslfoAttachScreenLocation);
+            // add attachmentName param to get an attachment namend "[oderId].pdf" instead of default "Details.pdf"
+            sendMap.put("attachmentName", orderId + ".pdf");
         } else {
             sendMap.put("bodyScreenUri", screenUri);
         }