fix lint errors

update to 2.4.1

app/build.gradle

@@ -25,8 +25,8 @@ android {
         minSdkVersion 23
         compileSdk 34
         targetSdkVersion 34
-        versionCode 300 // last digit is reserved for ABI, only ever end on 0!
-        versionName '2.4.0'
+        versionCode 310 // last digit is reserved for ABI, only ever end on 0!
+        versionName '2.4.1'
         resourceConfigurations += ['en', 'de']
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         buildConfigField "java.lang.String", "CLI", System.getenv('RCX_BUILD_CLI') ? System.getenv('RCX_BUILD_CLI') : "\"c03129b6-b09f-9cb4-8fcd-7f143b8f94ef\""
@@ -149,7 +149,7 @@ dependencies {
     implementation 'androidx.constraintlayout:constraintlayout:2.1.4'
     implementation 'androidx.browser:browser:1.6.0'
     implementation 'androidx.legacy:legacy-support-v4:1.0.0'
-    implementation 'androidx.recyclerview:recyclerview:1.3.1'
+    implementation 'androidx.recyclerview:recyclerview:1.3.2'
     implementation 'com.github.leinardi:FloatingActionButtonSpeedDial:3.1.1'
     implementation 'org.markdownj:markdownj-core:0.4'
     implementation 'jp.wasabeef:recyclerview-animators:4.0.2'

app/src/main/java/ca/pkay/rcloneexplorer/RemoteConfig/RemoteConfig.kt

@@ -153,6 +153,7 @@ class RemoteConfig : AppCompatActivity(), ProviderSelectedListener {
     }
 
     override fun onBackPressed() {
+        super.onBackPressed()
         handleBackAction()
     }
 

app/src/main/java/ca/pkay/rcloneexplorer/notifications/AppErrorNotificationManager.kt

@@ -1,15 +1,18 @@
 package ca.pkay.rcloneexplorer.notifications
 
+import android.annotation.SuppressLint
 import android.app.NotificationChannel
 import android.app.NotificationManager
 import android.app.PendingIntent
 import android.app.PendingIntent.FLAG_IMMUTABLE
 import android.content.Context
 import android.os.Build
+import android.util.Log
 import androidx.core.app.NotificationCompat
 import androidx.core.app.NotificationManagerCompat
 import ca.pkay.rcloneexplorer.R
 import ca.pkay.rcloneexplorer.util.PermissionManager
+import ca.pkay.rcloneexplorer.util.SyncLog
 
 
 class AppErrorNotificationManager(var mContext: Context) {
@@ -38,6 +41,7 @@ class AppErrorNotificationManager(var mContext: Context) {
         }
     }
 
+    @SuppressLint("MissingPermission")
     fun showNotification() {
 
 
@@ -59,6 +63,11 @@ class AppErrorNotificationManager(var mContext: Context) {
             .setOnlyAlertOnce(true)
 
         val notificationManager = NotificationManagerCompat.from(mContext)
-        notificationManager.notify(APP_ERROR_ID, b.build())
+
+        if(PermissionManager(mContext).grantedNotifications()) {
+            notificationManager.notify(APP_ERROR_ID, b.build())
+        } else {
+            Log.e("AppErrorNotificationManager", "We dont have Notification Permission!")
+        }
     }
 }

app/src/main/java/ca/pkay/rcloneexplorer/notifications/GenericSyncNotification.kt

@@ -66,12 +66,8 @@ class GenericSyncNotification(var mContext: Context) {
     }
 
     companion object {
-        public fun  getFlags(): Int {
-            var flags = 0
-            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
-                flags = FLAG_IMMUTABLE
-            }
-            return flags
+        fun  getFlags(): Int {
+            return FLAG_IMMUTABLE
         }
     }
 }

app/src/main/java/ca/pkay/rcloneexplorer/notifications/support/GenericNotification.kt

@@ -1,10 +1,14 @@
 package ca.pkay.rcloneexplorer.notifications.support
 
+import android.annotation.SuppressLint
 import android.content.Context
+import android.util.Log
 import androidx.core.app.NotificationCompat
 import androidx.core.app.NotificationManagerCompat
 import ca.pkay.rcloneexplorer.R
+import ca.pkay.rcloneexplorer.notifications.AppErrorNotificationManager
 import ca.pkay.rcloneexplorer.notifications.GenericSyncNotification
+import ca.pkay.rcloneexplorer.util.PermissionManager
 
 abstract class GenericNotification(var mContext: Context) {
 
@@ -132,8 +136,14 @@ abstract class GenericNotification(var mContext: Context) {
         notificationManagerCompat.cancel(this.persistentId)
     }
 
+    @SuppressLint("MissingPermission")
     private fun notify(builder: NotificationCompat.Builder, id: Int) {
         val notificationManager = NotificationManagerCompat.from(mContext)
-        notificationManager.notify(id, builder.build())
+
+        if(PermissionManager(mContext).grantedNotifications()) {
+            notificationManager.notify(id, builder.build())
+        } else {
+            Log.e("GenericNotification", "We dont have Notification Permission!")
+        }
     }
 }

app/src/main/res/layout-sw720dp/fragment_config_list.xml

@@ -24,8 +24,10 @@
         android:layout_marginEnd="32dp"
         android:layout_marginBottom="32dp"
         android:clickable="true"
+        android:focusable="true"
         android:src="@drawable/ic_appintro_arrow"
         android:visibility="invisible"
+        android:contentDescription="@string/remote_creation_next_step"
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintEnd_toEndOf="parent" />
 

app/src/main/res/layout-sw720dp/remote_config_form.xml

@@ -94,6 +94,7 @@
         android:layout_marginEnd="32dp"
         android:layout_marginBottom="32dp"
         android:clickable="true"
+        android:focusable="true"
         android:contentDescription="@string/save"
         android:src="@drawable/ic_twotone_save_24"
         app:layout_constraintBottom_toBottomOf="parent"

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

@@ -9,6 +9,7 @@
 
     <ImageView
         android:id="@+id/icon"
+        android:contentDescription="@string/breadcrumb_home_description"
         android:layout_width="20dp"
         android:layout_height="match_parent"
         android:layout_gravity="center_vertical"
@@ -24,14 +25,14 @@
         android:layout_height="match_parent"
         android:ellipsize="end"
         android:gravity="center_vertical"
-        android:singleLine="true"
-        android:text="TextView" />
+        android:singleLine="true" />
 
     <ImageView
         android:id="@+id/arrow"
         android:layout_width="20dp"
         android:layout_height="match_parent"
         android:layout_marginStart="2dp"
         android:layout_marginEnd="2dp"
+        android:contentDescription="@string/breadcrumb_next_description"
         android:src="@drawable/ic_arrow_right" />
 </LinearLayout>

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

@@ -19,10 +19,11 @@
         android:layout_marginEnd="32dp"
         android:layout_marginBottom="32dp"
         android:clickable="true"
+        android:focusable="true"
         android:src="@drawable/ic_appintro_arrow"
         android:visibility="invisible"
+        android:contentDescription="@string/remote_creation_next_step"
         app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintEnd_toEndOf="parent"
-        android:focusable="true" />
+        app:layout_constraintEnd_toEndOf="parent" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>

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

@@ -91,6 +91,7 @@
             android:layout_marginEnd="32dp"
             android:layout_marginBottom="32dp"
             android:clickable="true"
+            android:focusable="true"
             android:contentDescription="@string/save"
             android:src="@drawable/ic_twotone_save_24"
             app:layout_constraintBottom_toBottomOf="parent"

app/src/main/res/menu/remote_config_menu.xml

@@ -7,6 +7,6 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:icon="@drawable/ic_search_black_24dp"
-        android:title="Search"
+        android:title="@string/search"
         app:showAsAction="always" />
 </menu>

app/src/main/res/values-de/strings.xml

@@ -138,6 +138,9 @@
     <string name="operation_failed">Synchronisation fehlgeschlagen</string>
     <string name="operation_failed_unknown">%1$s schlug aufgrund eines unbekannten Fehlers fehl.</string>
     <string name="operation_failed_data_change">Wir konnten %1$s nicht synchronisieren da die Internetverbindung geändert worden ist und Mobile Daten nicht erlaubt sind.</string>
+    <string name="operation_failed_unknown_rclone_error">%1$s konnte wegen eines unbekannten Rclone-Fehlers nicht synchronisiert werden</string>
+    <string name="operation_failed_cancelled">Sync wurde abgebrochen</string>
+    <string name="operation_failed_notask">Es gab keine Aufgabe zum Starten!</string>
     <string name="moving_service">Verschieben-Dienst</string>
     <string name="delete_service">Lösch-Dienst</string>
     <string name="syncing_service">Synchronisiere %1$s</string>
@@ -411,6 +414,7 @@
     <string name="task_hint_wifi_ignore_defaults">Wenn eingeschaltet überschreibt diese die globale Einstellung bezüglich des Datentransfers über WLan</string>
     <string name="task_hint_md5checksum">Dies nutzt Prüfsummen anstelle der Dateigröße. Das kann länger dauern und mehr Batterie verbrauchen.</string>
     <string name="shortcut_start_service">Startete Sync</string>
+    <string name="shortcut_missing_id">Konnte Sync nicht starten</string>
     <string name="task_copy_suffix">(Kopie)</string>
     <string name="trigger_copy_suffix">(Kopie)</string>
     <string name="trigger_label_weekdays">Wochentage</string>
@@ -444,12 +448,38 @@
     <string name="description_sync_direction_copy_tolocal">Kopiere neue oder geänderte Dateien vom entfernten Speicher zum lokalen Speicher. Dateien werden nicht gelöscht, bestehende Dateien auf dem lokalen Speicher können überschrieben werden.</string>
     <string name="description_sync_direction_sync_bidirectional">Stellt sicher das lokaler und entfernter Speicher identisch sind. Dateien werden gelöscht, geupdated und erstellt nach Bedarf.</string>
     <string name="intro_success">Erfolg!</string>
-    <string name="intro_manage_external_storage_granted">Dateizugriff erlaubt!</string>
     <string name="dynamic_config_unknown_error">Der Anbieter ist unbekannt!</string>
     <string name="toggle_advanced_options">Erweiterte Optionen umschalten</string>
     <string name="intro_notifications_description">Wir wollen dir beim Sync den Sync-Fortschritt anzeigen, bitte erlaube uns dir Benachrichtigungen anzuzeigen. Wir werden dich nicht zuspammen!</string>
     <string name="error_getting_config">Wir konnten die Konfiguration nicht laden-</string>
     <string name="notification_reports">Erstelle Berichte für Syncronisationen. Dies ersetzt einzelne Benachrichtigungen für Syncs durch eine Zusammenfassung.</string>
     <string name="back">Zurück</string>
     <string name="save">Speichern</string>
+    <string name="dynamic_config_suffixselector_suffix_hint">Größen-Suffix</string>
+    <string name="dynamic_config_suffixselector_value_hint">Wert</string>
+    <string name="operation_no_identical_title">Sync abgebrochen</string>
+    <string name="operation_no_identical">Du kannst eine Aufgabe nicht mehrmals gleichzeitig starten: %s</string>
+    <string name="intro_permission_changed_title">Berechtigungen benötigt!</string>
+    <string name="intro_permission_changed_description">Entschuldigung, aber wir müssen nur kurz die Berechtigungen aktualisieren.</string>
+    <string name="intro_alarms_title">Alarme!</string>
+    <string name="intro_alarms_description">Wir möchten in der Lage sein Alarme einzurichten um Synchronisationen zu starten. Keine Sorge, wir klingeln nicht durch!</string>
+    <string name="operation_start_sync">Synchronisazion gestartet!</string>
+    <string name="operation_sync_cancelled">Synchronisazion gestoppt!</string>
+    <string name="intro_notifications_denied">Wir benachrichtigen dich nicht.</string>
+    <string name="intro_alarms_denied">Wir können jetzt keine Alarme einrichten</string>
+    <string name="app_error_notification_channel_name">App-Fehler</string>
+    <string name="app_error_notification_channel_description">Fehler die spezifisch für diese App sind erscheinen in diesem Kanal</string>
+    <string name="permissionFragment">Berechtigungen</string>
+    <string name="permissions_notifications_label">Benachrichtigungen</string>
+    <string name="permissions_notifications_description">Benachrichtigungen sind nützlich darin dich über den Zustand der App zu informieren. Du solltest sie einschalten, sodass du immer informiert bist!</string>
+    <string name="permissions_alarms_label">Alarme</string>
+    <string name="permissions_alarms_description">Wir müssen Alarme und Wecker anlegen, sodass die Synchronisationen pünktlich starten. Wenn diese Berechtigungen nicht gegeben ist, werden Synchronisationen nicht funktionieren.</string>
+    <string name="permissions_storage_label">Speicher</string>
+    <string name="permissions_storage_description">Diese Berechtigung erlaubt es uns deine Dateien zu verwalten. Auch ohne kannst du weiterhin auf die entfernten Speicher zugreifen, wir empfehlen aber dies trotzdem zu erlauben.</string>
+    <string name="permissions_button_request">Erlaube!</string>
+    <string name="app_error_notification_alarmpermission_missing">Alarm und Wecker-Berechtigung fehlt</string>
+    <string name="app_error_notification_alarmpermission_missing_description">Bitte erlaube uns Alarme festzulegen. Ohne diese Berechtigung werden Synchronisationen nicht starten!</string>
+    <string name="breadcrumb_home_description">Remote-Startpunkt</string>
+    <string name="breadcrumb_next_description">Nächster Breadcrumb-Pfeil-Indikator</string>
+    <string name="remote_creation_next_step">Nächster Schritt</string>
 </resources>

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

@@ -544,4 +544,7 @@
     <string name="permissions_button_request">Request!</string>
     <string name="app_error_notification_alarmpermission_missing">Alarm Scheduling Permission missing</string>
     <string name="app_error_notification_alarmpermission_missing_description">Please grant this permission to allow schedules to be started. Without it, syncs wont run!</string>
+    <string name="breadcrumb_home_description">Remote-Root</string>
+    <string name="breadcrumb_next_description">Next breadcrumb arrow indicator</string>
+    <string name="remote_creation_next_step">Next Step</string>
 </resources>