AccountFragment: Fehlender Ladeindikator bei automatischem Refresh

opacapp / opacclient

[unmaintained] A Java client library and an Android app to access more than 1,000 public and scientific libraries from all over the world.

https://opacapp.net

MIT License

156 stars 61 forks source link

AccountFragment: Fehlender Ladeindikator bei automatischem Refresh #301

Open hurzl opened 9 years ago

hurzl commented 9 years ago

Wäre außerdem nett, wenn man die automatischen Updates überhaupt abschalten könnte.

raphaelm commented 9 years ago

Es werden beim Programmstart keine Daten geholt, außer den Feldern des Suchformulars und die sind ja wohl notwendig (und werden nur beim ersten Start und nach einem App-Update geladen).

Es werden allerdings im Hintergrund Daten, also möglicherweise auch beim Programmstart, regelmäßig Daten geholt, und zwar genau dann, wenn die "an abgelaufene Medien erinnern"-Funktion aktiviert ist, weil das dann für sinnvolle Erinnerungen notwendig ist. Es gibt bereits auch einen Knopf, um dies auf WLAN-Verbindungen einzuschränken.

hurzl commented 9 years ago

Ok aber stündlich? Kommt mir jedenfalls so vor... einmal am Tag reicht doch dafür?

Zur fehlenen Anzeige: ich starte das Programm, das reagiert dann erstmal nicht, wenn ich auf update drücke, und nach ein paar Sekunden steht unten "Die Daten sind aktuell".

raphaelm commented 9 years ago

Ah, jetzt verstehe ich was du meinst. Okay, wir reden von verschiedenen Dingen, ich wusste ja nicht, dass "Programm öffnen" bei dir "Kontoansicht aufrufen" heißt ;)

Die App sollte eigentlich stündlich kurz aufwachen, aber nur alle sechs Stunden im Hintergrund Daten abrufen, öfter nur, wenn der vorherige Abruf fehlgeschlagen ist (z.B. kein Internet)
Wenn man die Kontoansicht (also wirklich die, nicht einfach nur das Programm) öffnet, und die Daten älter als ich glaube 2 Stunden sind, wird automatisch dieses Konto aktualisiert. Das hat zwei Gründe: Erstens sind die meisten User an aktuellen Daten interessiert, zweitens kann es bei vielen Bibliothekssystemen katastrophal schiefgehen, wenn man in einer veralteten Liste "verlängern" oder "stornieren" drückt, da dann u.U. das falsche Medium verlängert wird. Natürlich sollte dann aber eine Ladegrafik angezeigt werden. Welche Android-Version verwendest du?

hurzl commented 9 years ago

5.0.2 Die Kontoansicht kann ich als Hauptansicht einstellen, das heißt nicht, dass ich dann jedesmal mit der Bib "telefonieren" möchte. Das mit den veralteten Listen sollte man anders lösen. Eine 2 h alte Liste kann auch veraltet sein ...

raphaelm commented 9 years ago

Und wie stellst du dir das vor, dass man das anders lösen kann? :)

Am 16. April 2015 10:34:52 MESZ, schrieb hurzl notifications@github.com:

5.0.2 Die Kontoansicht kann ich als Hauptansicht einstellen, das heißt nicht, dass ich dann jedesmal mit der Bib "telefonieren" möchte. Das mit den veralteten Listen sollte man anders lösen. Eine 2 h alte Liste kann auch veraltet sein ...

Reply to this email directly or view it on GitHub: https://github.com/raphaelm/opacclient/issues/301#issuecomment-93678305

hurzl commented 9 years ago

Ich weiß nicht, welche Daten beim Stornieren etc. übergeben werden, nur eine lfd. Nummer? Vielleicht kann man zu der Nummer erst Infos abrufen? Sonst muss man halt diese Liste/Seite direkt davor neu laden, wenn man sicher sein will. Ist blöd, aber wenn's nicht anders geht?

johan12345 commented 9 years ago

Das ist bei jedem System anders. Manchmal gibt es Medien-IDs, oft ist es aber auch die Position in der Liste (und die kann sich ändern). Im Moment prüfen die APIs nicht, dass das richtige Medium verlängert wird, und das Problem wird halt dadurch reduziert, dass die Seite neu geladen wird.

raphaelm commented 9 years ago

Den Fehler, dass der Lade-Indikator nicht korrekt angezeigt wird, werden wir auf jeden Fall versuchen zu beheben.
Das andere Problem ist bereits in #79 als Ticket vorhanden. Ich sehe das durchaus als sinnvolle Änderung an, aber da sich in den letzten drei Jahren noch niemand beschwert hat, auch nicht als übermäßig dringend.

raphaelm commented 8 years ago

Notiz: Dies tritt idR dann auf, wenn das Laden der Accountdaten sehr schnell fehlschlägt. In Folge könnte https://code.google.com/p/android/issues/detail?id=211948 eine Problemursache sein.