-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Upload Date in StreamInfoItem with Error from PrettyTime Conversion in German #6050
Comments
I am submitting this issue here because I think this might be related to NewPipe itself because the issue really boils down to the used PrettyTime package. I tested the behaviour myself using only the package and strangely could not reproduce the errror so I am suspecting there could be a bug here. Maybe if somebody has some insight to what could be the problem I would love to fix this issue myself! |
My gut tells me this is a PrettyTime issue and not a NewPipe one. In each screenshot posted in this issue, there is a date with one day/month that is shown correctly (first item in both screenshots in fact). I suspect that it might be because it's more than one day but less than two for the other items or something like that? Edit: I noticed that this issue not applying to all dates was noted before. Silly mistake, but my point is the same. |
Also, I am unable to reproduce this issue now. Can you check if it's still present in the latest version (0.21.8)? |
I dug a little into PrettyTime's code, and it seems like my hunch was mostly right (I think something like rounding is happening internally here). I'll post what I found on the corresponding issue on PrettyTime's repo. |
The corresponding issue on PrettyTime's repo is ocpsoft/prettytime#218. |
Checklist
Steps to reproduce the bug
As the Image shows, if you open any View containing several StreamInfos, some of them have a bad PrettyTime conversion.
Expected behavior
In German (and other languages), the date is expressed with a certain gramatical case. So if the video was made 2 months ago, the date would have to be translated to "vor 2 Monaten". This suffix only appears with the plural of the word so "1 month ago" would translate to "vor 1 Monat" without the n.
Actual behaviour
The upload date is wrongly declinated in a way that it also receives the case suffix if the unit for the date is singular.
"1 month ago" -> "vor 1 Monatn"
This actually only happens with part of the dates as seen in the example picture.
Screenshots/Screen recordings
Logs
No Logs available
Device info
The text was updated successfully, but these errors were encountered: