https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28803

--- Comment #15 from Jonathan Druart <[email protected]> ---
This is tight to bug 28870. I think we should push both altogether.

However there is one thing that seems wrong to me:

+    my $email = try {
+        Koha::Email->create(
[SKIP]
+    }
+    catch {
+        _set_message_status(
+            {
+                message_id   => $message->{'message_id'},
+                status       => 'failed',
+                failure_code => 'INVALID_EMAIL'
+            }
+        );
+        return 0;
+    };

Koha::Email->create is raising different exception, and Email::Stuffer
certainly raised its own.

Shouldn't we actually base this on top of bug 28870 and simply call (the newly
added) Koha::Email->is_valid to check if the email if valid?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to