From 393bc9f5f7e04d4570c372c19417ca81db79eb13 Mon Sep 17 00:00:00 2001 From: Ave Date: Wed, 29 Jul 2020 23:34:57 +0300 Subject: [PATCH] no-mobileindicator: Finally fix no-mobileindicator Closes #72 --- patches/disable-mobileindicator/1208.patch | 98 +++++++++++++++++++--- patches/disable-mobileindicator/README.md | 7 +- 2 files changed, 90 insertions(+), 15 deletions(-) diff --git a/patches/disable-mobileindicator/1208.patch b/patches/disable-mobileindicator/1208.patch index 5eee56a4f6..b251b0fc9b 100644 --- a/patches/disable-mobileindicator/1208.patch +++ b/patches/disable-mobileindicator/1208.patch @@ -1,20 +1,96 @@ -diff -crB from/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali to/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali -*** from/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali 2020-06-20 04:18:51.683371812 +0100 ---- to/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali 2020-06-20 04:19:01.317301433 +0100 +Only in com.discord: build +Only in com.discord: dist +diff --color -crB com.discord-base/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali com.discord/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali +*** com.discord-base/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali 2020-07-29 21:27:28.163447492 +0300 +--- com.discord/smali/com/discord/utilities/analytics/AnalyticSuperProperties.smali 2020-07-29 23:32:55.123412443 +0300 *************** -*** 176,182 **** +*** 317,387 **** + .end method - const-string v2, "browser" + .method private final setSuperProperties(Ljava/util/Map;)V +! .locals 3 +! .annotation system Ldalvik/annotation/Signature; +! value = { +! "(", +! "Ljava/util/Map<", +! "Ljava/lang/String;", +! "+", +! "Ljava/lang/Object;", +! ">;)V" +! } +! .end annotation +! +! sget-object v0, Lcom/discord/utilities/analytics/AnalyticSuperProperties;->superProperties$delegate:Lkotlin/properties/ReadWriteProperty; +! +! sget-object v1, Lcom/discord/utilities/analytics/AnalyticSuperProperties;->$$delegatedProperties:[Lkotlin/reflect/KProperty; +! +! const/4 v2, 0x0 +! +! aget-object v1, v1, v2 +! +! invoke-interface {v0, p0, v1, p1}, Lkotlin/properties/ReadWriteProperty;->setValue(Ljava/lang/Object;Lkotlin/reflect/KProperty;Ljava/lang/Object;)V -! const-string v3, "Discord Android" + return-void + .end method - invoke-direct {v1, v2, v3}, Lkotlin/Pair;->(Ljava/lang/Object;Ljava/lang/Object;)V + .method private final declared-synchronized updateSuperProperties(Ljava/util/Map;)V +! .locals 1 +! .annotation system Ldalvik/annotation/Signature; +! value = { +! "(", +! "Ljava/util/Map<", +! "Ljava/lang/String;", +! "+", +! "Ljava/lang/Object;", +! ">;)V" +! } +! .end annotation +! +! monitor-enter p0 +! +! .line 1 +! :try_start_0 +! invoke-virtual {p0}, Lcom/discord/utilities/analytics/AnalyticSuperProperties;->getSuperProperties()Ljava/util/Map; +! +! move-result-object v0 +! +! invoke-static {v0, p1}, Lf/n/a/k/a;->plus(Ljava/util/Map;Ljava/util/Map;)Ljava/util/Map; +! +! move-result-object p1 +! +! invoke-direct {p0, p1}, Lcom/discord/utilities/analytics/AnalyticSuperProperties;->setSuperProperties(Ljava/util/Map;)V +! :try_end_0 +! .catchall {:try_start_0 .. :try_end_0} :catchall_0 +! +! .line 2 +! monitor-exit p0 ---- 176,182 ---- + return-void +- +- :catchall_0 +- move-exception p1 +- +- monitor-exit p0 +- +- throw p1 + .end method - const-string v2, "browser" -! const-string v3, "Discord Client" +--- 317,331 ---- + .end method - invoke-direct {v1, v2, v3}, Lkotlin/Pair;->(Ljava/lang/Object;Ljava/lang/Object;)V + .method private final setSuperProperties(Ljava/util/Map;)V +! .locals 0 + return-void + .end method + + .method private final declared-synchronized updateSuperProperties(Ljava/util/Map;)V +! .locals 0 + + return-void + .end method + + +Only in com.discord/smali/com/discord/utilities/analytics: AnalyticSuperProperties.smali.orig + diff --git a/patches/disable-mobileindicator/README.md b/patches/disable-mobileindicator/README.md index 9da79b175f..60dfd3af7c 100644 --- a/patches/disable-mobileindicator/README.md +++ b/patches/disable-mobileindicator/README.md @@ -1,12 +1,11 @@ ## DisTok CutTheCord: Disable Mobile Indicator Patch -This patch replaces the browser name from "Discord Android" to "Discord Client", disabling the Mobile Indicator which is a privacy violation without an explicit opt-out option. +Between 8.3.0 and 33.1: This patch replaces the browser name from "Discord Android" to "Discord Client", disabling the Mobile Indicator which is a privacy violation without an explicit opt-out option. -However, this causes some issues. See the `Bugs / Side effects` list below. +33.1+: This disables SuperProperties, which causes mobile indicator to be not displayed. #### Bugs / Side effects -- CRITICAL: Push notifications are broken -- NOTE: This is really obvious on discord's end, and may be bannable. +- CRITICAL (8.3.0-33.1): Push notifications are broken #### Available and tested on: - 8.3.0