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

fix getPreference function #527

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Conversation

peterstadler
Copy link
Member

the getPreference would only check the URL parameter and not the actual preferences for the key "application_language". This resulted in an always-English frontend independent of the preferences set in the prefs file.

The issue was reported by @obertsalome and can be reproduced with the current Pintos data by simply starting the Edirom without any URL parameter. This will show the frontend GUI elements in English although the "application_language" is set to "de".

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Improvement

Overview

  • I have performed a self-review of my code, according to the style guide
  • I have read the CONTRIBUTING document.
  • All new and existing tests passed.

the getPreference would only check the URL parameter and not the actual preferences for the key "application_language".
This resulted in an always-English frontend independent of the preferences set in the prefs file

Co-Authored-By: Daniel Röwenstrunk <[email protected]>
@peterstadler peterstadler added this to the 1.0.0 milestone Jan 14, 2025
@peterstadler peterstadler added Type: bugfix A pull request providing a bugfix Area: JS labels Jan 14, 2025
@krHERO krHERO self-requested a review January 15, 2025 09:01
Copy link
Member

@krHERO krHERO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested pinto data before and after the changes to the code. before: english, afterwards german.
can also confirm in the preferences.xml it is set to <entry key="application_language" value="de"/>

bug
Bildschirmfoto 2025-01-15 um 09 55 01

after fix from Peter
Bildschirmfoto 2025-01-15 um 09 59 02

@peterstadler peterstadler merged commit 69e0a36 into develop Jan 15, 2025
5 checks passed
@peterstadler peterstadler deleted the fix-preferencecontroller branch January 15, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: JS Type: bugfix A pull request providing a bugfix
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants