Fix some lint errors

SecUSo · Apr 8, 2024 · 53bad09 · 53bad09
1 parent d5b1536
commit 53bad09
Show file tree

Hide file tree

Showing 8 changed files with 46 additions and 37 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -23,7 +23,7 @@ android {
     }
     namespace 'org.secuso.privacyfriendlypaindiary'
     lint {
-        abortOnError false
+        lintConfig = file("lint.xml")
     }
 
     compileOptions {

diff --git a/app/lint.xml b/app/lint.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<lint>
+    <!-- Disable the NotificationPermission check for glide -->
+    <issue id="NotificationPermission">
+        <ignore regexp="com.bumptech.glide.request.target.NotificationTarget" />
+    </issue>
+
+    <!-- Set the severity of missing translations to warning instead of error -->
+    <issue id="MissingTranslation" severity="warning" />
+</lint>
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -2,6 +2,9 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools">
 
+    <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
+    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
+
     <supports-screens
         android:anyDensity="true"
         android:largeScreens="true"
@@ -134,9 +137,4 @@
         </provider>
 
     </application>
-
-    <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
-
-
 </manifest>
diff --git a/app/src/main/java/org/secuso/privacyfriendlypaindiary/activities/DiaryEntryActivity.java b/app/src/main/java/org/secuso/privacyfriendlypaindiary/activities/DiaryEntryActivity.java
@@ -16,6 +16,7 @@ License, or (at your option) any later version.
 */
 package org.secuso.privacyfriendlypaindiary.activities;
 
+import android.annotation.SuppressLint;
 import android.app.AlertDialog;
 import android.app.FragmentManager;
 import android.content.DialogInterface;
@@ -1156,6 +1157,7 @@ public boolean onOptionsItemSelected(MenuItem item) {
     }
 
 
+    @SuppressLint("MissingSuperCall")
     @Override
     public void onBackPressed() {
         if (changesMade) {

diff --git a/app/src/main/res/layout-land/diaryentry_slide3.xml b/app/src/main/res/layout-land/diaryentry_slide3.xml
@@ -78,9 +78,9 @@
                     android:layout_width="wrap_content"
                     android:layout_height="match_parent"
                     android:adjustViewBounds="true"
-                    android:tint="@color/yellow"
                     app:srcCompat="@drawable/paindiary_person_fullbody"
-                    android:visibility="gone"/>
+                    android:visibility="gone"
+                    app:tint="@color/yellow" />
 
             </RelativeLayout>
 
@@ -109,9 +109,9 @@
                     android:layout_width="wrap_content"
                     android:layout_height="match_parent"
                     android:adjustViewBounds="true"
-                    android:tint="@color/yellow"
                     app:srcCompat="@drawable/paindiary_person_fullbody"
-                    android:visibility="gone"/>
+                    android:visibility="gone"
+                    app:tint="@color/yellow" />
 
             </RelativeLayout>
 

diff --git a/app/src/main/res/layout/diaryentry_slide3.xml b/app/src/main/res/layout/diaryentry_slide3.xml
@@ -63,9 +63,9 @@
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:adjustViewBounds="true"
-                    android:tint="@color/yellow"
                     app:srcCompat="@drawable/paindiary_person_fullbody"
-                    android:visibility="gone"/>
+                    android:visibility="gone"
+                    app:tint="@color/yellow" />
 
             </RelativeLayout>
 
@@ -100,9 +100,9 @@
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:adjustViewBounds="true"
-                    android:tint="@color/yellow"
                     app:srcCompat="@drawable/paindiary_person_fullbody"
-                    android:visibility="gone"/>
+                    android:visibility="gone"
+                    app:tint="@color/yellow" />
 
             </RelativeLayout>
 

diff --git a/app/src/main/res/layout/diaryentry_summary.xml b/app/src/main/res/layout/diaryentry_summary.xml
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<ScrollView
+<ScrollView xmlns:app="http://schemas.android.com/apk/res-auto"
     android:id="@+id/diaryentry"
     xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
@@ -170,9 +170,9 @@
                         android:layout_width="90dp"
                         android:layout_height="220dp"
                         android:scaleType="centerCrop"
-                        android:tint="@color/yellow"
                         app:srcCompat="@drawable/paindiary_person_fullbody"
-                        android:visibility="gone" />
+                        android:visibility="gone"
+                        app:tint="@color/yellow" />
 
                 </RelativeLayout>
 
@@ -193,9 +193,9 @@
                         android:layout_width="90dp"
                         android:layout_height="220dp"
                         android:scaleType="centerCrop"
-                        android:tint="@color/yellow"
                         app:srcCompat="@drawable/paindiary_person_fullbody"
-                        android:visibility="gone" />
+                        android:visibility="gone"
+                        app:tint="@color/yellow" />
                 </RelativeLayout>
 
             </LinearLayout>

diff --git a/app/src/main/res/layout/pdf_person.xml b/app/src/main/res/layout/pdf_person.xml
@@ -1,26 +1,25 @@
 <?xml version="1.0" encoding="utf-8"?>
-<RelativeLayout
-    android:id="@+id/bodyregion"
-    xmlns:android="http://schemas.android.com/apk/res/android"
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:id="@+id/bodyregion"
     android:layout_width="wrap_content"
     android:layout_height="wrap_content"
-    android:orientation="vertical" >
+    android:orientation="vertical">
 
-        <ImageView
-            android:id="@+id/person"
-            android:layout_width="100dp"
-            android:layout_height="200dp"
-            android:scaleType="centerCrop"
-            app:srcCompat="@drawable/paindiary_person_fullbody" />
+    <ImageView
+        android:id="@+id/person"
+        android:layout_width="100dp"
+        android:layout_height="200dp"
+        android:scaleType="centerCrop"
+        app:srcCompat="@drawable/paindiary_person_fullbody" />
 
-        <ImageView
-            android:id="@+id/bodyregion_value"
-            android:layout_width="100dp"
-            android:layout_height="200dp"
-            android:scaleType="centerCrop"
-            android:tint="@color/yellow"
-            app:srcCompat="@drawable/paindiary_person_fullbody"
-            android:visibility="gone" />
+    <ImageView
+        android:id="@+id/bodyregion_value"
+        android:layout_width="100dp"
+        android:layout_height="200dp"
+        android:scaleType="centerCrop"
+        android:visibility="gone"
+        app:srcCompat="@drawable/paindiary_person_fullbody"
+        app:tint="@color/yellow" />
 
 </RelativeLayout>