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

Mystic "Ändern der Gruppierungsmethode" dialog after modfing a group #5065

Closed
buhtz opened this issue Jun 19, 2019 · 8 comments
Closed

Mystic "Ändern der Gruppierungsmethode" dialog after modfing a group #5065

buhtz opened this issue Jun 19, 2019 · 8 comments

Comments

@buhtz
Copy link

buhtz commented Jun 19, 2019

Current dev version from today.

I modified an existing group. I tried color and also the name/title.
When clicking "Ok" the attached dialog appears.

Zwischenablage01

Maybe it depends on the translation but for me as a user I have no idea what it means. I have not changed the "Gruppierungsmethode". Clicking on Ok should just close the group settings dialog not more.

I see no sense in that extra dialog.

@Siedlerchr
Copy link
Member

You modified a group, JabRef asks now if you want to reassign the original entries from that group to that modified group.
E.g. you change group name from A to B, so JabRef is asking if the entries assigned to your group A should be assigned to the group B now.
So it updates for each entry the group field from A to B.

@buhtz
Copy link
Author

buhtz commented Jun 19, 2019

I see. Technical it make sense.
But from the users point of view it does not. The user did not created a new group where the entries need to be moved to. The user just modified the group. The entries in their did not change.

See the point?

Do not bother the users with questions like that. I also see no case where it would make sense to press Cancel/Abbrechen.

@AEgit
Copy link

AEgit commented Jun 19, 2019

I disagree with the position of @noMICROSOFTbuhtz : If you, by mistake, modify a group name so that it suddenly has the same name as another group in your database and if entries are then automatically changed to the new group name (without asking the user), all hell breaks loose, i. e. it will be really difficult and tedious to get the original group assignment for those items.

So, I would argue to keep this feature as it is (and has been - I am still using 3.8.2 and I am quite happy, that this feature is still in the newer versions as well).

@buhtz
Copy link
Author

buhtz commented Jun 20, 2019

@AEgit I disagree also. If I rename the group to the name of another group that should be checked automaticly by JabRef. If this is the case I would like to have a (well formualted and understandable) warning message about it.

That would be what I expect as a user.

btw: The fact that the name of a group (combined with its path!) is its object-identifier also is a bug by design. But I want open that topic here. I think JabRef has other problems with more priority.

@AEgit
Copy link

AEgit commented Jun 20, 2019

@noMICROSOFTbuhtz While I see your point and I agree in an ideal world this would be a good solution, for now we should probably not rely on JabRef checking this. There are several bugs (as you allude to) that affect the stability and performance of the current JabRef version, that probably should have higher priority at the moment.
This is by no means meant as a critique of the developers: They are doing this work for free and mostly (all?) in their spare time and the transition to JavaFX, as far as I understand, was a necessary (even though painful) step. Right now, the focus should probably be on getting the current JabRef version back to the stability and performance level of version 3.8.2 (if I have time I might be able to provide some benchmarks this weekend that will show that the current JabRef version can be at least 3 times as slow as version 3.8.2 for large databases). I think there also a couple of issues that particularly affect the Linux version of JabRef.
After that, additional features like you mentioned might be worthwhile to think about.

Again a big shout-out and thanks to all the JabRef developers for their help and continuous support!

@tobiasdiez
Copy link
Member

According to the code

dialogService.showErrorDialogAndWait(Localization.lang("There exists already a group with the same name.", Character.toString(keywordDelimiter)));

a warning message should be displayed if there is a group with the same name. Do you still see a reason for the above warning dialog if duplicate group names are prevented?

@AEgit
Copy link

AEgit commented Jun 20, 2019

Ah there you go, this has already been solved in the current developer version (I thought I had seen this issue some time ago on Github but did not bother to check with the new version):

JabRef 5.0-dev--snapshot--2019-06-14--master--d811c175b
Windows 10 10.0 amd64
Java 1.8.0_211

So, there is a warning message which prevents you from creating groups with the same name (nice, this was not the case with JabRef 3.8.2). If this feature is absolutely rock-solid (I have not tested it enough to confirm that), then yes, maybe the dialog mentioned by @noMICROSOFTbuhtz could be removed. I have the feeling the dialog still might be needed in some other rare special-cases, but currently I cannot remember the details. The only thing I remember was, that I was grateful at the time that it existed and I have used the "Cancel" part of the dialog a couple of times (but maybe that was just because groups could have the same name at the time?).

@stefan-kolb stefan-kolb added this to the v5.1 milestone Mar 14, 2020
@tobiasdiez tobiasdiez removed this from the v5.1 milestone Apr 12, 2020
github-actions bot pushed a commit that referenced this issue Nov 1, 2020
5c376b8 Create karlstad-universitet-kau-harvard-swedish.csl (#5083)
6a62c24 Create ucl-institute-of-education-harvard.csl (#5084)
4d1bbb0 Update ferdinand-porsche-fern-fachhochschule.csl (#5073)
3a9f494 Update cureus.csl (#5087)
181c0dd Create apa-no.csl (#5059)
651ce48 Some fixes for Advanced Functional Materials (#5080)
4e3297a Acta Zoologica Hungarica: Don't demote nd particles
c659df7 Create medicina-delle-dipendenze-italian-journal-of-the-addictions.csl (#5078)
d243055 Update geografia-fisica-e-dinamica-quaternaria.csl (#5079)
1ee3809 Update mcgill-en.csl (#5076)
949cdc6 Update mcgill-fr.csl (#5075)
ee7d4e1 Create chem-catalysis.csl (#5074)
ec9faec Switch Molecular Therapy to Cell (#5070)
cf6034c Switch Elsevier to their own ad-vancouver (#5072)
78cca13 Updates in research-institute-for-nature-and-forest.csl (#4958)
68b7478 Update journal-of-neolithic-archaeology.csl (#5065)
524ba93 Create offa.csl (#5066)
9713a82 Update associacao-brasileira-de-normas-tecnicas-ufrgs-note-initials-with-ibid (#5067)
44449df Uodate associacao-brasileira-de-normas-tecnicas-ufrgs-initials.csl (#5061)
baf9efa Create İstanbul Üniversitesi Sosyal Bilimler Enstitüsü Ali Ekber Çına… (#5064)
c5c3531 Fix CORR
853a804 Major Update mcgill-fr.csl to 9th ed. (#5057)
d3e2cd9 Update mcgill-en.csl (#5056)

git-subtree-dir: src/main/resources/csl-styles
git-subtree-split: 5c376b8
github-actions bot pushed a commit to dimitra-karadima/jabref that referenced this issue Nov 15, 2020
55200d0 Create computational-intelligence.csl (JabRef#5102)
6d2efc3 Merge branch 'master' of github.com:citation-style-language/styles
60ab0fb Add Steinbeis SMI
d5e0130 Add AAQR Style
d964f9b Create bitonline-publikationen.csl (JabRef#5090)
0dc1a56 Add "APA 7th edition" to Norsk APA style titles (JabRef#5098)
3b6aaf7 Update norsk-apa-manual.csl (JabRef#5095)
0139c63 Create technische-universitat-dresden-betriebswirtschaftslehre-rechnungswesen-controlling.csl (JabRef#5034)
fdc4497 Update universidade-estadual-de-alagoas-abnt (JabRef#5085)
afdc2ea Create revista-espanola-de-nutricion-humana-y-dietetica.csl (JabRef#5082)
939189d Create citation-compass-apa-note-no.csl (JabRef#5060)
414f5e6  brazilian-journal-of-veterinary-research: add bib sorting (JabRef#5091)
dbae9d7 Add DOI to Thieme German
f4399aa APA: match="any" bugfixes (JabRef#5092)
d0cc2c4 Add APA numeric and update BioEssays (JabRef#5089)
5c376b8 Create karlstad-universitet-kau-harvard-swedish.csl (JabRef#5083)
6a62c24 Create ucl-institute-of-education-harvard.csl (JabRef#5084)
4d1bbb0 Update ferdinand-porsche-fern-fachhochschule.csl (JabRef#5073)
3a9f494 Update cureus.csl (JabRef#5087)
181c0dd Create apa-no.csl (JabRef#5059)
651ce48 Some fixes for Advanced Functional Materials (JabRef#5080)
4e3297a Acta Zoologica Hungarica: Don't demote nd particles
c659df7 Create medicina-delle-dipendenze-italian-journal-of-the-addictions.csl (JabRef#5078)
d243055 Update geografia-fisica-e-dinamica-quaternaria.csl (JabRef#5079)
1ee3809 Update mcgill-en.csl (JabRef#5076)
949cdc6 Update mcgill-fr.csl (JabRef#5075)
ee7d4e1 Create chem-catalysis.csl (JabRef#5074)
ec9faec Switch Molecular Therapy to Cell (JabRef#5070)
cf6034c Switch Elsevier to their own ad-vancouver (JabRef#5072)
78cca13 Updates in research-institute-for-nature-and-forest.csl (JabRef#4958)
68b7478 Update journal-of-neolithic-archaeology.csl (JabRef#5065)
524ba93 Create offa.csl (JabRef#5066)
9713a82 Update associacao-brasileira-de-normas-tecnicas-ufrgs-note-initials-with-ibid (JabRef#5067)
44449df Uodate associacao-brasileira-de-normas-tecnicas-ufrgs-initials.csl (JabRef#5061)
baf9efa Create İstanbul Üniversitesi Sosyal Bilimler Enstitüsü Ali Ekber Çına… (JabRef#5064)
c5c3531 Fix CORR
853a804 Major Update mcgill-fr.csl to 9th ed. (JabRef#5057)
d3e2cd9 Update mcgill-en.csl (JabRef#5056)

git-subtree-dir: src/main/resources/csl-styles
git-subtree-split: 55200d0
@github-actions
Copy link
Contributor

This issue has been inactive for half a year. Since JabRef is constantly evolving this issue may not be relevant any longer and it will be closed in two weeks if no further activity occurs.

As part of an effort to ensure that the JabRef team is focusing on important and valid issues, we would like to ask if you could update the issue if it still persists. This could be in the following form:

  • If there has been a longer discussion, add a short summary of the most important points as a new comment (if not yet existing).
  • Provide further steps or information on how to reproduce this issue.
  • Upvote the initial post if you like to see it implemented soon. Votes are not the only metric that we use to determine the requests that are implemented, however, they do factor into our decision-making process.
  • If all information is provided and still up-to-date, then just add a short comment that the issue is still relevant.

Thank you for your contribution!

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

No branches or pull requests

5 participants