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

Merging Entries does not allow to keep groups of all duplicated entries #8024

Closed
MarcelPa opened this issue Aug 26, 2021 · 2 comments · Fixed by #9022
Closed

Merging Entries does not allow to keep groups of all duplicated entries #8024

MarcelPa opened this issue Aug 26, 2021 · 2 comments · Fixed by #9022

Comments

@MarcelPa
Copy link

Is your suggestion for improvement related to a problem? Please describe.
When using the "merge duplicates" functionality, it is not possible to keep the assigned groups of two duplicates. I am only able to choose the groups of one of the entries. If I choose "keep merged entry", the entry is not assigned to any group afterwards.

Describe the solution you'd like
I would expect the "keep merged entry" function to assign the merged entry to all groups that any duplicated object was assigned to.

Additional context
I think it could also be up for discussion to merge the groups even when explicitly selecting one of the entries (i.e. "Keep left" or "Keep right") during merge. Personally, I cannot think of any use case that would need to merge an entry but not merge the groups.

I assume the the part of the code to fix this can be found here.

@ThiloteE
Copy link
Member

Roughly similar issue in different context: #8836

Renaming or editing groups in the entry editor does not seem to be linked with the rest of JabRef's group system. Apparently, group of an entry that have not been created explicitly in the sidebar, are ignored on the groups sidebar.

@HoussemNasri
Copy link
Member

Refs #6190

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

Successfully merging a pull request may close this issue.

4 participants