Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Property concept_scheme niet telkens opnieuw opbouwen #97

Closed
goessebr opened this issue Jan 15, 2021 · 0 comments
Closed

Property concept_scheme niet telkens opnieuw opbouwen #97

goessebr opened this issue Jan 15, 2021 · 0 comments
Assignees
Milestone

Comments

@goessebr
Copy link
Contributor

Komt uit https://github.com/OnroerendErfgoed/inventaris/pull/4029#issuecomment-760384053

Voorkomen dat bij het opvragen van de property concept_scheme telkens opnieuw een call naar de thesaurus gebeurt. Deze property 1x opzetten als ConceptScheme is voldoende

In https://github.com/OnroerendErfgoed/skosprovider_atramhasis/blob/master/skosprovider_atramhasis/providers.py#L82-L113

   @property
    def concept_scheme(self):
        return self._get_concept_scheme()

aanpassen naar

   # initialiseren van _conceptscheme variabele

   @property
    def concept_scheme(self):
        if self._conceptscheme is None:
            self._conceptscheme = self._get_concept_scheme()
        return self._conceptscheme
@goessebr goessebr changed the title Property concept_scheme Property concept_scheme niet telkens opnieuw opbouwen Jan 15, 2021
@cedrikv cedrikv added this to the Sprint 148 milestone Feb 1, 2021
@vancamti vancamti self-assigned this Feb 4, 2021
goessebr added a commit that referenced this issue Feb 9, 2021
…e_enhancement

#97 bugfix concept scheme initialization
@goessebr goessebr closed this as completed Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants