search

MyTFG / mytfg-app-android

Android App for MyTFG
GNU General Public License v3.0
4 stars 0 forks source link

Fixed new API #35

Closed lennart-bader closed 8 years ago

lennart-bader commented 8 years ago

Hier die Implementierung der neuen API für Nachrichtencenter und Terminal.

Es wurde jetzt jeweils die neue auf Referenzen angewiesene API umgesetzt.

Die ApiObjekte haben keine öffentlichen Konstruktoren mehr, alles geschieht über eine Factory in der jeweiligen Klasse. So können mehrfach Objekte vermieden werden und bestehende Objekte aktualisiert werden.

Auch ein kleiner Bugfix ist drin, da @Turakar einfach mal ein UI Element umbenannt hat, was zu einer schönen NPE führte :wink:

Turakar commented 8 years ago

Sorry für den Bug, der refactored manchmal ein bisschen viel, wenn man die Objekte kopiert.

Was die API im Moment nicht unterstützt, ist das Problem mit der Klasse von den Usern. Nach einem Jahr sind die Daten ja outdated.

lennart-bader commented 8 years ago

Oh ja, die ungenutzten Imports sind echt noch drin - sollte ich mal demnächst automatisch rausschmeißen lassen.

Wo ist das Problem mit den Usern?! Die werden durch die API ohnehin immer mitübertragen - diejenigen, die man braucht, sind ja nicht so viele. Wenn man dann irgendwas damit anstellen will, wird der neu geladen. Also outdated werden die nicht. Vorallem, da der Cache ja nicht die App überdauert.

lennart-bader commented 8 years ago

So hab die Imports, die nicht genutzt sind mal komplett entfernt. Waren doch ein paar mehr :smile: