search

immopoly / android

The Android client app
http://immopoly.appspot.com
Apache License 2.0
15 stars 4 forks source link

ü50 wohnungen #63

Closed mrschtief closed 12 years ago

mrschtief commented 12 years ago

Wenn 50 Wohnungen übernommen hat und versucht eine weitere zu übernehmen kommt eine wirre Fehlermeldung!

stefanhoth commented 12 years ago

Also die Fehlermeldung könnte im Text evtl. noch optimiert werden,aber prinzipiell sollen die Teilnehmer nicht mehr als 50 Wohnungen haben soll,um mehr Taktieren zu müssen.

mrschtief commented 12 years ago

ja es geht darum das wir bei der fehlermeldung abstrzen ;) und sie nicht anzeigen Am 17.03.2012 11:40 schrieb "Stefan Hoth" < reply@reply.github.com

:

Also die Fehlermeldung knnte im Text evtl. noch optimiert werden,aber prinzipiell sollen die Teilnehmer nicht mehr als 50 Wohnungen haben soll,um mehr Taktieren zu mssen.

Reply to this email directly or view it on GitHub: https://github.com/immopoly/android/issues/63#issuecomment-4553154

tresbjorn commented 12 years ago

Laut server code müsste kommen: "Du hast schon 50 Wohnungen in deinem Portfolio, du solltest es lieber optimieren!"

Deshalb nehme ich mal an die wirre Fehlermeldung ist die, die auch Doc Khan hatte: "Can't create handler inside thread that has not called Looper.prepare()"

Die ging (höchstwahrscheinlich) auf einen bug zurück, den ich vor 2 Wochen ein- und vorgestern wieder ausgebaut habe. Sorry nochmal.

Dabei hatte ich in der client version von ImmopolyException einen Toast, der mir ausgeben sollte welche Fehlermeldungen vom Server im Client noch übersetzt werden müssen. Leider hatte ich 1. nicht gecheckt, dass der Code anderswo in einem Task/Thread aufgerufen wird und 2. den Toast nicht wieder entfernt. Merke: auch Toast geht nur im UI Thread und ist kein gutes debugging tool.

Würds closen, wenn ich 50 wohnnungen hätte.

tresbjorn commented 12 years ago

unter einem tag! Weiter so ;)