hotfix - Quickmail exclude fix when multiselect is being used.

Author: ChaoticDalo

blocks/quickmail/classes/persistents/message.php

@@ -270,7 +270,7 @@ public function populate_recip_course_msg()
             // are added to the time of it being sent is within 1000s (plenty of time to send)
             // This msg recips can also be generated by viewing scheduled emails.
             $diffy = time() - $msgexists[0]->timemodified;
-            error_log("\n\nWhat is diffy: ". $diffy);
+            // error_log("\n\nWhat is diffy: ". $diffy);
             // This can get stale! Delete all recipients belonging to this
             // message and update with most current list.
             if ($diffy > 1000 && count($recipientuserids) > 0) {

blocks/quickmail/classes/requests/transformers/compose_transformer.php

@@ -79,13 +79,20 @@ public function get_transformed_excluded_entity_ids() {
      */
     public function get_transformed_excluded() {
         $excludes = "";
-        $counter = count($this->form_data->excluded_entity_ids) ?? 0;
-        for($c = 0; $c < $counter; $c++) {
-            $excludes .= $this->form_data->excluded_entity_ids[$c];
-            if ($c+1 < $counter) {
-                $excludes .= ',';
+        // In the config/settings on the course level the instructor can use multiselect
+        // instead of autocomplete which removes the excluded option in the form.
+        // This check stops it from breaking. 
+        if (property_exists($this->form_data, "excluded_entity_ids")) {
+
+            $counter = count($this->form_data->excluded_entity_ids) ?? 0;
+            for($c = 0; $c < $counter; $c++) {
+                $excludes .= $this->form_data->excluded_entity_ids[$c];
+                if ($c+1 < $counter) {
+                    $excludes .= ',';
+                }
             }
         }
+
         return $excludes;
     }