From 242a82dcf1c218dc06ad16f0e33e287050b9a2d8 Mon Sep 17 00:00:00 2001
From: DrDisagree <crazymahmud08@gmail.com>
Date: Wed, 4 Sep 2024 22:56:45 +0600
Subject: [PATCH] Add a few checks to avoid crashing

Signed-off-by: DrDisagree <crazymahmud08@gmail.com>
---
 .../iconify/xposed/modules/HeaderClockA14.kt       | 14 ++++++++------
 .../iconify/xposed/modules/OpQsHeader.kt           | 10 ++++------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/app/src/main/java/com/drdisagree/iconify/xposed/modules/HeaderClockA14.kt b/app/src/main/java/com/drdisagree/iconify/xposed/modules/HeaderClockA14.kt
index d6f8f72f7..c237c6e99 100644
--- a/app/src/main/java/com/drdisagree/iconify/xposed/modules/HeaderClockA14.kt
+++ b/app/src/main/java/com/drdisagree/iconify/xposed/modules/HeaderClockA14.kt
@@ -294,10 +294,9 @@ class HeaderClockA14(context: Context?) : ModPack(context!!) {
                     addView(mQsIconsContainer)
                 }
 
-                mQuickStatusBarHeader!!.addView(
-                    mQsHeaderContainer,
-                    -1
-                )
+
+                (mQsHeaderContainer.parent as? ViewGroup)?.removeView(mQsHeaderContainer)
+                mQuickStatusBarHeader!!.addView(mQsHeaderContainer, -1)
 
                 handleOldHeaderView(param)
 
@@ -395,7 +394,7 @@ class HeaderClockA14(context: Context?) : ModPack(context!!) {
                                 "switchToParent",
                                 view,
                                 parent,
-                                index,
+                                index.coerceAtMost(parent.childCount),
                                 tag
                             )
 
@@ -450,7 +449,10 @@ class HeaderClockA14(context: Context?) : ModPack(context!!) {
                                 mQsHeaderContainerShadeParent.indexOfChild(mQsHeaderContainerShade) != index
                             ) {
                                 mQsHeaderContainerShadeParent?.removeView(mQsHeaderContainerShade)
-                                parent.addView(mQsHeaderContainerShade, index)
+                                parent.addView(
+                                    mQsHeaderContainerShade,
+                                    index.coerceAtMost(parent.childCount)
+                                )
                             }
                         }
 
diff --git a/app/src/main/java/com/drdisagree/iconify/xposed/modules/OpQsHeader.kt b/app/src/main/java/com/drdisagree/iconify/xposed/modules/OpQsHeader.kt
index 8625d8f94..e4d76e709 100644
--- a/app/src/main/java/com/drdisagree/iconify/xposed/modules/OpQsHeader.kt
+++ b/app/src/main/java/com/drdisagree/iconify/xposed/modules/OpQsHeader.kt
@@ -408,10 +408,8 @@ class OpQsHeader(context: Context?) : ModPack(context!!) {
                     orientation = LinearLayout.VERTICAL
                 }
 
-                mQuickStatusBarHeader!!.addView(
-                    mQsHeaderContainer,
-                    -1
-                )
+                (mQsHeaderContainer.parent as? ViewGroup)?.removeView(mQsHeaderContainer)
+                mQuickStatusBarHeader!!.addView(mQsHeaderContainer, -1)
 
                 val relativeLayout = RelativeLayout(mContext).apply {
                     layoutParams = RelativeLayout.LayoutParams(
@@ -562,7 +560,7 @@ class OpQsHeader(context: Context?) : ModPack(context!!) {
                                 "switchToParent",
                                 view,
                                 parent,
-                                index,
+                                index.coerceAtMost(parent.childCount),
                                 tag
                             )
 
@@ -621,7 +619,7 @@ class OpQsHeader(context: Context?) : ModPack(context!!) {
                             }
                         }
 
-                        param.args[2] = (param.args[2] as Int) + 1
+                        param.args[2] = ((param.args[2] as Int) + 1).coerceAtMost(parent.childCount)
                     }
                 }
             )