Informasjonsforvaltning / modelldcatnotordf

Apache License 2.0
4 stars 0 forks source link

Undersøkelse : Fjerne overflødig skolemiseringssjekk #195

Closed frederik-ronnevig closed 3 years ago

frederik-ronnevig commented 3 years ago

Det at vi nå har skolemisering over alt i to_graph-metodene skulle også gjøre at vi kan sløyfe skolemiseringstesten if not getattr(self.forms_symmetry_with, "identifier", None):

        if getattr(self, "forms_symmetry_with", None):
            if isinstance(self.forms_symmetry_with, ModelProperty):
                if not getattr(self.forms_symmetry_with, "identifier", None):
                    self.forms_symmetry_with.identifier = Skolemizer.add_skolemization()
                _forms_symmetry_with = URIRef(self.forms_symmetry_with.identifier)
                for _s, p, o in self.forms_symmetry_with._to_graph().triples(
                    (None, None, None)
                ):
                    self._g.add((_s, p, o))
            elif isinstance(self.forms_symmetry_with, str):
                _forms_symmetry_with = URIRef(self.forms_symmetry_with)
frederik-ronnevig commented 3 years ago

Det er dessverre ikke mulig å forkorte koden noe ytterligere her. Ved serialisering av object properties kalles ellers ikke skolemiseringsfunksjonen før object property'en skal legges til i grafen og dermed blir objektet stående uten identifikator. Så skolemiseringssjekken må gjøres her likevel..