NMC-1920: Customized theming related changes for grid and list views.

NMC-2121: Theming applied to passcode.
NMC-2374: Image edit functionality customized.
NMC-1941: Fixed error message coming after camera permission allowed.
NMC-2052: Fix back arrow appearing as hamburger icon. NC PR: nextcloud#11769
NMC-2465: Fix local db version to avoid crash during app upgrade.
NMC-2576: Disabled switch account SnackBar.
NMC-2123: Widget disabled.
NMC-2127: System default night mode.
NMC-1919: Backup Calendar-Contacts configured with test cases.
NMC-2089: Reduce log levels
NMC-4633: disable permission check in search screen
Media View multi-selection customized
NMC-4799: uploads screen icon color fix
NMC-4862: deactivate github report snackbar

Author: surinder-tsys

app/src/main/AndroidManifest.xml

@@ -259,6 +259,7 @@
             android:exported="false" />
         <activity
             android:name="com.nextcloud.client.widget.DashboardWidgetConfigurationActivity"
+            android:enabled="false"
             android:exported="false">
             <intent-filter>
                 <action android:name="android.appwidget.action.APPWIDGET_CONFIGURE" />
@@ -289,6 +290,7 @@
             android:exported="false" />
         <receiver
             android:name="com.nextcloud.client.widget.DashboardWidgetProvider"
+            android:enabled="false"
             android:exported="false">
             <intent-filter>
                 <action android:name="android.appwidget.action.APPWIDGET_UPDATE" />
@@ -400,6 +402,7 @@
         <service
             android:name="com.nextcloud.client.widget.DashboardWidgetService"
             android:exported="true"
+            android:enabled="false"
             android:permission="android.permission.BIND_REMOTEVIEWS" />
 
         <provider

app/src/main/java/com/nextcloud/client/database/NextcloudDatabase.kt

@@ -98,7 +98,10 @@ abstract class NextcloudDatabase : RoomDatabase() {
     abstract fun recommendedFileDao(): RecommendedFileDao
 
     companion object {
-        const val FIRST_ROOM_DB_VERSION = 65
+        // NMC customization
+        // NMC play store version 7.21.9 had db version 64 before SqLite to Room migration
+        // Keeping it 65(as per NC) will lead to crash when user tried to upgrade the app
+        const val FIRST_ROOM_DB_VERSION = 64
         private var instance: NextcloudDatabase? = null
 
         @JvmStatic

app/src/main/java/com/nextcloud/client/editimage/EditImageActivity.kt

@@ -8,16 +8,15 @@
 package com.nextcloud.client.editimage
 
 import android.graphics.Bitmap
+import android.graphics.drawable.ColorDrawable
 import android.net.Uri
 import android.os.Build
 import android.os.Bundle
 import android.view.Menu
 import android.view.MenuItem
 import android.view.View
 import androidx.appcompat.content.res.AppCompatResources
-import androidx.core.graphics.drawable.DrawableCompat
-import androidx.core.view.WindowCompat
-import androidx.core.view.WindowInsetsCompat
+import androidx.core.content.ContextCompat
 import com.canhub.cropper.CropImageView
 import com.nextcloud.client.di.Injectable
 import com.nextcloud.client.jobs.upload.FileUploadHelper
@@ -70,20 +69,25 @@ class EditImageActivity :
         file = intent.extras?.getParcelableArgument(EXTRA_FILE, OCFile::class.java)
             ?: throw IllegalArgumentException("Missing file argument")
 
-        setSupportActionBar(binding.toolbar)
-        supportActionBar?.apply {
-            title = file.fileName
-            setDisplayHomeAsUpEnabled(true)
-        }
-
-        val windowInsetsController = WindowCompat.getInsetsController(window, window.decorView)
-        windowInsetsController.hide(WindowInsetsCompat.Type.statusBars())
-
-        window.navigationBarColor = getColor(R.color.black)
+        // NMC Customization
+        setupToolbar()
+        setupActionBar()
 
         setupCropper()
     }
 
+    //NMC Customization
+    private fun setupActionBar() {
+        supportActionBar?.let {
+            viewThemeUtils.platform.themeStatusBar(this)
+            it.setDisplayHomeAsUpEnabled(true)
+            it.setDisplayShowTitleEnabled(true)
+            //custom color for back arrow for NMC
+            viewThemeUtils.files.themeActionBar(this, it, file.fileName)
+            it.setBackgroundDrawable(ColorDrawable(resources.getColor(R.color.bg_default, null)))
+        }
+    }
+
     override fun onCropImageComplete(view: CropImageView, result: CropImageView.CropResult) {
         if (!result.isSuccessful) {
             DisplayUtils.showSnackMessage(this, getString(R.string.image_editor_unable_to_edit_image))
@@ -120,12 +124,18 @@ class EditImageActivity :
     override fun onCreateOptionsMenu(menu: Menu?): Boolean {
         // add save button to action bar
         menuInflater.inflate(R.menu.custom_menu_placeholder, menu)
-        val saveIcon = AppCompatResources.getDrawable(this, R.drawable.ic_check)?.also {
-            DrawableCompat.setTint(it, resources.getColor(R.color.white, theme))
-        }
+        // NMC: No need to apply NC tint here as we will be doing it later in code
+        val saveIcon = AppCompatResources.getDrawable(this, R.drawable.ic_tick)
         menu?.findItem(R.id.custom_menu_placeholder_item)?.apply {
             icon = saveIcon
             contentDescription = getString(R.string.common_save)
+            // NMC customization
+            icon = icon?.let {
+                viewThemeUtils.platform.colorDrawable(
+                    it,
+                    ContextCompat.getColor(this@EditImageActivity, R.color.fontAppbar)
+                )
+            }
         }
         return true
     }

app/src/main/java/com/nextcloud/client/errorhandling/ShowErrorActivity.kt

@@ -36,14 +36,16 @@ class ShowErrorActivity : AppCompatActivity() {
         setSupportActionBar(binding.toolbarInclude.toolbar)
         supportActionBar!!.title = createErrorTitle()
 
-        val snackbar = DisplayUtils.createSnackbar(
+        // NMC-4862
+        // deactivate github report snackbar
+        /* val snackbar = DisplayUtils.createSnackbar(
             binding.errorPageContainer,
             R.string.error_report_issue_text,
             Snackbar.LENGTH_INDEFINITE
         )
             .setAction(R.string.error_report_issue_action) { reportIssue() }
 
-        snackbar.show()
+        snackbar.show() */
     }
 
     private fun createErrorTitle() = String.format(getString(R.string.error_crash_title), getString(R.string.app_name))

app/src/main/java/com/nextcloud/ui/fileactions/FileActionsBottomSheet.kt

@@ -144,7 +144,8 @@ class FileActionsBottomSheet :
                 binding.thumbnailLayout.thumbnailShimmer,
                 syncedFolderProvider.preferences,
                 viewThemeUtils,
-                syncedFolderProvider
+                syncedFolderProvider,
+                false
             )
         }
     }

app/src/main/java/com/nextcloud/ui/trashbinFileActions/TrashbinFileActionsBottomSheet.kt

@@ -129,7 +129,8 @@ class TrashbinFileActionsBottomSheet :
                 binding.thumbnailLayout.thumbnailShimmer,
                 syncedFolderProvider.preferences,
                 viewThemeUtils,
-                syncedFolderProvider
+                syncedFolderProvider,
+                false
             )
         }
     }

app/src/main/java/com/nmc/android/ui/utils/Log_NMC.kt

@@ -0,0 +1,42 @@
+package com.nmc.android.ui.utils
+
+import com.owncloud.android.BuildConfig
+import com.owncloud.android.lib.common.utils.Log_OC
+
+/**
+ * NMC log interpreter class
+ * this class will be used whenever we have to reduce the logs writing
+ * this will avoid printing logs in release builds
+ * todo: can be extended later for more functions
+ */
+object Log_NMC {
+    @JvmStatic
+    fun v(tag: String?, msg: String?) {
+        if (BuildConfig.DEBUG) Log_OC.v(tag, msg)
+    }
+
+    @JvmStatic
+    fun d(tag: String?, msg: String?) {
+        if (BuildConfig.DEBUG) Log_OC.d(tag, msg)
+    }
+
+    @JvmStatic
+    fun d(tag: String, msg: String, e: Exception) {
+        if (BuildConfig.DEBUG) Log_OC.d(tag, msg, e)
+    }
+
+    @JvmStatic
+    fun i(tag: String?, msg: String?) {
+        if (BuildConfig.DEBUG) Log_OC.i(tag, msg)
+    }
+
+    @JvmStatic
+    fun e(tag: String?, msg: String?) {
+        if (BuildConfig.DEBUG) Log_OC.e(tag, msg)
+    }
+
+    @JvmStatic
+    fun w(tag: String?, msg: String?) {
+        if (BuildConfig.DEBUG) Log_OC.w(tag, msg)
+    }
+}

app/src/main/java/com/owncloud/android/MainApp.java

@@ -1002,11 +1002,8 @@ public AndroidInjector<Object> androidInjector() {
     }
 
     public static void setAppTheme(DarkMode mode) {
-        switch (mode) {
-            case LIGHT -> AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_NO);
-            case DARK -> AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_YES);
-            case SYSTEM -> AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_FOLLOW_SYSTEM);
-        }
+        // NMC Customization -> Always follow system theme
+        AppCompatDelegate.setDefaultNightMode(AppCompatDelegate.MODE_NIGHT_FOLLOW_SYSTEM);
     }
 
     @Override

app/src/main/java/com/owncloud/android/datamodel/MediaProvider.java

@@ -12,8 +12,8 @@
 import android.net.Uri;
 import android.provider.MediaStore;
 
+import com.nmc.android.ui.utils.Log_NMC;
 import com.owncloud.android.MainApp;
-import com.owncloud.android.lib.common.utils.Log_OC;
 import com.owncloud.android.utils.PermissionUtil;
 import com.owncloud.android.utils.theme.ViewThemeUtils;
 
@@ -107,7 +107,7 @@ public static List<MediaFolder> getImageFolders(ContentResolver contentResolver,
                                                                    MediaStore.Images.Media.DATE_TAKEN,
                                                                    ContentResolverHelper.SORT_DIRECTION_DESCENDING,
                                                                    itemLimit);
-                Log_OC.d(TAG, "Reading images for " + mediaFolder.folderName);
+                Log_NMC.d(TAG, "Reading images for " + mediaFolder.folderName);
 
                 if (cursorImages != null) {
                     String filePath;
@@ -217,7 +217,7 @@ public static List<MediaFolder> getVideoFolders(ContentResolver contentResolver,
                                                                    ContentResolverHelper.SORT_DIRECTION_DESCENDING,
                                                                    itemLimit);
 
-                Log_OC.d(TAG, "Reading videos for " + mediaFolder.folderName);
+                Log_NMC.d(TAG, "Reading videos for " + mediaFolder.folderName);
 
                 if (cursorVideos != null) {
                     String filePath;

app/src/main/java/com/owncloud/android/datamodel/ThumbnailsCacheManager.java

@@ -38,6 +38,7 @@
 import com.nextcloud.client.network.ConnectivityService;
 import com.nextcloud.utils.BitmapExtensionsKt;
 import com.nextcloud.utils.extensions.ThumbnailsCacheManagerExtensionsKt;
+import com.nmc.android.ui.utils.Log_NMC;
 import com.owncloud.android.MainApp;
 import com.owncloud.android.R;
 import com.owncloud.android.lib.common.OwnCloudAccount;
@@ -133,7 +134,7 @@ public static void initDiskCacheAsync() {
                         }
 
                         String cachePath = cacheDir.getPath() + File.separator + CACHE_FOLDER;
-                        Log_OC.d(TAG, "thumbnail cache dir: " + cachePath);
+                        Log_NMC.d(TAG, "thumbnail cache dir: " + cachePath);
                         File diskCacheDir = new File(cachePath);
 
                         // migrate from external cache to internal cache
@@ -150,7 +151,7 @@ public static void initDiskCacheAsync() {
                         mThumbnailCache = new DiskLruImageCache(diskCacheDir, DISK_CACHE_SIZE, mCompressFormat,
                                                                 mCompressQuality);
                     } catch (Exception e) {
-                        Log_OC.d(TAG, "Disk cache init failed", e);
+                        Log_NMC.d(TAG, "Disk cache init failed", e);
                         mThumbnailCache = null;
                     }
                 }
@@ -789,7 +790,7 @@ private Bitmap doThumbnailFromOCFileInBackground() {
                                         file.getLocalId() + "&x=" + pxW + "&y=" + pxH;
                                 }
 
-                                Log_OC.d(TAG, "generate thumbnail: " + file.getFileName() + " URI: " + uri);
+                                Log_NMC.d(TAG, "generate thumbnail: " + file.getFileName() + " URI: " + uri);
                                 getMethod = new GetMethod(uri);
                                 getMethod.setRequestHeader("Cookie",
                                                            "nc_sameSiteCookielax=true;nc_sameSiteCookiestrict=true");
@@ -822,7 +823,7 @@ private Bitmap doThumbnailFromOCFileInBackground() {
 
                     // Add thumbnail to cache
                     if (thumbnail != null) {
-                        Log_OC.d(TAG, "add thumbnail to cache: " + file.getFileName());
+                        Log_NMC.d(TAG, "add thumbnail to cache: " + file.getFileName());
                         addBitmapToCache(imageKey, thumbnail);
                     }
                 }
@@ -980,7 +981,7 @@ private Bitmap doFileInBackground(File file, Type type) {
                         thumbnail = retriever.getFrameAtTime(-1);
                     } catch (Exception ex) {
                         // can't create a bitmap
-                        Log_OC.w(TAG, "Failed to create bitmap from video " + file.getAbsolutePath());
+                        Log_NMC.w(TAG, "Failed to create bitmap from video " + file.getAbsolutePath());
                     }
 
                     if (thumbnail != null) {
@@ -1087,7 +1088,7 @@ Drawable doAvatarInBackground() {
 
                     int px = mResources.getInteger(R.integer.file_avatar_px);
                     String uri = mClient.getBaseUri() + "/index.php/avatar/" + Uri.encode(mUserId) + "/" + px;
-                    Log_OC.d("Avatar", "URI: " + uri);
+                    Log_NMC.d("Avatar", "URI: " + uri);
                     get = new GetMethod(uri);
 
                     // only use eTag if available and corresponding avatar is still there
@@ -1361,7 +1362,7 @@ public static void generateThumbnailFromOCFile(OCFile file, User user, Context c
             String uri = client.getBaseUri() + "/index.php/apps/files/api/v1/thumbnail/" +
                 pxW + "/" + pxH + Uri.encode(file.getRemotePath(), "/");
 
-            Log_OC.d(TAG, "generate thumbnail: " + file.getFileName() + " URI: " + uri);
+            Log_NMC.d(TAG, "generate thumbnail: " + file.getFileName() + " URI: " + uri);
             getMethod = new GetMethod(uri);
             getMethod.setRequestHeader("Cookie", "nc_sameSiteCookielax=true;nc_sameSiteCookiestrict=true");
 
@@ -1384,7 +1385,7 @@ public static void generateThumbnailFromOCFile(OCFile file, User user, Context c
                     thumbnail = handlePNG(thumbnail, pxW, pxH);
                 }
 
-                Log_OC.d(TAG, "add thumbnail to cache: " + file.getFileName());
+                Log_NMC.d(TAG, "add thumbnail to cache: " + file.getFileName());
                 addBitmapToCache(imageKey, thumbnail);
             }
         } catch (Exception e) {
@@ -1438,7 +1439,7 @@ private static Bitmap doResizedImageInBackground(OCFile file, FileDataStorageMan
                         String uri = mClient.getBaseUri() + "/index.php/core/preview?fileId="
                             + file.getLocalId()
                             + "&x=" + (pxW / 2) + "&y=" + (pxH / 2) + "&a=1&mode=cover&forceIcon=0";
-                        Log_OC.d(TAG, "generate resized image: " + file.getFileName() + " URI: " + uri);
+                        Log_NMC.d(TAG, "generate resized image: " + file.getFileName() + " URI: " + uri);
                         getMethod = new GetMethod(uri);
 
                         int status = mClient.executeMethod(getMethod);
@@ -1456,7 +1457,7 @@ private static Bitmap doResizedImageInBackground(OCFile file, FileDataStorageMan
 
                         // Add thumbnail to cache
                         if (thumbnail != null) {
-                            Log_OC.d(TAG, "add resized image to cache: " + file.getFileName());
+                            Log_NMC.d(TAG, "add resized image to cache: " + file.getFileName());
                             addBitmapToCache(imageKey, thumbnail);
                         }