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

Fixes #2046 Fixes crash when data is loaded which has been written on… #2094

Merged
merged 1 commit into from
Feb 4, 2020
Merged

Conversation

marcnause
Copy link
Contributor

@marcnause marcnause commented Jan 28, 2020

… device with German locale

Fixes #2046

Changes: The locale has been changed from default to ROOT where numbers which are parsed later are formatted. The CSV logger has been changed slightly: whole lines are added now which allows to move line break logic to a new class (CSVDataLine) instead of having it in the classes which create the data.

Screenshot/s for the changes: No layout changes

Checklist:

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: app-fdroid-debug.zip

@marcnause marcnause marked this pull request as ready for review January 28, 2020 20:46
@CloudyPadmal CloudyPadmal merged commit ab3a44e into fossasia:development Feb 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App crashes on devices with German locale when previously recorded data is loaded
3 participants