revert date workarounds in api models

Author: swrichards

src/open_inwoner/openzaak/api_models.py

@@ -173,8 +173,8 @@ class ZaakType(ZGWModel):
     # besluittypen: list
 
     # TODO: broken in DECOS
-    # begin_geldigheid: Optional[date] = None
-    # einde_geldigheid: Optional[date] = None
+    begin_geldigheid: Optional[date] = None
+    einde_geldigheid: Optional[date] = None
     versiedatum: Optional[date] = None
     concept: Optional[bool] = None
 
@@ -197,8 +197,8 @@ class InformatieObjectType(ZGWModel):
     omschrijving: str
     vertrouwelijkheidaanduiding: str
     # Decos gives missing as empty strings
-    # begin_geldigheid: Optional[date] = None
-    # einde_geldigheid: Optional[date] = None
+    begin_geldigheid: Optional[date] = None
+    einde_geldigheid: Optional[date] = None
     concept: bool = False
 
 

src/open_inwoner/openzaak/utils.py

@@ -1,13 +1,10 @@
 import logging
 
-from zgw_consumers.api_models.constants import (RolTypes,
-                                                VertrouwelijkheidsAanduidingen)
+from zgw_consumers.api_models.constants import RolTypes, VertrouwelijkheidsAanduidingen
 
-from open_inwoner.openzaak.api_models import (InformatieObject, Rol, Zaak,
-                                              ZaakType)
+from open_inwoner.openzaak.api_models import InformatieObject, Rol, Zaak, ZaakType
 
-from .models import (OpenZaakConfig, ZaakTypeConfig,
-                     ZaakTypeInformatieObjectTypeConfig)
+from .models import OpenZaakConfig, ZaakTypeConfig, ZaakTypeInformatieObjectTypeConfig
 
 logger = logging.getLogger(__name__)
 
@@ -46,13 +43,13 @@ def is_info_object_visible(
     We check on its definitive or archived status, and a maximum confidentiality
     level (compared the ordering from the VertrouwelijkheidsAanduidingen.choices)
     """
-    #    if info_object.status not in ["definitief", "gearchiveerd"]:
-    #        logger.info(
-    #            "Ignoring informatieobject %s as not visible for user: status is neither "
-    #            "'definitief' nor 'gearchiveerd'",
-    #            info_object.url,
-    #        )
-    #        return False
+    if info_object.status not in ["definitief", "gearchiveerd"]:
+        logger.info(
+            "Ignoring informatieobject %s as not visible for user: status is neither "
+            "'definitief' nor 'gearchiveerd'",
+            info_object.url,
+        )
+        return False
 
     return is_object_visible(info_object, max_confidentiality_level)