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

preferences: external file types duplicates #10271

Closed
2 tasks done
ghost opened this issue Sep 1, 2023 · 10 comments · Fixed by #10496
Closed
2 tasks done

preferences: external file types duplicates #10271

ghost opened this issue Sep 1, 2023 · 10 comments · Fixed by #10496
Assignees
Labels
bug Confirmed bugs or reports that are very likely to be bugs preferences

Comments

@ghost
Copy link

ghost commented Sep 1, 2023

JabRef version

5.9 (latest release)

Operating system

Windows

Details on version and operating system

Windows 10 22H2

Checked with the latest development build

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

  1. Go to preferences
  2. Go to external file types
  3. I'm using Dutch translation and when using that translation there are duplicate file types (one written in English, one in Dutch). When the language of jabref is set to English, the same duplicate files types (in Dutch even while English is selected!) stay in the list

afbeelding

afbeelding

afbeelding

afbeelding

Appendix

...

Log File
Paste an excerpt of your log file here
@ghost
Copy link
Author

ghost commented Sep 1, 2023

I just saw clicking "reset to default" removes the duplicates.
But I still like to know how they could be inside the list, because I never added any file type there.
Maybe it's a small bug?

@Siedlerchr
Copy link
Member

Might be related to #10176
But I wonder why it's using the language

@ghost
Copy link
Author

ghost commented Sep 1, 2023

Might be related to #10176 But I wonder why it's using the language

Yes, in either case, when jabref is set to English, it would be impossible to see Dutch translation anywhere.

Update:

  1. Clicking "reset to default" removes the duplicates
  2. Switch back to English/Dutch language (depending in what language you were) and the duplicates (in Dutch) are BACK!
  3. Clicking "reset to default" removes the duplicates AGAIN but another language change adds them again ...

@Siedlerchr Siedlerchr added bug Confirmed bugs or reports that are very likely to be bugs preferences labels Sep 1, 2023
@ThiloteE
Copy link
Member

ThiloteE commented Sep 3, 2023

Low priority, as while this is confusing and not visually pleasing, i expect, it does not break any workflow or has any other negative effects.

@ghost
Copy link
Author

ghost commented Sep 6, 2023

Low priority, as while this is confusing and not visually pleasing, i expect, it does not break any workflow or has any other negative effects.

Is it better not to report bugs like this that don't break anything?
Or is it OK to report them too?

@ThiloteE
Copy link
Member

ThiloteE commented Sep 6, 2023

There is nothing wrong with reporting those. How could we ever have new features or enhancements otherwise? ;-) Some of these isssues could be a "good first issue" for somebody or somebody with the willingness and knowledge how to fix them might do it. SInce JabRef is open source, we never know if, or when somebody opens a pull-request. The beauty of open source is that everybody can come and propose a fix :-) Often people have their own particular itch they want scratched, so there are not too many that fix issues for others, but they do exist.

Unfortunately, if you look at the issue tracker, you will see that it grows ever larger. Developers simply do not have the time to fix them all.

@papatekken
Copy link
Contributor

papatekken commented Sep 16, 2023

Currently when the program starts, it loaded list of external file types from saved preferences, and compare with list of default file types in the application. If the name of default file types is not found the saved file type list, that will added to the preferences file type list.

Problem occurs when langauage changes, as the default file type names will be different. (e.g. GIF image vs GIF afbeelding)

Please assign this ticket to me, I can help to change the comparsion by name to comparsion by file extension(.GIF vs .GIF).
Then problem should can be solved.

@Siedlerchr
Copy link
Member

thanks for looking into this! @papatekken That seems a likely explanation

@papatekken
Copy link
Contributor

papatekken commented Sep 24, 2023

In current codebase, I found that duplicate extensions are allowed.

Can I assume external file types list should contains unique extension only? If the answer is yes, I will modify corresponding codes as well.

@Siedlerchr
Copy link
Member

@papatekken Sorry for the delayed response, yes, external file types should be unique extensions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs preferences
Projects
Archived in project
3 participants