Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a2307ed7 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov
Browse files

[SB Refactor] Remove "wifi" from the wifi table column names. 

The logs are already put into the "WifiTableLog", so we don't need to
include "wifi" in all the column names too.

Bug: 238425913
Test: manual: Dumped WifiTableLog and verified new column names
Change-Id: I36555324c6f89a0981aecbd52a84d1273297ea02
parent 245bba60

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/data/repository/prod/WifiRepositoryImpl.kt

+3 −3
Original line number Diff line number Diff line
@@ -95,7 +95,7 @@ constructor( 
            .logDiffsForTable(

                wifiTableLogBuffer,

                columnPrefix = "",

                columnName = "isWifiEnabled",

                columnName = "isEnabled",

                initialValue = wifiManager.isWifiEnabled,

            )

            .stateIn(
@@ -141,7 +141,7 @@ constructor( 
            .logDiffsForTable(

                wifiTableLogBuffer,

                columnPrefix = "",

                columnName = "isWifiDefault",

                columnName = "isDefault",

                initialValue = false,

            )

            .stateIn(scope, started = SharingStarted.WhileSubscribed(), initialValue = false)
@@ -212,7 +212,7 @@ constructor( 
            .distinctUntilChanged()

            .logDiffsForTable(

                wifiTableLogBuffer,

                columnPrefix = "wifiNetwork",

                columnPrefix = "",

                initialValue = WIFI_NETWORK_DEFAULT,

            )

            // There will be multiple wifi icons in different places that will frequently

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/model/WifiIcon.kt

+1 −1
Original line number Diff line number Diff line
@@ -53,4 +53,4 @@ sealed interface WifiIcon : Diffable<WifiIcon> { 
    }

}



private const val COL_ICON = "wifiIcon"
 
private const val COL_ICON = "icon"