railwaystations/rsandroidapp

Bahnhofsfoto-App for Android

Railway-Stations Android App

Android-App for the Project Railway-Stations.org ("Deutschlands-Bahnhoefe")

Get nightly builds from https://fdroid.storchp.de/fdroid/repo

The App loads Data from api.railway-stations.org, which contains railway-stations of some countries. The RSAPI is a java-application on https://github.com/RailwayStations/RSAPI.

The project - website you will find on https://railway-stations.org and http://www.deutschlands-bahnhoefe.de.

The code is public under MIT-Licence: http://choosealicense.com/licenses/mit/

Help to translate the project via Crowdin: https://crowdin.com/project/railwaystationsandroidapp

rsandroidapp's Issues

Back Stack nicht korrekt nach Fotografieren

Starte App
Wechsel in "In der Nähe"
Wähle einen roten Marker aus
Klicke drauf und Fotografiere
Klicke OK
Klicke auf "Back"
Ist: die Bahnhofsliste geht auf.
Soll: Karte "In der Nähe"
issue_backstack.zip

Umstellung auf Station "idStr" Attribut

Die numerische Station "id" wurde in der API ersetzt durch "idStr".

Namen der polnischen Bahnhöfe zu lang

Die Namen der polnischen Bahnhöfe sind zu lang. Die sollten umbrechen.

Abfrage ob "Meine Daten" ausgefüllt sind

Abfrage vor dem Fotografieren, ob "Meine Daten" ausgefüllt sind, damit die Dateinamen sinnvoll vergeben werden können.

Dialogbox bei Uploadfehler

Der Toast verschwindet zu schnell.

Anleitung bzgl. Chatnotification anpassen

ist nicht mehr aktuell

Back Stack nicht korrekt nach Chat

Öffnet man den Chat über die Notification beendet sich die App, wenn man auf zurück geht.
Hier sollte sich die Hauptseite öffnen.

Chat Timestamp beim Senden setzen

Zur Zeit wird der Chat Timestamp in der Auth activity in onCreate() gesetzt. Der sollte aber gesetzt werden, wenn eine Nachricht gesendet wird. Im Moment haben mehrere Nachrichten eines Users den selben Timestamp.

Push-Notifications für den Chat

Anleitung bzgl. Bahnhofsortung an das letzte Update anpassen

ich vermute, das muss seit dem letzten Update angepasst werden.

Foto über Bahnhofsortung gibt kein Landscape vor

Wenn man über die Bahnhofsortung-Notification auf Foto klickt und dann ein Foto aufnimmt, kann man dies im Portraitmodus tun. Es wird kein Landscape forciert, wie wenn man es direkt über die App macht.

Vielleicht würde es helfen erstmal die DetailsActivity zu öffnen und nicht gleich die Kamera. Das ändert aber den Workflow.

Filter für die Bahnhofsortung (mit oder ohne Foto)

Filter für die Bahnhofsortung einbauen, damit man auswählen kann,
ob man alle Bahnhöfe angezeigt bekommt per Push Notification oder nur die,
die noch kein Foto haben

Fix project setup

At the moment Android Studio isn't able to properly import the project from the gradle files.

@androidoma I'm on it and send a Pull Request soon ;)

Zoom Buttons für die Karte

Auf der Karte wäre es schön, die Buttons zum Zoomen zu haben.

Dies sollte mit folgender Zeile möglich sein:

map.getUiSettings().setZoomControlsEnabled(true);

Einbindung Fahrstühle

Die Fahrstuhldaten sind in ElasticSearch und können von dort abgegriffen werden. Datensatz der Deutschen Bahn.
Benötigt wird:

1. Kennzeichen im Ländersatz, ob Fahrstuhldaten vorhanden sind
2. Import der deutschen Fahrstuhldaten
3. Unter Einstellungen sollen die Maße von Fahrrad und/oder Rollstuhl eingetragen werden können
4. Ausgabe der Fahrstuhldaten zu dem aktuellen Bahnhof (DetailsActivity) als extra Menüpunkt in Tabelle
5. Herunterladen der Fahrstuhldaten mit den Länderdaten, damit offline verfügbar
6. Online Fahrstuhlstatus aus FaSta abfragen
7. Berechnung, ob das Fahrrad / der Rollstuhl in den Aufzug passt anhand Kabinentiefe und Kabinentür und der Daten aus den Einstellungen - siehe Punkt 1.

Netzlosigkeit darf nicht zum Löschen der DB führen

Habe ich kein Netz und klicke ich auf "Bahnhofsdaten aktualisieren" führt dies zu einem Löschen der Bahnhofsliste.
Das führt dazu, dass die App nicht mehr verwendbar wird, solange ich kein Netz habe.

Dabei könnte man während der Netzlosigkeit durchaus Fotos aufgrund einer veralteten Datenbank schießen und auch später verschicken.

Ich für meinen Teil schätze das als extrem wichtig ein, denn ohne Bahnhofsliste ist die App nicht verwendbar.

Fotografieren ohne Nickname möglich

Installation der App
Starten der App
Alle Anleitungsbilder durchklickern
Bahnhof über die Liste aufrufen
Versuch Foto zu schiessen
Meldung, dass erst Daten gepflegt werden müssen mit ok bestätigen
==> Einstellungen
nur CC0 anklicken (Rest frei lassen) und speichern
zurück und Foto schießen
==> Es ist möglich ein Foto zu schießen ohne Nickname

issue_fotografierenohnenickname.zip

Absturz beim Drehen bei "In der Nähe"

Hochformat
Starte die App.
Wechsel zu "In der Nähe"
Drehe ins Querformat
Absturz
issue_absturz.zip

Änderung der Icons

Mit Hilfe von @tursics

Trim der Benutzereingaben in "Meine Daten"

Link zum Profil sollte bei Auswahl von "Nein" bei der Verlinkung optional sein.

Anonyme Registrierung: möchte der User anonym sein, so wählt er bei der Verlinkung unter "Meine Daten" einfach "Nein" aus. Zur Zeit wird dann beim anfordern des Upload Tokens trotzdem eine URL verlangt.

Handle disabled location services

Some users have disabled their location services (GPS off). The MapsActivity needs to handle this situation. At the moment it moves the map to the Zero Coordinates somewhere in the ozean.

The MapsActivity should query the location settings like described here: https://developer.android.com/training/location/change-location-settings.html.
A fallback could calculate the center of all the map markers. We cannot hardcode a single position (as before) because we support several countries.

Document Firebase setup in README

Steps to copy the https://github.com/RailwayStations/RSAndroidApp/blob/master/app/src/main/java/de/bahnhoefe/deutschlands/bahnhofsfotos/util/FireBaseConstantsTemplate.java class is missing in README

Translation into Spanish

The App has the possiblity to select different countries for railway stations. To make it better to use for local users we need a translation into Spanish. The translation is necessary for this file

Anzeigemöglichkeit der Bahnhöfe mit Foto

Im Moment werden nur die Bahnhöfe ohne Foto aufgelistet und auf der Karte angezeigt.
Der Wunsch ist auch optional die Bahnhöfe mit Foto zu sehen.

Fehler beim Upload und Registrierung als Dialog mit Bestätigung

Die Toasts werden schnell übersehen. Die Fehlermeldungen beim Upload und der Registrierung sollten in eine Dialog mit Bestätigung geändert werden.

Lizenzen unter Fotos hinzufügen

Die Fotos brauchen noch die Lizenz mit Link zum Lizenztext.

PhotoUpload antwortet mit HTTP 409, falls der Bahnhof schon ein Foto hat

Die API antwortet beim photoUpload mit HTTP status code 409 (Conflict) falls der Bahnhof schon ein Foto hat oder es sich bereits ein Foto für diesen Bahnhof in der Inbox zum Review befindet.
Die App sollte dem User einen entsprechenden Hinweis geben, dass bereits ein Foto existiert.
Der Upload erfolgt trotzdem, es könnte ja sein, dass nur ein besseres Foto hochgeladen wurde.

Ignore own chat message

If I send a chat message I get notified of my own message, that should be ignored.

Links in Chat message clickable

Links in Chat message should be clickable and open the browser.

Race Condition in MapsActivity

Problematische Geräte
LG G3 (g3) - Android 4.4

Ausnahmen
java.lang.RuntimeException: Unable to pause activity {de.bahnhoefe.deutschlands.bahnhofsfotos/de.bahnhoefe.deutschlands.bahnhofsfotos.MapsActivity}: java.lang.IllegalStateException: GoogleApiClient is not connected yet.

Klingt so, als könnte man dasTracking pausieren, bevor es richtig initialisiert ist!

FATAL EXCEPTION: main Process: de.bahnhoefe.deutschlands.bahnhofsfotos, PID: 10266 java.lang.RuntimeException: Unable to pause activity {de.bahnhoefe.deutschlands.bahnhofsfotos/de.bahnhoefe.deutschlands.bahnhofsfotos.MapsActivity}: java.lang.IllegalStateException: GoogleApiClient is not connected yet. at android.app.ActivityThread.performPauseActivity(ActivityThread.java:3079) at android.app.ActivityThread.performPauseActivity(ActivityThread.java:3034) at android.app.ActivityThread.handleRelaunchActivity(ActivityThread.java:3729) at android.app.ActivityThread.access$900(ActivityThread.java:142) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1209) at android.os.Handler.dispatchMessage(Handler.java:102) at android.os.Looper.loop(Looper.java:136) at android.app.ActivityThread.main(ActivityThread.java:5118) at java.lang.reflect.Method.invokeNative(Native Method) at java.lang.reflect.Method.invoke(Method.java:515) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:794) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:610) at dalvik.system.NativeStart.main(Native Method) Caused by: java.lang.IllegalStateException: GoogleApiClient is not connected yet. at com.google.android.gms.internal.zzpw.zzd(Unknown Source) at com.google.android.gms.internal.zzqa.zzd(Unknown Source) at com.google.android.gms.internal.zzpy.zzd(Unknown Source) at com.google.android.gms.location.internal.zzd.removeLocationUpdates(Unknown Source) at de.bahnhoefe.deutschlands.bahnhofsfotos.MapsActivity.stopLocationUpdates(MapsActivity.java:381) at de.bahnhoefe.deutschlands.bahnhofsfotos.MapsActivity.onPause(MapsActivity.java:291) at android.app.Activity.performPause(Activity.java:5379) at android.app.Instrumentation.callActivityOnPause(Instrumentation.java:1233) at android.app.ActivityThread.performPauseActivity(ActivityThread.java:3065) ... 12 more

Translation into English

The App has the possiblity to select different countries for railway stations. To make it better to use for local users we need a translation into English. The translation is necessary for this file

Foto Upload

Automatischer Foto upload über die App.

URL-Templates in Länderdaten aufnehmen

Die URL-Templates für Abfahrttafeln sollten in die Länderdaten unter https://railway-stations.org/laenderdaten.json aufgenommen werden, damit die App diese Infos dynamisch mit den Länderdaten auslesen kann. Anschließend sollen die hart kodierten URLs aus der App entfernt werden, entsprechend der todo-Annotationen.

Translation into French

The App has the possiblity to select different countries for railway stations. To make it better to use for local users we need a translation into French. The translation is necessary for this file

Klick auf Bahnhofsortung Notification sollte App öffnen

Die Notification, dass die Bahnhofsortung aktiv ist, ist z.Zt.ohne weitere Funktion. Ein Klick darauf könnte die App öffnen, wo man die Bahnhofsortung ausschalten kann.

NearbyNotificationService.java uses unchecked or unsafe operations

Sehe diese Warnung im CI build:

Note: /home/travis/build/RailwayStations/RSAndroidApp/app/src/main/java/de/bahnhoefe/deutschlands/bahnhofsfotos/NearbyNotificationService.java uses unchecked or unsafe operations.

Translation into Polish

The App has the possiblity to select different countries for railway stations. To make it better to use for local users we need a translation into Polish. The translation is necessary for this file

Back Stack nicht korrekt nach Antippen einer Bahnhofsortungs-Notification

To share a photo receiving App needs filestorage permission

When sharing a photo (either to send a mail or elsewhere) the recieving App needs permission to access the (external) filestorage. If the user hasn't granted that, the receiving app can't read the photo (results in an empty mail).

Android has a solution for that: Content/FileProvider. I'll implement it and send a PullRequest.

Translation into Finnish

The App has the possiblity to select different countries for railway stations. To make it better to use for local users we need a translation in Finnish. The translation is necessary for this file

Upload von Fotos aus dem System

Neben Fotos aus der App direkt über die Kamera aufzunehmen, wäre es auch praktisch bestehende Fotos vom System auszuwählen und als Upload anzubieten.

prominenterer Hinweis, wenn das mobile Endgerät im Hochformat ist (Bild knipsern)

NPE during station list update

From crash report in Google Play:
java.lang.RuntimeException: An error occured while executing doInBackground() at android.os.AsyncTask$3.done(AsyncTask.java:300) at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:355) at java.util.concurrent.FutureTask.setException(FutureTask.java:222) at java.util.concurrent.FutureTask.run(FutureTask.java:242) at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587) at java.lang.Thread.run(Thread.java:841) Caused by: java.lang.NullPointerException at de.bahnhoefe.deutschlands.bahnhofsfotos.MainActivity$JSONTask.doInBackground(MainActivity.java:434) at de.bahnhoefe.deutschlands.bahnhofsfotos.MainActivity$JSONTask.doInBackground(MainActivity.java:395) at android.os.AsyncTask$2.call(AsyncTask.java:288) at java.util.concurrent.FutureTask.run(FutureTask.java:237) ... 4 more

Introslider

1. Wofür ist die App überhaupt da
2. Was ist "In der Nähe"
3. Was sind "alle fehlenden Bahnhöfe"
4. Hochladen nicht möglich, da Qualitätskontrolle notwendig - Versand per Mail
5. Teilen-Button erklären
6. Hinweis "Meine Daten ausfüllen" (unbedingt als Letztes, da es in Erinnerung bleiben soll)

Abfahrtstafeln auch in DetailsActivity

Im Moment werden die Abfahrtstafeln nur in der Bahnhofsortung angezeigt, wenn man in der Nähe eines Bahnhofes befindet.

Umstellung der Karte auf OSM

Add link to the local railway apps into the detailsview

There could be a link to the local railway app of the current country into the google playstore.

NullPointerException in DetailsActivity.onActivityResult()

Reported von Google Developer Console:

Caused by: java.lang.NullPointerException: 
  at de.bahnhoefe.deutschlands.bahnhofsfotos.DetailsActivity.onActivityResult(DetailsActivity.java:312)

Menüpunkt "Eigene Fotos' ersetzen durch Liste von Bahn-Apps des jeweiligen Landes

Wenn RailwayStations/RSWebsite#21 implementiert ist, könnten die Fotografen ihre Fotos kontrollieren und wir könnten den Menüpunkt "Eigene Fotos' rauswerfen.
Stattdessen dann eine Liste mit den lokalen Bahn-Apps des jeweiligen Landes (unabhängig von der Länderauswahl).