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

Bug: Duplication of ingridients ending with Öl #314

Closed
LDaxin opened this issue Dec 4, 2023 · 4 comments · Fixed by #316
Closed

Bug: Duplication of ingridients ending with Öl #314

LDaxin opened this issue Dec 4, 2023 · 4 comments · Fixed by #316
Labels
bug Something isn't working

Comments

@LDaxin
Copy link

LDaxin commented Dec 4, 2023

Is this urgent?

No

What parts are affected

Backend

What is the server version

V91

What is the client version

v0.4.19

What platform are you using

Android, Web

What's the problem 🤔

If a ingridient with the letter Öl at the end gets selectet a duplicate gets created. This is just with the Öl at the end with Öln, Ön, öl or testÖL at the end it's not happening just with the upper case Letter Ö and the lower case Letter l. The code of the Letters in the JSON Export is \u00D6l.

2023-12-04-223818_crop

This hapens in Android and in the Web version.

I hope this is helpfull

Share your logs

No response

Share your configuration

No response

@LDaxin LDaxin added the bug Something isn't working label Dec 4, 2023
@LDaxin
Copy link
Author

LDaxin commented Dec 5, 2023

the Renaming of the ingredients makes no difference. if the ingredient was originally created with Öl at the end it will Duplikate even wenn renamed.

@LDaxin
Copy link
Author

LDaxin commented Dec 5, 2023

okay sorry for correcting me again I have played around a little more and it seems that the problem is with the uppercase Ö and the l is not so important. what I mean I can recreate the duplication with adding the ingredient from the recently used with just Ö in the ingredient name and it is not important if there is a another letter

@TomBursch
Copy link
Owner

okay sorry for correcting me again

No worries, thank you for testing this so much!

@LDaxin
Copy link
Author

LDaxin commented Dec 6, 2023

I tested it a little bit further the uppercase Ä and Ü are although affected.

@TomBursch TomBursch moved this from Next Up to In Progress in KitchenOwl Project Dec 6, 2023
@TomBursch TomBursch linked a pull request Dec 6, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from In Progress to Ready for Testing in KitchenOwl Project Dec 6, 2023
@TomBursch TomBursch moved this from Ready for Testing to Testing in KitchenOwl Project Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Testing
Development

Successfully merging a pull request may close this issue.

2 participants