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

Fix: call to read_restricted? only if instance_to_base_on not nil #86

Merged
merged 1 commit into from
May 31, 2023

Conversation

syphax-bouazzouni
Copy link

Fix a bug, when checking the access writes of a list of models; but one it depending models is nil (e.g ontologies with views)

@syphax-bouazzouni syphax-bouazzouni merged commit 0ddc070 into development May 31, 2023
syphax-bouazzouni added a commit that referenced this pull request Sep 5, 2023
…d_on-nil

Fix: call to read_restricted?  only if instance_to_base_on not nil
syphax-bouazzouni added a commit that referenced this pull request Dec 5, 2023
* fix  json serializer nil exception (#79)

if hash was missing @context key the json serializer throwed a nil exception

* Merge pull request #72 from ontoportal-lirmm/feature/migrate-attrbiutes-metadata-to-scheme-file

Feature: Update ontology submission model 2023 - Part 1  -  Update attributes validators and metadata

* Merge pull request #82 from ontoportal-lirmm/feature/embed-meterics-for-submissions

Feature: embed metrics for submission model

* Merge pull request #86 from ontoportal-lirmm/fix/read_restricted_based_on-nil

Fix: call to read_restricted?  only if instance_to_base_on not nil

* Merge pull request #90 from ontoportal-lirmm/feature/add-persons-and-organization-models

Feature: Update ontology submission model 2023 - Part 2 - Add person & organization models

* in tests no more use the reload argument in the exist? method

---------

Co-authored-by: Raimi Solorzano Niederhausen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant