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

Datumseingabe beim Kauf/Einzahlung falsch #301

Closed
mgehre opened this issue Jun 2, 2015 · 9 comments
Closed

Datumseingabe beim Kauf/Einzahlung falsch #301

mgehre opened this issue Jun 2, 2015 · 9 comments
Labels
bug

Comments

@mgehre
Copy link

@mgehre mgehre commented Jun 2, 2015

Hallo,

wenn ich Wertpapier kaufe, und das Datumsfeld in diesem Dialog auf z.B.
1/18/2015 setzte,
dann steht danach in der Transaktionsliste die Transaktion mit dem Datum 2015-06-18. Der Monat ist falsch!

Dieses Problem ist reproduzierbar.

@mgehre
Copy link
Author

@mgehre mgehre commented Jun 3, 2015

Jetzt hatte ich das Problem mit 1/ 8/2014. Er machte daraus 2014-06-03 (den heutigen Tag im richtigen Jahr).

@mgehre
Copy link
Author

@mgehre mgehre commented Jun 3, 2015

Das Problem scheint nicht aufzutreten, wenn ich das Datum mit Pfeil-aufwärts und Pfeil-abwärts einstelle.

@buchen
Copy link
Owner

@buchen buchen commented Jun 3, 2015

Du meinst schon diesen Kaufdialog, oder? Auf welcher Plattform bist Du unterwegs? Auf meinem Mac kann ich da so ein Datum gar nicht eingeben. Einen "/" akzeptiert er nicht als Eingabe.

bildschirmfoto 2015-06-03 um 22 22 02

@mgehre
Copy link
Author

@mgehre mgehre commented Jun 3, 2015

screen
Bei dir kann man das Datum im deutschen Format eingeben, bei mir ist es komischerweise MM-DD-YYYY.
Ich habe die letzte git revision und laufe auf Linux/Ubuntu.

@buchen
Copy link
Owner

@buchen buchen commented Jun 3, 2015

Das ist ein "natives" SWT Widget, d.h. es sollte sich die Plattform-Einstellungen ziehen. Aber eine Eingabe von "1/18/2015" entspricht dem Format. Meint er plötzlich, es ist der 18. Monat?!?

@buchen
Copy link
Owner

@buchen buchen commented Jun 3, 2015

Das Attribute wird hier an das Widget gebunden. Die Konvertierung macht meinem Verständnis nach komplett das DateTime Module. Hier findet das Plumbing statt: https://github.com/buchen/portfolio/blob/master/name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/util/SimpleDateTimeSelectionProperty.java

@buchen
Copy link
Owner

@buchen buchen commented Jun 3, 2015

DateTime does not support localization
https://bugs.eclipse.org/bugs/show_bug.cgi?id=248075

DateTime.getSpokenText() needs to be locale-sensitive
https://bugs.eclipse.org/bugs/show_bug.cgi?id=339725

SWT DateTime should support customized locale
https://bugs.eclipse.org/bugs/show_bug.cgi?id=359753

Alternative:
Nebula CDateTime Widget

@buchen buchen removed the quick_win label Jun 3, 2015
@buchen buchen added this to the 2015 Q2 Release milestone Jun 3, 2015
@mgehre
Copy link
Author

@mgehre mgehre commented Jun 3, 2015

Danke, dass du dich darum kümmerst 👍

buchen added a commit that referenced this issue Jun 4, 2015
On Linux, DateTime does not work very well because of the missing
support for localization. On Mac OS X, CDateTime runs sluggish and does
not render the drop down box correctly. And on Windows, I prefer the
DateTime widget. Because therefore the DateTimePicker will choose
between CDateTime and DateTime depending on the platform.

Issue: #301
@buchen
Copy link
Owner

@buchen buchen commented Jun 4, 2015

CDateTime unter Linux funktioniert besser (zumindest bei meinem kurzen Test).
Ist in der nächsten Version drin.

@buchen buchen closed this Jun 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.