Target android14 (#74)

* targetSdk 34 * Declare DfuService foreground type * onPause/Resume don't signal back/forground * Fix call to LayoutInflater; add StrictMode logging * Fix screen update after DFU error
microbit-foundation · Jul 24, 2024 · baac741 · baac741
1 parent 8626caa
commit baac741
Show file tree

Hide file tree

Showing 6 changed files with 43 additions and 7 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -5,7 +5,7 @@ android {
     defaultConfig {
         applicationId "com.samsung.microbit"
         minSdkVersion 21
-        targetSdk 33
+        targetSdk 34
         // When target SDK version is 30+,
         // requestLegacyExternalStorage will continue to work for 29 Android 10
     }

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -33,6 +33,7 @@
     <uses-permission android:name="android.permission.READ_PHONE_STATE"/>
     <uses-permission android:name="android.permission.CAMERA"/>
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
+    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_CONNECTED_DEVICE" />
 
     <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" android:maxSdkVersion="30"/>
     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" android:maxSdkVersion="30"/>
@@ -167,7 +168,9 @@
 
         <service
             android:name=".service.DfuService"
-            android:enabled="true"/>
+            android:foregroundServiceType="connectedDevice"
+            android:enabled="true"
+            android:exported="false"/>
 
         <service
             android:name=".service.PluginService"
@@ -189,7 +192,7 @@
             android:name=".service.PartialFlashingService"
             android:enabled="true"
             android:launchMode="singleInstance"
-        />
+            android:exported="false"/>
 
         <activity
             android:name=".ui.activity.SplashScreenActivity"

diff --git a/app/src/main/java/com/samsung/microbit/MBApp.java b/app/src/main/java/com/samsung/microbit/MBApp.java
@@ -4,6 +4,9 @@
 
 import android.app.Application;
 import android.graphics.Typeface;
+import android.os.Build;
+import android.os.StrictMode;
+import android.os.strictmode.Violation;
 import android.util.Log;
 
 import androidx.annotation.NonNull;
@@ -13,6 +16,8 @@
 
 import com.samsung.microbit.MBAppState;
 
+import java.util.concurrent.Executors;
+
 /**
  * Represents a custom class of the app.
  * Provides some resources that use along app modules,
@@ -39,6 +44,30 @@ public void logi(String message) {
 
     @Override
     public void onCreate() {
+//        if (DEBUG) {
+//            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
+//                StrictMode.setThreadPolicy(new StrictMode.ThreadPolicy.Builder()
+//                        .detectAll()
+//                        .permitDiskReads()
+//                        .penaltyLog()
+//                        .penaltyListener(Executors.newSingleThreadExecutor(), new StrictMode.OnThreadViolationListener() {
+//                            @Override
+//                            public void onThreadViolation(Violation v) {
+//                            }
+//                        })
+//                        .build());
+//
+//                StrictMode.setVmPolicy(new StrictMode.VmPolicy.Builder()
+//                        .detectAll()
+//                        .penaltyLog()
+//                        .penaltyListener(Executors.newSingleThreadExecutor(), new StrictMode.OnVmViolationListener() {
+//                            @Override
+//                            public void onVmViolation(Violation v) {
+//                            }
+//                        })
+//                        .build());
+//            }
+//        }
         super.onCreate();
         app = this;
         ProcessLifecycleOwner.get().getLifecycle().addObserver(this);
@@ -96,16 +125,18 @@ public void onStart(@NonNull LifecycleOwner owner) {
         MBApp.getAppState().eventPairForeground();
     }
 
+    //onPause & onResume occur when pairing confirmation dialogue shows
+
     @Override
     public void onResume(@NonNull LifecycleOwner owner) {
         logi("onResume");
-        MBApp.getAppState().eventPairForeground();
+        //MBApp.getAppState().eventPairForeground();
     }
 
     @Override
     public void onPause(@NonNull LifecycleOwner owner) {
         logi("onPause");
-        MBApp.getAppState().eventPairBackground();
+         //MBApp.getAppState().eventPairBackground();
     }
 
     @Override

diff --git a/app/src/main/java/com/samsung/microbit/ui/activity/ProjectActivity.java b/app/src/main/java/com/samsung/microbit/ui/activity/ProjectActivity.java
@@ -2337,6 +2337,8 @@ public void onClick(View v) {
                 final int errorType = intent.getIntExtra(DfuBaseService.EXTRA_ERROR_TYPE, 0);
                 String error_message = "";
 
+                PopUp.hide();
+
                 switch (errorType) {
                     case DfuBaseService.ERROR_TYPE_COMMUNICATION_STATE:
                         if ( errorCode == 0x0085) {

diff --git a/app/src/main/java/com/samsung/microbit/ui/adapter/ProjectAdapter.java b/app/src/main/java/com/samsung/microbit/ui/adapter/ProjectAdapter.java
@@ -361,7 +361,7 @@ public View getView(int position, View convertView, ViewGroup parent) {
 
         Project project = mProjects.get(position);
         if(convertView == null) {
-            LayoutInflater inflater = LayoutInflater.from(MBApp.getApp());
+            LayoutInflater inflater = LayoutInflater.from( mProjectActivity);
             convertView = inflater.inflate(R.layout.project_items, null);
         }
 

diff --git a/pfLibrary b/pfLibrary