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

bugfix of issue #373: handling of nested metadata fields with different visibility settings #374

Open
wants to merge 1 commit into
base: dspace-cris-7
Choose a base branch
from

Conversation

saschaszott
Copy link

This PR fixes the bug described in Github issue #373.

@abollini
Copy link
Member

@saschaszott this cannot be considered for 2023.02.00 as, after further investigation, we found that is not appropriate to keep in version 7 the security at the level of each individual value of a nested metadata. We are going to enforce the UI to manage a single security value for a whole "row" of a nested group so that the data will be more consistent. If the need is to keep only part of information of a nested restricted this will be managed via the layout providing different layout where the same nested is shown with more or less details according to the logged in user.
This mean that the import script and your fix must be revisited to get the security level from the leading metadata of each group instead than from the individual metadata in the nested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants