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

Clear "rows" on multiple MetaField::Compound fields #368

Open
drn05r opened this issue Jan 30, 2024 · 1 comment
Open

Clear "rows" on multiple MetaField::Compound fields #368

drn05r opened this issue Jan 30, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@drn05r
Copy link
Contributor

drn05r commented Jan 30, 2024

Most MetaField::Compound fields tend to have multiple values (i.e. rows). If a compound field has lots of subfields, having to delete all the values on a row is annoying. This is a particular problem if a repository uses the orcid_support_advance plugin as this disables the ORCID sub-field. Therefore, if a user realises they have added the wrong creator/editor they cannot delete the ORCID. Although the plugin will unset or update the ORCID field based on the ID (e.g. email) field for creators/editors this is not apparent in the user interface. Also, up until recently (not fix yet available in a released version) orcid_support's cgi/users/lookup/name has AND-ed the ORCID in the lookup, making it impossible to find any creator/editor other than the one previously entered.

It should be fairly simple to added a cross icon next to the existing up/down arrows that will clear the adjacent row. This would make it quicker to edit multiple compound fields and also avoid any confusion to the user about what metadata will be saved to the record, where disabled sub-fields exist.

@drn05r drn05r added the enhancement New feature or request label Jan 30, 2024
@drn05r drn05r added this to the 3.4.6 milestone Jan 30, 2024
@drn05r drn05r self-assigned this Jan 30, 2024
@dgc
Copy link
Contributor

dgc commented Jan 31, 2024

Doing this in JavaScript is trivial: just delete the row element in the DOM and the EPrints workflow code will sort it out from there.

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

No branches or pull requests

2 participants