Updated permissions and changed to more recent BLE api to try to get scanning working.

.gitignore

@@ -3,7 +3,8 @@ build
 .idea
 local.properties
 *.iml
-app/app-release.apk
+app/release/app-release.apk
 app/build
 app/app.iml
+app/release/output-metadata.json
 

app/release/output-metadata.json

@@ -1,18 +0,0 @@
-{
-  "version": 2,
-  "artifactType": {
-    "type": "APK",
-    "kind": "Directory"
-  },
-  "applicationId": "uk.org.openseizuredetector",
-  "variantName": "processReleaseResources",
-  "elements": [
-    {
-      "type": "SINGLE",
-      "filters": [],
-      "versionCode": 89,
-      "versionName": "3.6.3c",
-      "outputFile": "app-release.apk"
-    }
-  ]
-}

app/src/main/AndroidManifest.xml

@@ -2,11 +2,14 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     package="uk.org.openseizuredetector"
-    android:versionCode="89"
+    android:versionCode="90"
-    android:versionName="3.6.3c">
+    android:versionName="3.6.3d">
     <!-- android:allowBackup="false" -->
     <uses-permission android:name="android.permission.BLUETOOTH" />
     <uses-permission android:name="android.permission.BLUETOOTH_ADMIN" />
+    <uses-permission android:name="android.permission.BLUETOOTH_PRIVILEGED" />
+    <uses-permission android:name="android.permission.BLUETOOTH_SCAN" />
+    <uses-permission android:name="android.permission.BLUETOOTH_CONNECT" />
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.ACCESS_WIFI_STATE" />
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
@@ -26,6 +29,9 @@
         android:name="android.hardware.telephony"
         android:required="false" />
 
+    <uses-feature android:name="android.hardware.bluetooth_le"  android:required="false" />
+
+
     <application
         android:icon="@drawable/star_of_life_48x48"
         android:label="@string/app_name"

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

@@ -23,6 +23,9 @@ import android.app.ListActivity;
 import android.bluetooth.BluetoothAdapter;
 import android.bluetooth.BluetoothDevice;
 import android.bluetooth.BluetoothManager;
+import android.bluetooth.le.BluetoothLeScanner;
+import android.bluetooth.le.ScanCallback;
+import android.bluetooth.le.ScanResult;
 import android.content.Context;
 import android.content.Intent;
 import android.content.SharedPreferences;
@@ -31,6 +34,7 @@ import android.os.Bundle;
 import android.os.Handler;
 import android.preference.PreferenceManager;
 import android.support.v4.app.ActivityCompat;
+import android.support.v4.content.ContextCompat;
 import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.Menu;
@@ -46,12 +50,15 @@ import android.widget.Toast;
 
 import java.util.ArrayList;
 
+import static android.support.v4.content.PermissionChecker.PERMISSION_GRANTED;
+
 /**
  * Activity for scanning and displaying available Bluetooth LE devices.
  */
 public class BLEScanActivity extends ListActivity {
     private LeDeviceListAdapter mLeDeviceListAdapter;
     private BluetoothAdapter mBluetoothAdapter;
+    private BluetoothLeScanner mBluetoothLeScanner;
     private boolean mScanning;
     private Handler mHandler;
     private boolean bleAvailable = false;
@@ -60,9 +67,11 @@ public class BLEScanActivity extends ListActivity {
     private final String TAG = "BLEScanActivity";
 
     private final String[] REQUIRED_PERMISSIONS = {
+            Manifest.permission.ACCESS_FINE_LOCATION,
             Manifest.permission.ACCESS_COARSE_LOCATION,
             Manifest.permission.BLUETOOTH,
             Manifest.permission.BLUETOOTH_ADMIN,
+            //Manifest.permission.BLUETOOTH_PRIVILEGED,
     };
 
     private static final int REQUEST_ENABLE_BT = 1;
@@ -98,6 +107,8 @@ public class BLEScanActivity extends ListActivity {
             finish();
             return;
         }
+
+        mBluetoothLeScanner = mBluetoothAdapter.getBluetoothLeScanner();
     }
 
     @Override
@@ -166,6 +177,43 @@ public class BLEScanActivity extends ListActivity {
             tv.setText("Bluetooth Adapter Enabled OK");
         }
 
+        requestBTPermissions(this);
+
+        for (int i = 0; i < REQUIRED_PERMISSIONS.length; i++) {
+            if (ContextCompat.checkSelfPermission(this, REQUIRED_PERMISSIONS[i]) == PERMISSION_GRANTED) {
+                Log.i(TAG, "Permission " + REQUIRED_PERMISSIONS[i] + " OK");
+            } else {
+                Log.e(TAG, "Permission " + REQUIRED_PERMISSIONS[i] + " NOT GRANTED");
+                Toast.makeText(this, "ERROR - Permission " + REQUIRED_PERMISSIONS[i] + " not Granted - this will not work!!!!!", Toast.LENGTH_SHORT).show();
+            }
+        }
+
+        tv = (TextView) findViewById(R.id.ble_perm1_tv);
+        if (ContextCompat.checkSelfPermission(this, REQUIRED_PERMISSIONS[0]) == PERMISSION_GRANTED) {
+            tv.setText("Permission " + REQUIRED_PERMISSIONS[0] + " OK");
+        } else {
+            tv.setText("ERROR: Permission " + REQUIRED_PERMISSIONS[0] + " NOT GRANTED");
+        }
+        tv = (TextView) findViewById(R.id.ble_perm2_tv);
+        if (ContextCompat.checkSelfPermission(this, REQUIRED_PERMISSIONS[1]) == PERMISSION_GRANTED) {
+            tv.setText("Permission " + REQUIRED_PERMISSIONS[1] + " OK");
+        } else {
+            tv.setText("ERROR: Permission " + REQUIRED_PERMISSIONS[1] + " NOT GRANTED");
+        }
+        tv = (TextView) findViewById(R.id.ble_perm3_tv);
+        if (ContextCompat.checkSelfPermission(this, REQUIRED_PERMISSIONS[2]) == PERMISSION_GRANTED) {
+            tv.setText("Permission " + REQUIRED_PERMISSIONS[2] + " OK");
+        } else {
+            tv.setText("ERROR: Permission " + REQUIRED_PERMISSIONS[2] + " NOT GRANTED");
+        }
+        tv = (TextView) findViewById(R.id.ble_perm4_tv);
+        if (ContextCompat.checkSelfPermission(this, REQUIRED_PERMISSIONS[3]) == PERMISSION_GRANTED) {
+            tv.setText("Permission " + REQUIRED_PERMISSIONS[3] + " OK");
+        } else {
+            tv.setText("ERROR: Permission " + REQUIRED_PERMISSIONS[3] + " NOT GRANTED");
+        }
+
+
         // Initializes list view adapter.
         mLeDeviceListAdapter = new LeDeviceListAdapter();
         setListAdapter(mLeDeviceListAdapter);
@@ -196,7 +244,7 @@ public class BLEScanActivity extends ListActivity {
         if (device == null) return;
         Log.v(TAG, "onListItemClick: Device=" + device.getName() + ", Addr=" + device.getAddress());
         if (mScanning) {
-            mBluetoothAdapter.stopLeScan(mLeScanCallback);
+            mBluetoothLeScanner.stopScan(mLeScanCallback);
             mScanning = false;
         }
         Log.v(TAG, "Saving Device Details");
@@ -210,7 +258,7 @@ public class BLEScanActivity extends ListActivity {
 
             Log.v(TAG, "Saved Device Name=" + device.getName() + " and Address=" + device.getAddress());
         } catch (Exception ex) {
-            Log.e(TAG, "Error Saving Devie Name and Address!");
+            Log.e(TAG, "Error Saving Device Name and Address!");
             Toast toast = Toast.makeText(this, "Problem Saving Device Name and Address", Toast.LENGTH_SHORT);
             toast.show();
         }
@@ -220,16 +268,35 @@ public class BLEScanActivity extends ListActivity {
         finish();
     }
 
+    public void requestBTPermissions(Activity activity) {
+        if (mPermissionsRequested) {
+            Log.i(TAG, "requestPermissions() - request already sent - not doing anything");
+        } else {
+            Log.i(TAG, "requestPermissions() - requesting permissions");
+            for (int i = 0; i < REQUIRED_PERMISSIONS.length; i++) {
+                if (ActivityCompat.shouldShowRequestPermissionRationale(activity,
+                        REQUIRED_PERMISSIONS[i])) {
+                    Log.i(TAG, "shouldShowRationale for permission" + REQUIRED_PERMISSIONS[i]);
+                }
+            }
+            ActivityCompat.requestPermissions(activity,
+                    REQUIRED_PERMISSIONS,
+                    42);
+            mPermissionsRequested = true;
+        }
+    }
+
+
     private void scanLeDevice(final boolean enable) {
         TextView tv;
-        requestPermissions(this);
+
         if (enable) {
             // Stops scanning after a pre-defined scan period.
             mHandler.postDelayed(new Runnable() {
                 @Override
                 public void run() {
                     mScanning = false;
-                    mBluetoothAdapter.stopLeScan(mLeScanCallback);
+                    mBluetoothLeScanner.stopScan(mLeScanCallback);
                     invalidateOptionsMenu();
                     TextView tv = (TextView) (findViewById(R.id.ble_scan_status_tv));
                     tv.setText("Stopped");
@@ -240,7 +307,7 @@ public class BLEScanActivity extends ListActivity {
             }, SCAN_PERIOD);
 
             mScanning = true;
-            mBluetoothAdapter.startLeScan(mLeScanCallback);
+            mBluetoothLeScanner.startScan(mLeScanCallback);
             tv = (TextView) (findViewById(R.id.ble_scan_status_tv));
             tv.setText("Scanning");
             Button b = (Button) findViewById(R.id.startScanButton);
@@ -248,7 +315,7 @@ public class BLEScanActivity extends ListActivity {
 
         } else {
             mScanning = false;
-            mBluetoothAdapter.stopLeScan(mLeScanCallback);
+            mBluetoothLeScanner.stopScan(mLeScanCallback);
             tv = (TextView) (findViewById(R.id.ble_scan_status_tv));
             tv.setText("Stopped");
             Button b = (Button) findViewById(R.id.startScanButton);
@@ -326,20 +393,15 @@ public class BLEScanActivity extends ListActivity {
     }
 
     // Device scan callback.
-    private BluetoothAdapter.LeScanCallback mLeScanCallback =
+    private ScanCallback mLeScanCallback =
-            new BluetoothAdapter.LeScanCallback() {
+            new ScanCallback() {
-
                 @Override
-                public void onLeScan(final BluetoothDevice device, int rssi, byte[] scanRecord) {
+                public void onScanResult(int callbackType, ScanResult result) {
-                    Log.v(TAG,"LEScanCallback - device="+device.getName());
+                    //super.onScanResult(callbackType, result);
-                    runOnUiThread(new Runnable() {
+                    Log.v(TAG, "ScanCallback - " + result.getDevice().getName());
-                        @Override
+                    mLeDeviceListAdapter.addDevice(result.getDevice());
-                        public void run() {
-                            mLeDeviceListAdapter.addDevice(device);
                     mLeDeviceListAdapter.notifyDataSetChanged();
                 }
-                    });
-                }
             };
 
     static class ViewHolder {
@@ -348,21 +410,4 @@ public class BLEScanActivity extends ListActivity {
     }
 
 
-    public void requestPermissions(Activity activity) {
-        if (mPermissionsRequested) {
-            Log.i(TAG, "requestPermissions() - request already sent - not doing anything");
-        } else {
-            Log.i(TAG, "requestPermissions() - requesting permissions");
-            for (int i = 0; i < REQUIRED_PERMISSIONS.length; i++) {
-                if (ActivityCompat.shouldShowRequestPermissionRationale(activity,
-                        REQUIRED_PERMISSIONS[i])) {
-                    Log.i(TAG, "shouldShowRationale for permission" + REQUIRED_PERMISSIONS[i]);
-                }
-            }
-            ActivityCompat.requestPermissions(activity,
-                    REQUIRED_PERMISSIONS,
-                    42);
-            mPermissionsRequested = true;
-        }
-    }
 }

app/src/main/res/layout/ble_scan_activity.xml

@@ -42,6 +42,28 @@
         android:layout_height="wrap_content"
         android:text="current_ble_device_tv" />
 
+    <TextView
+        android:id="@+id/ble_perm1_tv"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="ble_perm1_tv" />
+    <TextView
+        android:id="@+id/ble_perm2_tv"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="ble_perm2_tv" />
+    <TextView
+        android:id="@+id/ble_perm3_tv"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="ble_perm3_tv" />
+    <TextView
+        android:id="@+id/ble_perm4_tv"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="ble_perm4_tv" />
+
+
     <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
         android:layout_width="match_parent"
         android:layout_height="fill_parent"