[Mobile] Added FOREGROUND_SERVICE permission, required by newer Andro…

…id APIs. [Mobile] Added Apache HTTP legacy library hack. [Mobile] Fixed intermittent crash issue due to null Activity. [Trivial] Bumped version numbers for release v0.126.
thecosmicfrog · Jan 7, 2019 · d2fd932 · d2fd932
1 parent 66c1521
commit d2fd932
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 21 deletions.
diff --git a/mobile/build.gradle b/mobile/build.gradle
@@ -8,8 +8,8 @@ android {
         applicationId "org.thecosmicfrog.luasataglance"
         minSdkVersion 21
         targetSdkVersion 28
-        versionCode 121
+        versionCode 126
-        versionName "0.121"
+        versionName "0.126"
     }
     buildTypes {
         release {

diff --git a/mobile/src/main/AndroidManifest.xml b/mobile/src/main/AndroidManifest.xml
@@ -4,6 +4,7 @@
     package="org.thecosmicfrog.luasataglance">
 
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
+    <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
     <uses-permission android:name="android.permission.VIBRATE" />
@@ -33,6 +34,14 @@
             android:name="com.google.firebase.messaging.default_notification_color"
             android:resource="@color/luas_purple" />
 
+        <!--
+            Official Google hack required since targeting API level >= 28 (Android 9.0).
+            https://developers.google.com/maps/documentation/android-sdk/config#specify_requirement_for_apache_http_legacy_library
+        -->
+        <uses-library
+            android:name="org.apache.http.legacy"
+            android:required="false" />
+
         <activity
             android:name=".activity.MainActivity"
             android:label="@string/app_name">

diff --git a/mobile/src/main/java/org/thecosmicfrog/luasataglance/activity/LineFragment.java b/mobile/src/main/java/org/thecosmicfrog/luasataglance/activity/LineFragment.java
@@ -394,8 +394,8 @@ public void onItemSelected(AdapterView<?> parent, View view, int position,
                          */
                         if (isVisibleToUser) {
                             /*
-                             * If the Spinner's selected item is "Select a stop...", we don't need to
+                             * If the Spinner's selected item is "Select a stop...", we don't need
-                             * do anything. Just clear the stop forecast and get out of here.
+                             * to do anything. Just clear the stop forecast and get out of here.
                              */
                             if (position == 0) {
                                 shouldAutoReload = false;
@@ -775,7 +775,7 @@ public void run() {
                             if (shouldAutoReload) {
                                 loadStopForecast(
                                         Preferences.selectedStopName(
-                                                getContext(),
+                                                getActivity().getApplicationContext(),
                                                 line
                                         ),
                                         false
@@ -869,10 +869,12 @@ public void failure(RetrofitError retrofitError) {
                         Log.e(LOG_TAG, "Response: " + retrofitError.getResponse().getUrl());
                     }
 
-                    Log.e(LOG_TAG, "Status: " + Integer.toString(retrofitError.getResponse().getStatus()));
+                    Log.e(LOG_TAG, "Status: " +
+                            Integer.toString(retrofitError.getResponse().getStatus()));
 
                     if (retrofitError.getResponse().getHeaders() != null) {
-                        Log.e(LOG_TAG, "Headers: " + retrofitError.getResponse().getHeaders().toString());
+                        Log.e(LOG_TAG, "Headers: " +
+                                retrofitError.getResponse().getHeaders().toString());
                     }
 
                     if (retrofitError.getResponse().getBody() != null) {

diff --git a/mobile/src/main/res/values/strings.xml b/mobile/src/main/res/values/strings.xml
@@ -23,8 +23,8 @@
         App
     -->
     <string name="app_name">Luas at a Glance</string>
-    <string name="version_code" translatable="false">121</string>
+    <string name="version_code" translatable="false">126</string>
-    <string name="version_name" translatable="false">0.121</string>
+    <string name="version_name" translatable="false">0.126</string>
 
     <!--
         Activities
@@ -166,16 +166,19 @@
     <!--
         What's New
     -->
-    <string name="whatsnew_title_current" translatable="false">Version 0.116 (19th November 2018)
+    <string name="whatsnew_title_current" translatable="false">Version 0.126 (18th December 2018)
     </string>
-    <string name="whatsnew_content_current" translatable="false">\u2022 Added link to privacy policy
+    <string name="whatsnew_content_current" translatable="false">\u2022 [NEW] Swiping down to
-        in About dialog.
+        refresh the stop forecast now displays when the times were generated by the Luas RTPI
+        system.\n\u2022 [FIXED] Fixed bug where <b>Alerts</b> button could turn red for an alert on
+        the wrong line.\n\u2022 [FIXED] Fixed intermittent crash on some devices. Thanks to everyone
+        who reported this.\n\u2022 [FIXED] Fixed widget crash affecting some users. Thanks to
+        everyone who reported this.
     </string>
-    <string name="whatsnew_title_previous" translatable="false">Version 0.112 (10th October 2018)
+    <string name="whatsnew_title_previous" translatable="false">Version 0.116 (19th November 2018)
     </string>
-    <string name="whatsnew_content_previous" translatable="false">\u2022 [FIXED] Fixed 404 error
+    <string name="whatsnew_content_previous" translatable="false">\u2022 Added link to privacy
-        which occurred when tapping the <b>Alerts</b> button for some users. Many thanks to the user
+        policy in About dialog.
-        who reported this issue to my email!
     </string>
 
     <!--

diff --git a/wear/build.gradle b/wear/build.gradle
@@ -10,8 +10,8 @@ android {
         applicationId "org.thecosmicfrog.luasataglance"
         minSdkVersion 21
         targetSdkVersion 28
-        versionCode 12100
+        versionCode 12600
-        versionName "0.121"
+        versionName "0.126"
     }
     buildTypes {
         release {
@@ -23,7 +23,7 @@ android {
     productFlavors {
         wear2 {
             minSdkVersion 25
-            versionCode 12101
+            versionCode 12601
         }
     }
 }

diff --git a/wear/src/main/res/values/strings.xml b/wear/src/main/res/values/strings.xml
@@ -23,8 +23,8 @@
         App
     -->
     <string name="app_name">Luas at a Glance</string>
-    <string name="version_code" translatable="false">121</string>
+    <string name="version_code" translatable="false">126</string>
-    <string name="version_name" translatable="false">0.121</string>
+    <string name="version_name" translatable="false">0.126</string>
 
     <!--
         Activities