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

Fehler beim Parsen eines Datum Attributes #488

Closed
hjbflyer opened this issue Apr 6, 2016 · 8 comments
Closed

Fehler beim Parsen eines Datum Attributes #488

hjbflyer opened this issue Apr 6, 2016 · 8 comments

Comments

@hjbflyer
Copy link
Contributor

@hjbflyer hjbflyer commented Apr 6, 2016

Ich habe heute ein Datums-Attribut zu meinen Ansichten hinzugefügt. Die Eingabe wurde nur im ISO-Format akzeptiert. Die Anzeige war ebenfalls in diesem Format.

Ich habe als Änderungsvorschlag 2 Patches beigefügt.

@hjbflyer
Copy link
Contributor Author

@hjbflyer hjbflyer commented Apr 6, 2016

@buchen
Copy link
Owner

@buchen buchen commented Apr 6, 2016

Bist Du auf dem aktuellen HEAD?
Mit dd4dc31 habe ich eigentlich das
Datumsformat von ISO auf ein localized medium format (also für Deutschland
dd.MM.yyyy) umgestellt.

@buchen
Copy link
Owner

@buchen buchen commented Apr 6, 2016

Ah, verstanden, es geht um frei definierbare Attribute.
Danke. Nehme ich mit auf.

@buchen buchen added this to the 2016 Q1 Short-List milestone Apr 6, 2016
buchen added a commit that referenced this issue Apr 9, 2016
@buchen
Copy link
Owner

@buchen buchen commented Apr 9, 2016

Danke für den Patch.

Ich habe den LocalDateConverter nicht angefasst weil der für die Konvertierung von Java -> XML verwendet wird. Und im XML soll weiterhin das ISO Format verwendet werden.

Die Idee mit den mehreren DateTimeFormatter patterns ist super! Die habe ich auch für das Editieren eines Datums in einer Spalte übernommen.

@buchen buchen closed this Apr 9, 2016
@hjbflyer
Copy link
Contributor Author

@hjbflyer hjbflyer commented Apr 10, 2016

Schade dass Du meine Änderungen am LocalDateConverter nicht übernommen hast. Als Ergebnis kann ich meine Portfolios nicht mehr öffnen, da sie jetzt lauter DateTimeParseExceptions werfen.

@buchen
Copy link
Owner

@buchen buchen commented Apr 10, 2016

Schade dass Du meine Änderungen am LocalDateConverter nicht übernommen hast. Als Ergebnis kann ich meine Portfolios nicht mehr öffnen, da sie jetzt lauter DateTimeParseExceptions werfen.

Ah, Du hattest Deine Datei schon einmal gespeichert. Ändere doch die #fromString Methode temporär um die Daten einmal einzulesen. Und dann korrekt wieder mit der #toString Methode zu schreiben.

@buchen
Copy link
Owner

@buchen buchen commented Apr 10, 2016

Im XML Format möchte ich nur das Datum nur in ISO Format stehen haben. Ansonsten würde es von der Locale abhängen ob die Datei eingelesen werden können oder nicht.

@hjbflyer
Copy link
Contributor Author

@hjbflyer hjbflyer commented Apr 10, 2016

Verstehe, ich werde fromString kurzzeitig anpassen.

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