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: Programm will Speichern obwohl nichts geändert wurde #144

Closed
Marfir opened this issue Apr 26, 2014 · 8 comments
Closed

Bug: Programm will Speichern obwohl nichts geändert wurde #144

Marfir opened this issue Apr 26, 2014 · 8 comments

Comments

@Marfir
Copy link

@Marfir Marfir commented Apr 26, 2014

Kleiner Bug: Immer wenn ich das Tool öffne wird mir mein Portfolio-xml sofort als geändert gekennzeichnet. Es kann sich aber nichts geändert haben, da ich noch nichts gemacht habe. Auch sind alle Kurse aktuell.

Speichern sollte nur notwendig sein, wenn sich wirklich etwas geändert hat.

@buchen buchen added bug and removed bug labels May 7, 2014
@buchen
Copy link
Owner

@buchen buchen commented May 7, 2014

Danke für die Rückmeldung. Wenn ich Kurse abfrage, dann setze ich das Dokument auf "verändert" - egal ob keine Kurse vorliegen, keine Verbindung zum Internet besteht, oder sich gar nichts geändert hat. Das sollte ich bei Gelegenheit mal ändern :-)

@tamueller
Copy link

@tamueller tamueller commented Sep 20, 2014

Hallo,
ich weiß nicht wie genau du es ändern willst, ich finde, Kursupdates sollten überhaupt nicht zu "verändert" führen, also auch wenn neue Kurse runtergeladen wurden.

@Marfir
Copy link
Author

@Marfir Marfir commented Sep 21, 2014

Soweit ich gesehen habe werden die Kurse in der XML gespeichert. Für den Fall dass Kurse herunter geladen wurden, muss das Dokument auch gespeichert werden. Man könnte höchstens das Speichern durch Kursenupdates beim Starten automatisch geschehen lassen.

@tamueller
Copy link

@tamueller tamueller commented Sep 22, 2014

@marfik Ja, die Kurse werden im XML gespeichert. Wenn man auf Speichern drückt, sollte das auch passieren, aber der (Stern) der anzeigt, dass etwas geändert wurde, was gespeichert werden muss, sollte nicht kommen.
Der sollte denke ich nur kommen, wenn man etwas geändert hat. Nicht durch automatische backgroundtasks. Wenn ich die Kursdaten nicht speichere, lade ich sie halt beim nächsten mal wieder runter.

@buchen
Copy link
Owner

@buchen buchen commented Sep 24, 2014

Ich denke wir sind uns einig, dass der "Änderungsindikator (Stern)" nur gezeigt werden sollte, wenn sich tatsächlich etwas geändert hat. Also eine Kursabfrage, die keine Kurse ändert, sollte auch den Indikator auch nicht setzen.

Die zweite Frage ist, ob der Indikator nur gesetzt werden sollte, wenn der Benutzer aktiv eine Änderung vorgenommen hat (Vorschlag von @Marfir).

Ehrlich gesagt tendiere ich eher dazu, den Indikator bei jeder Änderung zu zeigen - auch wenn die Datei im Hintergrund geändert wurde. Zum Beispiel ist die Extraktion von Kursen aus HTML Seiten eher instabil. Manchmal werden nur die letzten 20 Kurse angezeigt. Dann muss man mindestens alle 20 Tage mal auf "Speichern" drucken.

@Marfir
Copy link
Author

@Marfir Marfir commented Sep 24, 2014

@ALL

Ich finde das Kurse Speichern (und entfernen uralter Kurse) könnte automatisch passieren. Bei einigen anderen Anwendungen gibt es auch ein auto-save. Aber das ist eine Verbesserung auf die ich auch verzichten könnte.

Hauptsache der mini-Bug ist weg ;-)
Danke!

@buchen buchen added this to the 2014 Oktober Release milestone Sep 24, 2014
buchen added a commit that referenced this issue Nov 8, 2014
@buchen
Copy link
Owner

@buchen buchen commented Nov 8, 2014

Ich habe jetzt mal folgende Änderung vorgenommen: wenn sich keine Kursinformationen (oder Verbraucherpreise) ändern, dann wird die Datei auch nicht als geändert markiert.

Aber die Kurse werden weiterhin in der XML Datei gespeichert und neue Kurse werden auch die Datei als geändert markieren.

@buchen buchen closed this Nov 8, 2014
@Marfir
Copy link
Author

@Marfir Marfir commented Nov 12, 2014

Super danke! :-)

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
3 participants
You can’t perform that action at this time.