From 8e4b5328bd41e5dd5062b7b760c5bd37b36c48f9 Mon Sep 17 00:00:00 2001
From: Tad <tad@spotco.us>
Date: Thu, 14 Dec 2017 15:56:30 -0500
Subject: [PATCH] Add database update button

---
 .../java/us/spotco/malwarescanner/MainActivity.java    | 10 ++++------
 .../java/us/spotco/malwarescanner/MalwareScanner.java  | 10 +++++-----
 app/src/main/res/menu/menu_main.xml                    |  7 +++++--
 3 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/us/spotco/malwarescanner/MainActivity.java b/app/src/main/java/us/spotco/malwarescanner/MainActivity.java
index 8a08fb1..5eeede4 100644
--- a/app/src/main/java/us/spotco/malwarescanner/MainActivity.java
+++ b/app/src/main/java/us/spotco/malwarescanner/MainActivity.java
@@ -12,7 +12,7 @@ import android.widget.TextView;
 
 public class MainActivity extends AppCompatActivity {
 
-    private boolean scanSystem = true;
+    private boolean scanSystem = false;
     private boolean scanApps = true;
     private boolean scanInternal = true;
     private boolean scanExternal = false;
@@ -32,11 +32,6 @@ public class MainActivity extends AppCompatActivity {
         logView.append("\nDisclaimer: This is an extremely basic signature scanner\n\n");
 
         final Database database = new Database(this, logView);
-
-        if (!Database.doesDatabaseExist()) {
-            Database.updateDatabase(Database.signatureDatabases);
-        }
-
         final MalwareScanner scanner = new MalwareScanner(this, logView);
 
         FloatingActionButton fab = findViewById(R.id.fab);
@@ -61,6 +56,9 @@ public class MainActivity extends AppCompatActivity {
     @Override
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
+            case R.id.mnuUpdateDatabase:
+                Database.updateDatabase(Database.signatureDatabases);
+                break;
             case R.id.mnuScanSystem:
                 scanSystem = !item.isChecked();
                 item.setChecked(scanSystem);
diff --git a/app/src/main/java/us/spotco/malwarescanner/MalwareScanner.java b/app/src/main/java/us/spotco/malwarescanner/MalwareScanner.java
index 0cb7a98..421d304 100644
--- a/app/src/main/java/us/spotco/malwarescanner/MalwareScanner.java
+++ b/app/src/main/java/us/spotco/malwarescanner/MalwareScanner.java
@@ -30,11 +30,11 @@ public class MalwareScanner {
     }
 
     public void startScanner(boolean scanSystem, boolean scanApps, boolean scanInternal, boolean scanExternal) {
-        //if(Database.doesDatabaseExist()) {
-        malwareScannerTask = new MalwareScannerTask().execute(scanSystem, scanApps, scanInternal, scanExternal);
-        //} else {
-        //   log.append("No database found... download one first!\n");
-        //}
+        if(Database.doesDatabaseExist()) {
+            malwareScannerTask = new MalwareScannerTask().execute(scanSystem, scanApps, scanInternal, scanExternal);
+        } else {
+           log.append("No database found... download one first!\n");
+        }
     }
 
     public void stopScanner() {
diff --git a/app/src/main/res/menu/menu_main.xml b/app/src/main/res/menu/menu_main.xml
index 481b174..cabbb92 100644
--- a/app/src/main/res/menu/menu_main.xml
+++ b/app/src/main/res/menu/menu_main.xml
@@ -1,13 +1,16 @@
 <menu xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     tools:context="us.spotco.malwarescanner.MainActivity">
 
+    <item
+        android:id="@+id/mnuUpdateDatabase"
+        android:title="@string/lblUpdateDatabase" />
+
     <item
         android:id="@+id/mnuScanSystem"
         android:title="@string/lblScanSystem"
         android:checkable="true"
-        android:checked="true" />
+        android:checked="false" />
     <item
         android:id="@+id/mnuScanApps"
         android:title="@string/lblScanApps"