Merge pull request #916 from maykinmedia/issue/sentry-346889-fix-and-…

…logging-3 [Sentry 346889] Fixing fallback and additional logging for determinin…
maykinmedia · Dec 28, 2023 · ff6dba9 · ff6dba9
2 parents aeff3b3 + 300842a
commit ff6dba9
Showing 1 changed file with 19 additions and 3 deletions.
diff --git a/src/open_inwoner/cms/cases/views/status.py b/src/open_inwoner/cms/cases/views/status.py
@@ -269,9 +269,17 @@ def get_second_status_preview(self, statustypen: list) -> Optional[StatusType]:
 
     @property
     def is_file_upload_enabled_for_case_type(self) -> bool:
-        return ZaakTypeInformatieObjectTypeConfig.objects.filter_enabled_for_case_type(
-            self.case.zaaktype
-        ).exists()
+        case_upload_enabled = (
+            ZaakTypeInformatieObjectTypeConfig.objects.filter_enabled_for_case_type(
+                self.case.zaaktype
+            ).exists()
+        )
+        logger.info(
+            "Case {url} has case type file upload: {case_upload_enabled}".format(
+                url=self.case.url, case_upload_enabled=case_upload_enabled
+            )
+        )
+        return case_upload_enabled
 
     @property
     def is_file_upload_enabled_for_statustype(self) -> bool:
@@ -286,13 +294,21 @@ def is_file_upload_enabled_for_statustype(self) -> bool:
                     case=self.case
                 )
             )
+            return False
         except KeyError:
             logger.info(
                 "Could not retrieve status type config for url {url}".format(
                     url=self.case.status.statustype.url
                 )
             )
             return False
+        logger.info(
+            "Case {url} status type {status_type} has status type file upload: {enabled_for_status_type}".format(
+                url=self.case.url,
+                status_type=self.case.status.statustype,
+                enabled_for_status_type=enabled_for_status_type,
+            )
+        )
         return enabled_for_status_type
 
     @property