From f15605a26184cb82c2a95085478e2362e47026a8 Mon Sep 17 00:00:00 2001
From: AgarwalsRahul <rahul.agarwal2019@vitstudent.ac.in>
Date: Wed, 2 Mar 2022 10:41:07 +0530
Subject: [PATCH 1/5] Fix screen rotation issue in permission activity layout

---
 .../main/res/layout/activity_permission.xml   | 66 ++++++++++++-------
 1 file changed, 42 insertions(+), 24 deletions(-)

diff --git a/app/src/main/res/layout/activity_permission.xml b/app/src/main/res/layout/activity_permission.xml
index 2279dd6e5..5ebba0bb5 100644
--- a/app/src/main/res/layout/activity_permission.xml
+++ b/app/src/main/res/layout/activity_permission.xml
@@ -29,33 +29,51 @@
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toBottomOf="@id/appNameText" />
 
-    <code.name.monkey.retromusic.views.PermissionItem
-        android:id="@+id/storagePermission"
-        android:layout_width="0dp"
-        android:layout_height="wrap_content"
+    <ScrollView
+        android:layout_width="match_parent"
+        android:layout_height="0dp"
+        app:layout_constraintBottom_toTopOf="@+id/finish"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toBottomOf="@id/divider"
-        app:permissionButtonTitle="Grant access"
-        app:permissionIcon="@drawable/ic_sd_storage"
-        app:permissionTitle="@string/permission_title"
-        app:permissionTitleNumber="1"
-        app:permissionTitleSubTitle="@string/permission_summary" />
+        android:fillViewport="true"
+        app:layout_constraintTop_toBottomOf="@+id/divider">
+
+       <LinearLayout
+           android:layout_width="match_parent"
+           android:layout_height="match_parent"
+           android:orientation="vertical">
+
+           <code.name.monkey.retromusic.views.PermissionItem
+               android:id="@+id/storagePermission"
+               android:layout_width="match_parent"
+               android:layout_height="wrap_content"
+               app:layout_constraintEnd_toEndOf="parent"
+               app:layout_constraintStart_toStartOf="parent"
+               app:layout_constraintTop_toBottomOf="@id/divider"
+               app:permissionButtonTitle="Grant access"
+               app:permissionIcon="@drawable/ic_sd_storage"
+               app:permissionTitle="@string/permission_title"
+               app:permissionTitleNumber="1"
+               app:permissionTitleSubTitle="@string/permission_summary" />
+
+           <code.name.monkey.retromusic.views.PermissionItem
+               android:id="@+id/audioPermission"
+               android:layout_width="match_parent"
+               android:layout_height="wrap_content"
+               android:visibility="gone"
+               app:layout_constraintEnd_toEndOf="parent"
+               app:layout_constraintStart_toStartOf="parent"
+               app:layout_constraintTop_toBottomOf="@id/storagePermission"
+               app:permissionButtonTitle="Grant access"
+               app:permissionIcon="@drawable/ic_phonelink_ring"
+               app:permissionTitle="@string/ringtone_title"
+               app:permissionTitleNumber="2"
+               app:permissionTitleSubTitle="@string/ringtone_summary"
+               tools:visibility="visible" />
+       </LinearLayout>
+    </ScrollView>
+
 
-    <code.name.monkey.retromusic.views.PermissionItem
-        android:id="@+id/audioPermission"
-        android:layout_width="0dp"
-        android:layout_height="wrap_content"
-        android:visibility="gone"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toBottomOf="@id/storagePermission"
-        app:permissionButtonTitle="Grant access"
-        app:permissionIcon="@drawable/ic_phonelink_ring"
-        app:permissionTitle="@string/ringtone_title"
-        app:permissionTitleNumber="2"
-        app:permissionTitleSubTitle="@string/ringtone_summary"
-        tools:visibility="visible" />
 
     <com.google.android.material.button.MaterialButton
         android:id="@+id/finish"

From 5a2d43fcd73f3f5c382079d48755d6f7563c8e8a Mon Sep 17 00:00:00 2001
From: AgarwalsRahul <rahul.agarwal2019@vitstudent.ac.in>
Date: Thu, 3 Mar 2022 09:12:01 +0530
Subject: [PATCH 2/5] Fix Orientation Issue

---
 .../main/res/layout/activity_permission.xml   | 61 +++++++++----------
 app/src/main/res/values-land/integers.xml     |  5 ++
 app/src/main/res/values-sw600dp/integers.xml  |  2 +
 app/src/main/res/values/integers.xml          |  2 +
 4 files changed, 37 insertions(+), 33 deletions(-)
 create mode 100644 app/src/main/res/values-land/integers.xml

diff --git a/app/src/main/res/layout/activity_permission.xml b/app/src/main/res/layout/activity_permission.xml
index 5ebba0bb5..8b6798acf 100644
--- a/app/src/main/res/layout/activity_permission.xml
+++ b/app/src/main/res/layout/activity_permission.xml
@@ -32,49 +32,44 @@
     <ScrollView
         android:layout_width="match_parent"
         android:layout_height="0dp"
+        android:fillViewport="true"
         app:layout_constraintBottom_toTopOf="@+id/finish"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        android:fillViewport="true"
         app:layout_constraintTop_toBottomOf="@+id/divider">
 
-       <LinearLayout
-           android:layout_width="match_parent"
-           android:layout_height="match_parent"
-           android:orientation="vertical">
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:orientation="@integer/permission_orientation">
 
-           <code.name.monkey.retromusic.views.PermissionItem
-               android:id="@+id/storagePermission"
-               android:layout_width="match_parent"
-               android:layout_height="wrap_content"
-               app:layout_constraintEnd_toEndOf="parent"
-               app:layout_constraintStart_toStartOf="parent"
-               app:layout_constraintTop_toBottomOf="@id/divider"
-               app:permissionButtonTitle="Grant access"
-               app:permissionIcon="@drawable/ic_sd_storage"
-               app:permissionTitle="@string/permission_title"
-               app:permissionTitleNumber="1"
-               app:permissionTitleSubTitle="@string/permission_summary" />
+            <code.name.monkey.retromusic.views.PermissionItem
+                android:id="@+id/storagePermission"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:layout_weight="@integer/permission_layout_weight"
+                app:permissionButtonTitle="Grant access"
+                app:permissionIcon="@drawable/ic_sd_storage"
+                app:permissionTitle="@string/permission_title"
+                app:permissionTitleNumber="1"
+                app:permissionTitleSubTitle="@string/permission_summary" />
 
-           <code.name.monkey.retromusic.views.PermissionItem
-               android:id="@+id/audioPermission"
-               android:layout_width="match_parent"
-               android:layout_height="wrap_content"
-               android:visibility="gone"
-               app:layout_constraintEnd_toEndOf="parent"
-               app:layout_constraintStart_toStartOf="parent"
-               app:layout_constraintTop_toBottomOf="@id/storagePermission"
-               app:permissionButtonTitle="Grant access"
-               app:permissionIcon="@drawable/ic_phonelink_ring"
-               app:permissionTitle="@string/ringtone_title"
-               app:permissionTitleNumber="2"
-               app:permissionTitleSubTitle="@string/ringtone_summary"
-               tools:visibility="visible" />
-       </LinearLayout>
+            <code.name.monkey.retromusic.views.PermissionItem
+                android:id="@+id/audioPermission"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:visibility="gone"
+                android:layout_weight="@integer/permission_layout_weight"
+                app:permissionButtonTitle="Grant access"
+                app:permissionIcon="@drawable/ic_phonelink_ring"
+                app:permissionTitle="@string/ringtone_title"
+                app:permissionTitleNumber="2"
+                app:permissionTitleSubTitle="@string/ringtone_summary"
+                tools:visibility="visible" />
+        </LinearLayout>
     </ScrollView>
 
 
-
     <com.google.android.material.button.MaterialButton
         android:id="@+id/finish"
         style="@style/Widget.Material3.Button"
diff --git a/app/src/main/res/values-land/integers.xml b/app/src/main/res/values-land/integers.xml
new file mode 100644
index 000000000..e56b97117
--- /dev/null
+++ b/app/src/main/res/values-land/integers.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+    <integer name="permission_orientation">0</integer>
+    <integer name="permission_layout_weight">1</integer>
+</resources>
\ No newline at end of file
diff --git a/app/src/main/res/values-sw600dp/integers.xml b/app/src/main/res/values-sw600dp/integers.xml
index 826769fc2..f4fd56db6 100644
--- a/app/src/main/res/values-sw600dp/integers.xml
+++ b/app/src/main/res/values-sw600dp/integers.xml
@@ -8,4 +8,6 @@
 
     <integer name="max_columns">6</integer>
     <integer name="max_columns_land">8</integer>
+    <integer name="permission_orientation">1</integer>
+    <integer name="permission_layout_weight">1</integer>
 </resources>
\ No newline at end of file
diff --git a/app/src/main/res/values/integers.xml b/app/src/main/res/values/integers.xml
index 70dd6eb89..f038ad856 100644
--- a/app/src/main/res/values/integers.xml
+++ b/app/src/main/res/values/integers.xml
@@ -10,4 +10,6 @@
     <integer name="max_columns_land">6</integer>
 
     <integer name="anim_duration_medium">200</integer>
+    <integer name="permission_orientation">1</integer>
+    <integer name="permission_layout_weight">0</integer>
 </resources>
\ No newline at end of file

From 30b1f031ff8ea843a0dced588bad9ea1684e1001 Mon Sep 17 00:00:00 2001
From: cdhiraj40 <chauhandhiraj40@gmail.com>
Date: Thu, 3 Mar 2022 15:19:28 +0530
Subject: [PATCH 3/5] issue#123 added an option to share crash report

---
 app/src/main/AndroidManifest.xml              | 13 ++-
 .../java/code/name/monkey/retromusic/App.kt   |  5 ++
 .../retromusic/activities/ErrorActivity.kt    | 80 +++++++++++++++++++
 .../name/monkey/retromusic/util/FileUtils.kt  | 44 ++++++++++
 .../code/name/monkey/retromusic/util/Share.kt | 14 ++++
 app/src/main/res/values/strings.xml           |  1 +
 6 files changed, 155 insertions(+), 2 deletions(-)
 create mode 100644 app/src/main/java/code/name/monkey/retromusic/activities/ErrorActivity.kt

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 0dbf3e677..0552a7232 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -38,8 +38,8 @@
             android:name=".activities.MainActivity"
             android:exported="true"
             android:label="@string/app_name"
-            android:theme="@style/SplashTheme"
-            android:launchMode="singleTop">
+            android:launchMode="singleTop"
+            android:theme="@style/SplashTheme">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
                 <action android:name="android.intent.action.MUSIC_PLAYER" />
@@ -183,6 +183,15 @@
                 android:value=".activities.MainActivity" />
         </activity>
 
+        <activity
+            android:name=".activities.ErrorActivity"
+            android:exported="true">
+            <intent-filter>
+                <action android:name="cat.ereza.customactivityoncrash.RESTART" />
+            </intent-filter>
+        </activity>
+
+
         <provider
             android:name=".misc.GenericFileProvider"
             android:authorities="${applicationId}.provider"
diff --git a/app/src/main/java/code/name/monkey/retromusic/App.kt b/app/src/main/java/code/name/monkey/retromusic/App.kt
index 65520ca2f..7269a5a69 100644
--- a/app/src/main/java/code/name/monkey/retromusic/App.kt
+++ b/app/src/main/java/code/name/monkey/retromusic/App.kt
@@ -16,9 +16,11 @@ package code.name.monkey.retromusic
 
 import android.app.Application
 import android.widget.Toast
+import cat.ereza.customactivityoncrash.config.CaocConfig
 import code.name.monkey.appthemehelper.ThemeStore
 import code.name.monkey.appthemehelper.util.VersionUtils
 import code.name.monkey.retromusic.Constants.PRO_VERSION_PRODUCT_ID
+import code.name.monkey.retromusic.activities.ErrorActivity
 import code.name.monkey.retromusic.appshortcuts.DynamicShortcutManager
 import code.name.monkey.retromusic.helper.WallpaperAccentManager
 import com.anjlab.android.iab.v3.BillingProcessor
@@ -69,6 +71,9 @@ class App : Application() {
 
                 override fun onBillingInitialized() {}
             })
+
+        // setting Error activity
+        CaocConfig.Builder.create().errorActivity(ErrorActivity::class.java).apply()
     }
 
     override fun onTerminate() {
diff --git a/app/src/main/java/code/name/monkey/retromusic/activities/ErrorActivity.kt b/app/src/main/java/code/name/monkey/retromusic/activities/ErrorActivity.kt
new file mode 100644
index 000000000..409750bcb
--- /dev/null
+++ b/app/src/main/java/code/name/monkey/retromusic/activities/ErrorActivity.kt
@@ -0,0 +1,80 @@
+package code.name.monkey.retromusic.activities
+
+import android.os.Bundle
+import android.widget.Button
+import android.widget.ImageView
+import androidx.appcompat.app.AlertDialog
+import androidx.appcompat.app.AppCompatActivity
+import cat.ereza.customactivityoncrash.CustomActivityOnCrash
+import code.name.monkey.retromusic.R
+import code.name.monkey.retromusic.util.FileUtils.createFile
+import code.name.monkey.retromusic.util.Share.shareFile
+import java.text.DateFormat
+import java.text.SimpleDateFormat
+import java.util.*
+
+class ErrorActivity : AppCompatActivity() {
+    private val dayFormat: DateFormat = SimpleDateFormat("yyyy-MM-dd", Locale.getDefault())
+    private val ReportPrefix = "bug_report-"
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.customactivityoncrash_default_error_activity)
+
+        val restartButton =
+            findViewById<Button>(R.id.customactivityoncrash_error_activity_restart_button)
+
+        val config = CustomActivityOnCrash.getConfigFromIntent(intent)
+        if (config == null) {
+            finish()
+            return
+        }
+        restartButton.setText(R.string.customactivityoncrash_error_activity_restart_app)
+        restartButton.setOnClickListener {
+            CustomActivityOnCrash.restartApplication(
+                this@ErrorActivity,
+                config
+            )
+        }
+        val moreInfoButton =
+            findViewById<Button>(R.id.customactivityoncrash_error_activity_more_info_button)
+
+        moreInfoButton.setOnClickListener { //We retrieve all the error data and show it
+            AlertDialog.Builder(this@ErrorActivity)
+                .setTitle(R.string.customactivityoncrash_error_activity_error_details_title)
+                .setMessage(
+                    CustomActivityOnCrash.getAllErrorDetailsFromIntent(
+                        this@ErrorActivity,
+                        intent
+                    )
+                )
+                .setPositiveButton(
+                    R.string.customactivityoncrash_error_activity_error_details_close,
+                    null
+                )
+                .setNeutralButton(
+                    R.string.customactivityoncrash_error_activity_error_details_share
+                ) { dialog, which ->
+
+                    val bugReport = createFile(
+                        context = this,
+                        "Bug Report",
+                        "$ReportPrefix${dayFormat.format(Date())}",
+                        CustomActivityOnCrash.getAllErrorDetailsFromIntent(
+                            this@ErrorActivity,
+                            intent
+                        ), ".txt"
+                    )
+                    shareFile(this, bugReport)
+                }
+                .show()
+        }
+        val errorActivityDrawableId = config.errorDrawable
+        val errorImageView =
+            findViewById<ImageView>(R.id.customactivityoncrash_error_activity_image)
+        if (errorActivityDrawableId != null) {
+            errorImageView.setImageResource(
+                errorActivityDrawableId
+            )
+        }
+    }
+}
\ No newline at end of file
diff --git a/app/src/main/java/code/name/monkey/retromusic/util/FileUtils.kt b/app/src/main/java/code/name/monkey/retromusic/util/FileUtils.kt
index 8719231b4..e38e006ee 100644
--- a/app/src/main/java/code/name/monkey/retromusic/util/FileUtils.kt
+++ b/app/src/main/java/code/name/monkey/retromusic/util/FileUtils.kt
@@ -2,7 +2,10 @@ package code.name.monkey.retromusic.util
 
 import android.content.Context
 import android.net.Uri
+import android.util.Log
 import java.io.File
+import java.io.IOException
+
 
 object FileUtils {
     fun copyFileToUri(context: Context, fromFile: File, toUri: Uri) {
@@ -13,4 +16,45 @@ object FileUtils {
                 }
             }
     }
+
+    /**
+     * creates a new file in storage in app specific directory.
+     *
+     * @return the file
+     * @throws IOException
+     */
+    fun createFile(context: Context, directoryName: String, fileName: String, body: String, fileType: String): File {
+        val root = createDirectory(context, directoryName)
+        val filePath = "$root/$fileName$fileType"
+        val file = File(filePath)
+
+        // create file if not exist
+        if (!file.exists()) {
+            try {
+                // create a new file and write text in it.
+                file.createNewFile()
+                file.writeText(body)
+                Log.d(FileUtils::class.java.name, "File has been created and saved")
+            } catch (e: IOException) {
+                Log.d(FileUtils::class.java.name, e.message.toString())
+            }
+        }
+        return file
+    }
+
+    /**
+     * creates a new directory in storage in app specific directory.
+     *
+     * @return the file
+     */
+    private fun createDirectory(context: Context, directoryName: String): File {
+        val file = File(
+            context.getExternalFilesDir(directoryName)
+                .toString()
+        )
+        if (!file.exists()) {
+            file.mkdir()
+        }
+        return file
+    }
 }
\ No newline at end of file
diff --git a/app/src/main/java/code/name/monkey/retromusic/util/Share.kt b/app/src/main/java/code/name/monkey/retromusic/util/Share.kt
index 21d3f8ca4..de1b30605 100644
--- a/app/src/main/java/code/name/monkey/retromusic/util/Share.kt
+++ b/app/src/main/java/code/name/monkey/retromusic/util/Share.kt
@@ -18,6 +18,8 @@ import android.content.Context
 import android.content.Intent
 import android.net.Uri
 import androidx.core.app.ActivityCompat
+import androidx.core.content.FileProvider
+import java.io.File
 
 /**
  * Created by hemanths on 2020-02-02.
@@ -30,4 +32,16 @@ object Share {
         feedIntent.putExtra(Intent.EXTRA_STREAM, uri)
         ActivityCompat.startActivity(context, feedIntent, null)
     }
+
+    fun shareFile(context: Context, file: File) {
+        val attachmentUri = FileProvider.getUriForFile(
+            context,
+            context.applicationContext.packageName,
+            file
+        )
+        val sharingIntent = Intent(Intent.ACTION_SEND)
+        sharingIntent.type = "text/*"
+        sharingIntent.putExtra(Intent.EXTRA_STREAM, attachmentUri)
+        context.startActivity(Intent.createChooser(sharingIntent, "send bug report"))
+    }
 }
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index d33bf0cd2..835adf89d 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -541,4 +541,5 @@
     <string name="you_have_to_select_at_least_one_category">You have to select at least one category.</string>
     <string name="you_will_be_forwarded_to_the_issue_tracker_website">You will be forwarded to the issue tracker website.</string>
     <string name="your_account_data_is_only_used_for_authentication">Your account data is only used for authentication.</string>
+    <string name="customactivityoncrash_error_activity_error_details_share">Share Crash Report</string>
 </resources>

From 5678140acf8ba94bc97fafc9988b1ea313ccc36e Mon Sep 17 00:00:00 2001
From: cdhiraj40 <chauhandhiraj40@gmail.com>
Date: Thu, 3 Mar 2022 15:21:17 +0530
Subject: [PATCH 4/5] removed unnecessary changes

---
 app/src/main/AndroidManifest.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 0552a7232..4695c97fc 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -38,8 +38,8 @@
             android:name=".activities.MainActivity"
             android:exported="true"
             android:label="@string/app_name"
-            android:launchMode="singleTop"
-            android:theme="@style/SplashTheme">
+            android:theme="@style/SplashTheme"
+            android:launchMode="singleTop">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
                 <action android:name="android.intent.action.MUSIC_PLAYER" />

From fb2904193a8918accc71f12edc29dc2b1a3e69bb Mon Sep 17 00:00:00 2001
From: AgarwalsRahul <rahul.agarwal2019@vitstudent.ac.in>
Date: Thu, 3 Mar 2022 19:23:04 +0530
Subject: [PATCH 5/5] Solved on back pressed issue

---
 .../name/monkey/retromusic/activities/PermissionActivity.kt  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/src/main/java/code/name/monkey/retromusic/activities/PermissionActivity.kt b/app/src/main/java/code/name/monkey/retromusic/activities/PermissionActivity.kt
index 9f69761ed..a0bad95dd 100644
--- a/app/src/main/java/code/name/monkey/retromusic/activities/PermissionActivity.kt
+++ b/app/src/main/java/code/name/monkey/retromusic/activities/PermissionActivity.kt
@@ -109,4 +109,9 @@ class PermissionActivity : AbsMusicServiceActivity() {
     private fun hasAudioPermission(): Boolean {
         return Settings.System.canWrite(this)
     }
+
+    override fun onBackPressed() {
+        super.onBackPressed()
+        finishAffinity()
+    }
 }