immopoly / android

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

FC bei falscher URL in badge #49

Closed mrschtief closed 12 years ago

mrschtief commented 12 years ago

01-15 13:42:12.568: E/AndroidRuntime(4192): Caused by: java.lang.IllegalArgumentException: Illegal character in query at index 65:

01-15 13:42:12.568: E/AndroidRuntime(4192): at java.net.URI.create(URI.java:727) 01-15 13:42:12.568: E/AndroidRuntime(4192): at org.apache.http.client.methods.HttpGet.(HttpGet.java:75) 01-15 13:42:12.568: E/AndroidRuntime(4192): at org.immopoly.android.helper.ImageListDownloader$BitmapDownloaderTask.doInBackground(ImageListDownloader.java:285) 01-15 13:42:12.568: E/AndroidRuntime(4192): at org.immopoly.android.helper.ImageListDownloader$BitmapDownloaderTask.doInBackground(ImageListDownloader.java:1)

http://chart.googleapis.com/chart?chst=d_text_outline&chld=FFCC33|26|h|FF0000|b|111

Fehler is ja ok, aber kein Force Close

01-15 13:42:12.568: E/AndroidRuntime(4192): java.lang.RuntimeException: An error occured while executing doInBackground()

mrschtief commented 12 years ago

missing urlencoding in imagelistdownloader

URLEncoder.encode(myurltoencode,"UTF-8");

tresbjorn commented 12 years ago

wollts gerade einbauen, aber URLEncoder.encode() ist nur für einzelne Parameter gedacht. Wenn man da eine ganze URL reinsteckt, geht alles kaputt.

Also entweder Speziallösung oder jene Instanz, die das hier erzeugt: "chld=FFCC33|26|h|FF0000|b|111" endoded das schon, bevor's ins json kommt.

PS: wird nicht mehr gebraucht, oder? PPS: der komische ImageListDownloader ist 'eh doof und sein caching funktioniert irgendwie nur manchmal...

mrschtief commented 12 years ago

ok ich merks mir, wenn ich urls ausliefere dann nur urlencoded