request permissions for camera and sdcard together

Author: vkocheryzhkin

FaceTracker/app/src/main/AndroidManifest.xml

@@ -9,7 +9,7 @@
 
     <uses-permission android:name="android.permission.CAMERA" />
     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
+    <!--<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />-->
 
     <uses-sdk
         android:minSdkVersion="15"

FaceTracker/app/src/main/java/com/google/android/gms/samples/vision/face/facetracker/FaceTrackerActivity.java

@@ -54,20 +54,19 @@
  * overlay graphics to indicate the position, size, and ID of each face.
  */
 public final class FaceTrackerActivity extends AppCompatActivity {
-    private static final String TAG = "FaceTracker";
+    private static final String TAG = "FR";
 
     private CameraSource mCameraSource = null;
 
     private CameraSourcePreview mPreview;
     private GraphicOverlay mGraphicOverlay;
-//    private Button mBtnDetect;
+    //private Button mBtnDetect;
     private CustomDetector customDetector;
     //private FaceDetector mPictureDetector;
 
     private static final int RC_HANDLE_GMS = 9001;
     // permission request codes need to be < 256
-    private static final int RC_HANDLE_CAMERA_PERM = 2;
-    private static final int REQUEST_WRITE_STORAGE = 112;
+    private static final int RC_HANDLE_CAMERA_AND_SDCARD_PERM = 7;
 
     private FaceRecognizer mFaceRecognizer;
 
@@ -88,26 +87,14 @@ public void onCreate(Bundle icicle) {
 //        mBtnDetect = (Button) findViewById(R.id.btnDetect);
         mFaceRecognizer = new FaceRecognizer();
 
-        // Check for the sdcard write permission.  If the
-        // permission is not granted yet, request permission.
-        int rc = ActivityCompat.checkSelfPermission(this, Manifest.permission.WRITE_EXTERNAL_STORAGE);
-        if (rc == PackageManager.PERMISSION_GRANTED) {
-            //createCameraSource();
-            Log.w(TAG, "SDCard write permission is granted.");
-        } else {
-            //requestCameraPermission();
-            requestSdCardPermission();
-            Log.w(TAG, "SDCard write permission is NOT granted.");
-        }
-
-
         // Check for the camera permission before accessing the camera.  If the
         // permission is not granted yet, request permission.
-        rc = ActivityCompat.checkSelfPermission(this, Manifest.permission.CAMERA);
-        if (rc == PackageManager.PERMISSION_GRANTED) {
+        int rc = ActivityCompat.checkSelfPermission(this, Manifest.permission.CAMERA);
+        int rs = ActivityCompat.checkSelfPermission(this, Manifest.permission.READ_EXTERNAL_STORAGE);
+        if (rc == PackageManager.PERMISSION_GRANTED && rs == PackageManager.PERMISSION_GRANTED) {
             createCameraSource();
         } else {
-            requestCameraPermission();
+            requestCameraAndSdCardPermission();
         }
     }
 
@@ -116,41 +103,15 @@ public void onCreate(Bundle icicle) {
      * showing a "Snackbar" message of why the permission is needed then
      * sending the request.
      */
-    private void requestCameraPermission() {
-        Log.w(TAG, "Camera permission is not granted. Requesting permission");
+    private void requestCameraAndSdCardPermission() {
+        Log.w(TAG, "Camera and sdcard permissions are not granted. Requesting permission");
 
-        final String[] permissions = new String[]{Manifest.permission.CAMERA};
+        final String[] permissions = new String[]{Manifest.permission.CAMERA,
+        Manifest.permission.READ_EXTERNAL_STORAGE};
 
         if (!ActivityCompat.shouldShowRequestPermissionRationale(this,
                 Manifest.permission.CAMERA)) {
-            ActivityCompat.requestPermissions(this, permissions, RC_HANDLE_CAMERA_PERM);
-            return;
-        }
-
-        final Activity thisActivity = this;
-
-        View.OnClickListener listener = new View.OnClickListener() {
-            @Override
-            public void onClick(View view) {
-                ActivityCompat.requestPermissions(thisActivity, permissions,
-                        RC_HANDLE_CAMERA_PERM);
-            }
-        };
-
-        Snackbar.make(mGraphicOverlay, R.string.permission_camera_rationale,
-                Snackbar.LENGTH_INDEFINITE)
-                .setAction(R.string.ok, listener)
-                .show();
-    }
-
-    private void requestSdCardPermission() {
-        Log.w(TAG, "SdCard permission is not granted. Requesting permission");
-
-        final String[] permissions = new String[]{Manifest.permission.WRITE_EXTERNAL_STORAGE};
-
-        if (!ActivityCompat.shouldShowRequestPermissionRationale(this,
-                Manifest.permission.WRITE_EXTERNAL_STORAGE)) {
-            ActivityCompat.requestPermissions(this, permissions, REQUEST_WRITE_STORAGE);
+            ActivityCompat.requestPermissions(this, permissions, RC_HANDLE_CAMERA_AND_SDCARD_PERM);
             return;
         }
 
@@ -160,11 +121,11 @@ private void requestSdCardPermission() {
             @Override
             public void onClick(View view) {
                 ActivityCompat.requestPermissions(thisActivity, permissions,
-                        REQUEST_WRITE_STORAGE);
+                        RC_HANDLE_CAMERA_AND_SDCARD_PERM);
             }
         };
 
-        Snackbar.make(mGraphicOverlay, R.string.permission_sdcard_rationale,
+        Snackbar.make(mGraphicOverlay, R.string.permission_camera_and_sdcard_rationale,
                 Snackbar.LENGTH_INDEFINITE)
                 .setAction(R.string.ok, listener)
                 .show();
@@ -310,7 +271,7 @@ protected void onDestroy() {
      */
     @Override
     public void onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults) {
-        if (requestCode != RC_HANDLE_CAMERA_PERM) {
+        if (requestCode != RC_HANDLE_CAMERA_AND_SDCARD_PERM) {
             Log.d(TAG, "Got unexpected permission result: " + requestCode);
             super.onRequestPermissionsResult(requestCode, permissions, grantResults);
             return;
@@ -334,8 +295,8 @@ public void onClick(DialogInterface dialog, int id) {
         };
 
         AlertDialog.Builder builder = new AlertDialog.Builder(this);
-        builder.setTitle("Face Tracker sample")
-                .setMessage(R.string.no_camera_permission)
+        builder.setTitle("FR demo")
+                .setMessage(R.string.no_camera_sdcard_permission)
                 .setPositiveButton(R.string.ok, listener)
                 .show();
     }
@@ -435,8 +396,4 @@ public void onDone() {
             mOverlay.remove(mFaceGraphic);
         }
     }
-
-    //public native int loadResources();
-    //public native String recognizeFromImage(Bitmap bmp);
-    //public native int nativeAdd(int a, int b);
 }

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

@@ -1,8 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <string name="ok">OK</string>
-    <string name="permission_camera_rationale">Access to the camera is needed for detection</string>
-    <string name="permission_sdcard_rationale">Access to the sdcard is needed for debug</string>
-    <string name="no_camera_permission">This application cannot run because it does not have the camera permission.  The application will now exit.</string>
+    <string name="permission_camera_and_sdcard_rationale">Access to the camera and read sdcard is needed for detection</string>
+    <string name="no_camera_sdcard_permission">This application cannot run because it does not have the camera or sdcard permissions.  The application will now exit.</string>
     <string name="low_storage_error">Face detector dependencies cannot be downloaded due to low device storage</string>
 </resources>