refactor(Spotify): Add extensions debug logging (#5110)

extensions/spotify/build.gradle.kts

@@ -6,7 +6,7 @@ dependencies {
 
 android {
     defaultConfig {
-        minSdk = 24
+        minSdk = 21
     }
 
     compileOptions {

extensions/spotify/src/main/java/app/revanced/extension/spotify/layout/hide/createbutton/HideCreateButtonPatch.java

@@ -2,6 +2,7 @@ package app.revanced.extension.spotify.layout.hide.createbutton;
 
 import java.util.List;
 
+import app.revanced.extension.shared.Logger;
 import app.revanced.extension.shared.Utils;
 
 @SuppressWarnings("unused")
@@ -31,10 +32,21 @@ public final class HideCreateButtonPatch {
         }
 
         String stringifiedNavigationBarItem = navigationBarItem.toString();
-        boolean isCreateButton = CREATE_BUTTON_TITLE_RES_ID_LIST.stream()
+
-                .anyMatch(stringifiedNavigationBarItem::contains);
+        boolean isCreateButton = false;
+        String matchedTitleResId = null;
+
+        for (String titleResId : CREATE_BUTTON_TITLE_RES_ID_LIST) {
+            if (stringifiedNavigationBarItem.contains(titleResId)) {
+                isCreateButton = true;
+                matchedTitleResId = titleResId;
+            }
+        }
 
         if (isCreateButton) {
+            String finalMatchedTitleResId = matchedTitleResId;
+            Logger.printInfo(() -> "Hiding Create button because the navigation bar item " + navigationBarItem +
+                    " matched the title resource id " + finalMatchedTitleResId);
             return null;
         }
 
@@ -46,6 +58,14 @@ public final class HideCreateButtonPatch {
      * Create button.
      */
     public static boolean isOldCreateButton(int oldNavigationBarItemTitleResId) {
-        return oldNavigationBarItemTitleResId == OLD_CREATE_BUTTON_TITLE_RES_ID;
+        boolean isCreateButton = oldNavigationBarItemTitleResId == OLD_CREATE_BUTTON_TITLE_RES_ID;
+
+        if (isCreateButton) {
+            Logger.printInfo(() -> "Hiding old Create button because the navigation bar item title resource id" +
+                    " matched " + OLD_CREATE_BUTTON_TITLE_RES_ID);
+            return true;
+        }
+
+        return false;
     }
 }

extensions/spotify/src/main/java/app/revanced/extension/spotify/misc/privacy/SanitizeSharingLinksPatch.java

@@ -33,10 +33,11 @@ public final class SanitizeSharingLinksPatch {
                 }
             }
 
-            return builder.build().toString();
+            String sanitizedUrl = builder.build().toString();
+            Logger.printInfo(() -> "Sanitized url " + url + " to " + sanitizedUrl);
+            return sanitizedUrl;
         } catch (Exception ex) {
-            Logger.printException(() -> "sanitizeUrl failure", ex);
+            Logger.printException(() -> "sanitizeUrl failure with " + url, ex);
-
             return url;
         }
     }

extensions/spotify/stub/build.gradle.kts

@@ -7,7 +7,7 @@ android {
     compileSdk = 34
 
     defaultConfig {
-        minSdk = 24
+        minSdk = 21
     }
 
     compileOptions {

patches/src/main/kotlin/app/revanced/patches/spotify/layout/hide/createbutton/HideCreateButtonPatch.kt

@@ -72,7 +72,7 @@ val hideCreateButtonPatch = bytecodePatch(
 
         if (oldNavigationBarAddItemMethod != null) {
             // In case an older version of the app is being patched, hook the old method which adds navigation bar items.
-            // Return null early if the navigation bar item title resource id is old Create button title resource id.
+            // Return null early if the navigation bar item title resource id is the old Create button title resource id.
             oldNavigationBarAddItemFingerprint.methodOrNull?.apply {
                 val getNavigationBarItemTitleStringIndex = indexOfFirstInstructionOrThrow {
                     val reference = getReference<MethodReference>()

patches/src/main/kotlin/app/revanced/patches/spotify/layout/theme/CustomThemePatch.kt

@@ -129,7 +129,7 @@ val customThemePatch = resourcePatch(
     val accentColorPressed by stringOption(
         key = "accentColorPressed",
         default = "#FF1ABC54",
-        title = "Pressed dark theme accent color",
+        title = "Pressed accent color",
         description = "The color when accented buttons are pressed, by default slightly darker than accent. " +
                 "Can be a hex color or a resource reference.",
         required = true,

patches/src/main/kotlin/app/revanced/patches/spotify/misc/fix/login/FixFacebookLoginPatch.kt

@@ -15,7 +15,7 @@ val fixFacebookLoginPatch = bytecodePatch(
         // The Facebook SDK tries to handle the login using the Facebook app in case it is installed.
         // However, the Facebook app does signature checks with the app that is requesting the authentication,
         // which ends up making the Facebook server reject with an invalid key hash for the app signature.
-        // Override the Faceboook SDK to always handle the login using the web browser, which does not perform
+        // Override the Facebook SDK to always handle the login using the web browser, which does not perform
         // signature checks.
 
         val katanaProxyLoginMethodHandlerClass = katanaProxyLoginMethodHandlerClassFingerprint.originalClassDef

patches/src/main/kotlin/app/revanced/patches/spotify/misc/widgets/Fingerprints.kt

@@ -1,7 +1,6 @@
 package app.revanced.patches.spotify.misc.widgets
 
 import app.revanced.patcher.fingerprint
-import app.revanced.util.indexOfFirstInstruction
 import com.android.tools.smali.dexlib2.Opcode
 
 internal val canBindAppWidgetPermissionFingerprint = fingerprint {