Add debug level log message when afform submissions catch exceptions
authorRich Lott / Artful Robot <code.commits@artfulrobot.uk>
Wed, 5 Jul 2023 15:57:47 +0000 (16:57 +0100)
committerRich Lott / Artful Robot <code.commits@artfulrobot.uk>
Wed, 5 Jul 2023 15:57:47 +0000 (16:57 +0100)
The code comments suggest we don't really know what to do here,
previously we did nothing, which was not helpful to a developer
trying to figure out why their code did nothing.

Now we emit the exception's message to the log file.

ext/afform/core/Civi/Api4/Action/Afform/Submit.php

index df072b1d7b6dc385af0fbf558428b6a689dff3b0..d31b29d1a1798601707712038ca58b619150b77f 100644 (file)
@@ -331,6 +331,7 @@ class Submit extends AbstractProcessor {
       catch (\CRM_Core_Exception $e) {
         // What to do here? Sometimes we should silently ignore errors, e.g. an optional entity
         // intentionally left blank. Other times it's a real error the user should know about.
+        \Civi::log()->debug("Silently ignoring exception in Afform processGenericEntity call: " . $e->getMessage());
       }
     }
   }