Sending E-mail, related-to and adresses (bug?)

wips · 7 May 2020 13:55

Hi

not sure if this is a bug or simply a weird feature, but I managed to send e-mail to a related-to account email address by accident, while I was only supposed to send the message internally to a colleague. This happened when I marked a message is related to Account and then used the advanced search to find that account. Apparently, when doing this, the email address was added automatically to the To: field. I noticed this at the moment I clicked send…
This only happens when using the advanced search window, not when typing directly into the related-to item box.
I think it is incorrect behavior. Thoughts?

pgr · 7 May 2020 15:19

Hi, welcome to the Community!

What are you calling “Advanced search”? Is it in the Email Compose window, this bit?

wips · 7 May 2020 15:56

Image tells more…

pgr · 7 May 2020 17:13

Ah ok, I get it.

I think this is intended behaviour. It looks like a facilitator, something to speed up work. I guess if your work flow typically doesn’t require this, you better warn your users to pay attention.

If it’s really annoying you might look for a way to customize it, I think this is where it’s coming from:

github.com

salesagility/SuiteCRM/blob/master/modules/Emails/include/ComposeView/EmailsComposeView.js#L1359


    confirmed(args);
    mb.remove();
  });


  mb.on('cancel', function () {
    "use strict";
    mb.remove();
  });
};


$.fn.EmailsComposeView.onParentSelect = function (args) {
  set_return(args);
  if (isValidEmail(args.name_to_value_array.email1)) {
    var emailAddress = args.name_to_value_array.email1;
    var self = $('[name="' + args.form_name + '"]');
    var toField = $(self).find('[name=to_addrs_names]');
    if (toField.val().indexOf(emailAddress) === -1) {
      var toFieldVal = toField.val();
      if (toFieldVal === '') {
        toField.val(emailAddress);
      } else {

(I don’t know if you’re a developer or if that is all chinese to you, sorry if it is)

wips · 7 May 2020 19:06

Thanks, not really a developer, but I can comment out code when need be
It is an annoyance and if it is intended behavior then I would say it is a logic flaw. It is actually a huge risk that sensitive messages end up in the wrong mailboxes due to simple human error. Related to is not the same as a target for me.
I just migrated an antique sugar ce to the latest suite. Only two users so we got our first lesson.

pgr · 8 May 2020 09:19

You definitely have a point. Sending internal email about a customer, to that customer, unknowingly, is not a small matter. However I wonder what would be the best way to get this UI right, as I am sure that in many companies this is a significant time-saver, and that their flow is always to send to the customer when they select the relationship… Maybe we should just flash a warning (something frictionless) when we’re adding the email into the “to” field…

I suggest that you open an Issue on Github explaining this.