Fixed crash if LogManager fails to start (FIXES #196)

CHANGELOG.md

@@ -1,6 +1,6 @@
 	OpenSeizureDetector Android App - Change Log
 	============================================
-    V4.2.10 - fixed (infrequent) crash when opening data sharing page.
+    V4.2.10 - fixed (infrequent) crash when opening data sharing page (#195), and crash if log manager fails to start (#196)
     V4.2.9 - fixed crash when using Polish translation.
     V4.2.8 - 
            - Fixed crash in export data function when using european style comma based decimal separator.

app/src/main/java/uk/org/openseizuredetector/LogManager.java

@@ -381,9 +381,15 @@ public class LogManager {
         values.put("notes", desc);
         values.put("dataJSON", dataJSON);
 
-        long newRowId = mOsdDb.insert(mEventsTableName, null, values);
-        Log.d(TAG, "createLocalEvent(): Created Row ID" + newRowId);
-        return true;
+        if (mOsdDb != null) {
+            long newRowId = mOsdDb.insert(mEventsTableName, null, values);
+            Log.d(TAG, "createLocalEvent(): Created Row ID" + newRowId);
+            return true;
+        } else {
+            Log.e(TAG,"createLocalEvent() - mOsdDb is null");
+            mUtil.showToast(mContext.getString(R.string.error_failed_to_create_local_event));
+            return false;
+        }
     }
 
     /**

app/src/main/java/uk/org/openseizuredetector/LogManagerControlActivity.java

@@ -219,7 +219,7 @@ public class LogManagerControlActivity extends AppCompatActivity {
             });
         } else {
             Log.e(TAG,"ERROR: initialiseServiceConnection() - mLm is null");
-            mUtil.showToast("ERROR: Failed to start Log Manager");
+            mUtil.showToast(getString(R.string.error_failed_to_start_log_manager));
         }
     }
 

app/src/main/res/values/strings.xml

@@ -576,4 +576,6 @@
     <string name="BTpermissions_required">Bluetooth Permissions Required</string>
     <string name="BT_permissions_rationale">Bluetooth permissions are required to communicate with the bluetooth (BLE) data source</string>
     <string name="restart_server">Restart Server</string>
+    <string name="error_failed_to_create_local_event">ERROR: failed to create local event</string>
+    <string name="error_failed_to_start_log_manager">ERROR: Failed to start Log Manager</string>
 </resources>