Android Permissions

[kheldar13Participant]

I am having trouble figuring out why the permissions for storage aren’t being set at install. When the app installs it says that there are no special permissions, and then I have to go into settings and manually set the storage permission. After that everything works fine. I have the READ_EXTERNAL_STORAGE and WRITE_EXTERNAL_STORAGE set in the manifest, so I’m not sure why it isn’t being set at install. Any ideas?

[Mark SiblyKeymaster]

Can you post/attach the manifest file here?

[kheldar13Participant]

Here is the manifest.

[kheldar13Participant]

<?xml version=”1.0″ encoding=”utf-8″?>

<manifest
xmlns:android=”http://schemas.android.com/apk/res/android”
package=”com.sscalculator.bighatgames”>

<uses-sdk android:minSdkVersion=”10″ android:targetSdkVersion=”23″ />

<uses-feature android:glEsVersion=”0x00020000″ />

<uses-permission android:name=”android.permission.INTERNET” />
<uses-permission android:name=”android.permission.ACCESS_NETWORK_STATE” />
<uses-permission android:name=”android.permission.READ_EXTERNAL_STORAGE” />
<uses-permission android:name=”android.permission.WRITE_EXTERNAL_STORAGE” />

<application
android:allowBackup=”true”
android:icon=”@mipmap/ic_launcher”
android:label=”@string/app_name”
android:supportsRtl=”true”
android:theme=”@style/AppTheme”>
<activity
android:name=”.SSCalculator”
android:label=”SSCalculator”
android:screenOrientation=”portrait”
android:configChanges=”keyboardHidden|orientation|screenSize”
android:theme=”@android:style/Theme.NoTitleBar.Fullscreen”
android:launchMode=”singleTop”>

android:configChanges=”orientation”>

<intent-filter>
<action android:name=”android.intent.action.MAIN” />
<category android:name=”android.intent.category.LAUNCHER” />
</intent-filter>

</activity>
</application>

</manifest>

[CopperCircleParticipant]

HI, if you are building for Android SDK 23+ you have to request permissions from the user not just in the manifest.

This is the java module code I created for Monkey1 to deal with this:

Monkey

import android.support.v4.content.ContextCompat; import android.support.v4.app.ActivityCompat; import android.Manifest; import android.content.pm.PackageManager; class PermissionNative extends ActivityDelegate implements ActivityCompat.OnRequestPermissionsResultCallback { private boolean done; private boolean result; public void CheckPermissionsNative(boolean calendar, boolean camera, boolean location, boolean storage){ done = false; if(Build.VERSION.SDK_INT >= 23){ BBAndroidGame.AndroidGame().AddActivityDelegate(this); AskForPermissionsNative(calendar, camera, location, storage); }else { done = true; result = true; } } public void AskForPermissionsNative(boolean calendar, boolean camera, boolean location, boolean storage){ Activity activity = BBAndroidGame.AndroidGame().GetActivity(); List<String> requestedPermissionsList = new ArrayList<String>(); if(calendar == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.READ_CALENDAR) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.READ_CALENDAR); } if(camera == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.CAMERA) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.CAMERA); } if(location == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.ACCESS_FINE_LOCATION); } if(storage == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.READ_EXTERNAL_STORAGE) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.READ_EXTERNAL_STORAGE); } if(requestedPermissionsList.size() >0){ String[] requestedPermissions = new String[requestedPermissionsList.size()]; requestedPermissions = requestedPermissionsList.toArray(requestedPermissions); ActivityCompat.requestPermissions( activity, requestedPermissions, 0); }else { done = true; result = true; } } public boolean CheckPermissionsDoneNative(){ return done; } public boolean CheckPermissionsResultNative(){ return result; } @Override public void onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults) { switch (requestCode) { case 0: { done = true; if (grantResults.length > 0) { result = true; for (int i=0; i<permissions.length; i++) { if (grantResults[i] != PackageManager.PERMISSION_GRANTED) { result = false; } } } else { result = false; } return; } } } }

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78

import android.support.v4.content.ContextCompat; import android.support.v4.app.ActivityCompat; import android.Manifest; import android.content.pm.PackageManager;   class PermissionNative extends ActivityDelegate implements ActivityCompat.OnRequestPermissionsResultCallback { private boolean done; private boolean result;   public void CheckPermissionsNative(boolean calendar, boolean camera, boolean location, boolean storage){ done = false; if(Build.VERSION.SDK_INT >= 23){ BBAndroidGame.AndroidGame().AddActivityDelegate(this); AskForPermissionsNative(calendar, camera, location, storage); }else { done = true; result = true; } } public void AskForPermissionsNative(boolean calendar, boolean camera, boolean location, boolean storage){ Activity activity = BBAndroidGame.AndroidGame().GetActivity();   List<String> requestedPermissionsList = new ArrayList<String>();   if(calendar == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.READ_CALENDAR) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.READ_CALENDAR); } if(camera == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.CAMERA) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.CAMERA); } if(location == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.ACCESS_FINE_LOCATION); } if(storage == true && ActivityCompat.checkSelfPermission(activity, Manifest.permission.READ_EXTERNAL_STORAGE) != PackageManager.PERMISSION_GRANTED){ requestedPermissionsList.add(Manifest.permission.READ_EXTERNAL_STORAGE); }   if(requestedPermissionsList.size() >0){ String[] requestedPermissions = new String[requestedPermissionsList.size()]; requestedPermissions = requestedPermissionsList.toArray(requestedPermissions); ActivityCompat.requestPermissions( activity, requestedPermissions, 0); }else { done = true; result = true; } } public boolean CheckPermissionsDoneNative(){ return done; } public boolean CheckPermissionsResultNative(){ return result; }   @Override public void onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults) { switch (requestCode) { case 0: { done = true;   if (grantResults.length > 0) { result = true; for (int i=0; i<permissions.length; i++) { if (grantResults[i] != PackageManager.PERMISSION_GRANTED) { result = false; } } } else { result = false; } return; } } }   }

[Mark SiblyKeymaster]

Ok, just adding a permission module now.

Thanks for the sample code coppercircle. Am I correct in thinking you also has to add the permissions callback to AppDelagate and main Actitiy class?

[CopperCircleParticipant]

Hi, yeah just a delagate and callback for the response

Monkey

Class MyApp Extends App Implements PermissionDelegate Method OnCreate:Int() Permission.CheckPermissions(Self, True, True, True, False) Return 0 End Method Method OnRender:Int() Cls 255, 255, 200 Return 0 End Method Method OnUpdate:Int() Permission.Update() Return 0 End Method Method OnPermissionsReply:Void(allAllowed:Bool) Print "OnPermissionsReply: " + Int(allAllowed) End End

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

Class MyApp Extends App Implements PermissionDelegate Method OnCreate:Int() Permission.CheckPermissions(Self, True, True, True, False) Return 0 End Method Method OnRender:Int() Cls 255, 255, 200 Return 0 End Method Method OnUpdate:Int() Permission.Update() Return 0 End Method Method OnPermissionsReply:Void(allAllowed:Bool) Print "OnPermissionsReply: " + Int(allAllowed) End End

[Mark SiblyKeymaster]

I actually meant the native target MonkeyActivity java class – my version doesn’t have the onPermissionsReply method.

Ditto it’s not supported via the delegate mechanism. It’s easy enough to add, just wondering if I  missed a trick somewhere. I hate having to continually mung up the core Activity class each time we find something else that needs to be delegated out. It’s a pretty crappy design on the part of Android IMO.

[CopperCircleParticipant]

Yes unfortunately you have to add it to the target, I haven’t found anyway of of creating the delegate on the fly.

[kheldar13Participant]

Thanks for the answers, guys. Greatly Appreciated.

So how would I use this in Monkey2?

[Mark SiblyKeymaster]

A version of this has been pushed to the develop branch of github. It will also be included in the v1.1.08 release due in a few days, so you can just wait for that if you want.

Here it is in action, hopefully pretty straightforward:

Monkey

Namespace myapp #Import "<std>" #Import "<mojo>" Using std.. Using mojo.. Class MyWindow Extends Window Method New( title:String="Simple mojo app",width:Int=640,height:Int=480,flags:WindowFlags=Null ) Super.New( title,width,height,flags ) Print "Hello World from permissions demo!" Local permissions:=New String[]( "android.permission.INTERNET", "android.permission.ACCESS_NETWORK_STATE", "android.permission.READ_EXTERNAL_STORAGE", "android.permission.WRITE_EXTERNAL_STORAGE", "com.android.vending.BILLING" ) For Local i:=0 Until permissions.Length Print "CheckPermission(~q"+permissions[i]+"~q)="+CheckPermission( permissions[i] ) Next RequestPermissions( permissions,Lambda( results:int[] ) If Not results Print "RequestPermissions cancelled" Return Endif For Local i:=0 Until results.Length If results[i] Print "Permission granted for "+permissions[i] Else Print "Permission NOT granted for "+permissions[i] Endif Next End ) End Method OnRender( canvas:Canvas ) Override Global n:Int n+=1 App.RequestRender() canvas.DrawText( "Hello World! n="+n,Width/2,Height/2,.5,.5 ) End End Function Main() New AppInstance New MyWindow App.Run() End

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68

Namespace myapp   #Import "<std>" #Import "<mojo>"   Using std.. Using mojo..   Class MyWindow Extends Window   Method New( title:String="Simple mojo app",width:Int=640,height:Int=480,flags:WindowFlags=Null )   Super.New( title,width,height,flags ) Print "Hello World from permissions demo!" Local permissions:=New String[]( "android.permission.INTERNET", "android.permission.ACCESS_NETWORK_STATE", "android.permission.READ_EXTERNAL_STORAGE", "android.permission.WRITE_EXTERNAL_STORAGE", "com.android.vending.BILLING" ) For Local i:=0 Until permissions.Length Print "CheckPermission(~q"+permissions[i]+"~q)="+CheckPermission( permissions[i] ) Next RequestPermissions( permissions,Lambda( results:int[] ) If Not results Print "RequestPermissions cancelled" Return Endif For Local i:=0 Until results.Length If results[i] Print "Permission granted for "+permissions[i] Else Print "Permission NOT granted for "+permissions[i] Endif Next End ) End   Method OnRender( canvas:Canvas ) Override Global n:Int n+=1 App.RequestRender() canvas.DrawText( "Hello World! n="+n,Width/2,Height/2,.5,.5 ) End End   Function Main()   New AppInstance New MyWindow App.Run() End

You might also want to read up on runtime permissions – was certainly news to me!

https://developer.android.com/training/permissions/requesting.html

You apparently only need to do this if targetSDK is >=23, which it is in the default manifest for monkey2 projects. So one way to avoid all this I guess is to just change targetSDK to 22!

Thanks to CopperCircle for pointing me in the right direction.

[kheldar13Participant]

Awesome!
Thanks Mark.
and Thanks CopperCircle

[Mark SiblyKeymaster]

Also, can you let me know what you want to do with external storage?

There are a lot of directories and APIS involved and I just wanna know what to concentrate on first.

[kheldar13Participant]

Right now, I’m just using it to store user settings in a text file. Later, in a created directory, I may have some records(also in a text file) readable by other apps. So just text files at this point.

[DiffrenzyKeymaster]

I would like to save some generated images and svg files to a user specified folder, maybe just “downloads” on Android. Not sure where to put them on iOS, but the new 11, might have some pointeres, because theres a file manager in that.

[Author]

Posts