Re: EmailServices.sendMail causes a NPE, when sendFrom is missing

2018-12-16 Thread Deepak Dixit
I agree with Scott, we have a default email address in general.properties file. And wrapper service should honor this while implementation. It's not a good idea to make sendFrom options false in sendMail interface. Thanks & Regards -- Deepak Dixit On Sat, Dec 15, 2018 at 4:23 AM Scott Gray wro

Re: EmailServices.sendMail causes a NPE, when sendFrom is missing

2018-12-14 Thread Scott Gray
Hi Benjamin, If I were to place a bet, I would say that the system used to provide a default email address and that code has disappeared for some reason. It might pay to look at the history of the service and try to determine why it changed or if it was always broken. FWIW I always try and look i

EmailServices.sendMail causes a NPE, when sendFrom is missing

2018-12-14 Thread Benjamin Jugl
Hello all, at the moment an NPE can be thrown by sending an Email (via the service "sendMail" without giving a value for "sendFrom". I provided a patch for this in OFBIZ-10706 . This simply alters the service configuration, so that sendFrom