Merge "Rename AppSettings to AppInfoSettings for Spa"

This commit is contained in:
Chaohui Wang
2022-10-13 06:04:41 +00:00
committed by Android (Google) Code Review
17 changed files with 28 additions and 27 deletions

View File

@@ -23,7 +23,7 @@ import android.util.FeatureFlagUtils;
import com.android.settings.SettingsActivity; import com.android.settings.SettingsActivity;
import com.android.settings.applications.appinfo.AppInfoDashboardFragment; import com.android.settings.applications.appinfo.AppInfoDashboardFragment;
import com.android.settings.spa.SpaActivity; import com.android.settings.spa.SpaActivity;
import com.android.settings.spa.app.appsettings.AppSettingsProvider; import com.android.settings.spa.app.appinfo.AppInfoSettingsProvider;
public class InstalledAppDetailsTop extends SettingsActivity { public class InstalledAppDetailsTop extends SettingsActivity {
@@ -35,7 +35,7 @@ public class InstalledAppDetailsTop extends SettingsActivity {
} }
String packageName = super.getIntent().getData().getSchemeSpecificPart(); String packageName = super.getIntent().getData().getSchemeSpecificPart();
SpaActivity.startSpaActivity( SpaActivity.startSpaActivity(
this, AppSettingsProvider.INSTANCE.getRoute(packageName, getUserId())); this, AppInfoSettingsProvider.INSTANCE.getRoute(packageName, getUserId()));
finish(); finish();
} }

View File

@@ -18,7 +18,7 @@ package com.android.settings.spa
import com.android.settings.spa.app.AllAppListPageProvider import com.android.settings.spa.app.AllAppListPageProvider
import com.android.settings.spa.app.AppsMainPageProvider import com.android.settings.spa.app.AppsMainPageProvider
import com.android.settings.spa.app.appsettings.AppSettingsProvider import com.android.settings.spa.app.appinfo.AppInfoSettingsProvider
import com.android.settings.spa.app.specialaccess.AlarmsAndRemindersAppListProvider import com.android.settings.spa.app.specialaccess.AlarmsAndRemindersAppListProvider
import com.android.settings.spa.app.specialaccess.AllFilesAccessAppListProvider import com.android.settings.spa.app.specialaccess.AllFilesAccessAppListProvider
import com.android.settings.spa.app.specialaccess.DisplayOverOtherAppsAppListProvider import com.android.settings.spa.app.specialaccess.DisplayOverOtherAppsAppListProvider
@@ -54,7 +54,7 @@ object SettingsSpaEnvironment : SpaEnvironment() {
HomePageProvider, HomePageProvider,
AppsMainPageProvider, AppsMainPageProvider,
AllAppListPageProvider, AllAppListPageProvider,
AppSettingsProvider, AppInfoSettingsProvider,
SpecialAppAccessPageProvider, SpecialAppAccessPageProvider,
NotificationMainPageProvider, NotificationMainPageProvider,
AppListNotificationsPageProvider, AppListNotificationsPageProvider,

View File

@@ -22,7 +22,7 @@ import androidx.compose.runtime.Composable
import androidx.compose.runtime.remember import androidx.compose.runtime.remember
import androidx.compose.ui.res.stringResource import androidx.compose.ui.res.stringResource
import com.android.settings.R import com.android.settings.R
import com.android.settings.spa.app.appsettings.AppSettingsProvider import com.android.settings.spa.app.appinfo.AppInfoSettingsProvider
import com.android.settingslib.spa.framework.common.SettingsEntryBuilder import com.android.settingslib.spa.framework.common.SettingsEntryBuilder
import com.android.settingslib.spa.framework.common.SettingsPage import com.android.settingslib.spa.framework.common.SettingsPage
import com.android.settingslib.spa.framework.common.SettingsPageProvider import com.android.settingslib.spa.framework.common.SettingsPageProvider
@@ -65,7 +65,7 @@ private fun AllAppListPage() {
) { itemModel -> ) { itemModel ->
AppListItem( AppListItem(
itemModel = itemModel, itemModel = itemModel,
onClick = AppSettingsProvider.navigator(app = itemModel.record.app), onClick = AppInfoSettingsProvider.navigator(app = itemModel.record.app),
) )
} }
} }

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.app.ActivityManager import android.app.ActivityManager
import android.content.ComponentName import android.content.ComponentName

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.pm.PackageInfo import android.content.pm.PackageInfo
import androidx.compose.runtime.Composable import androidx.compose.runtime.Composable

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.pm.PackageInfo import android.content.pm.PackageInfo
import androidx.compose.material.icons.Icons import androidx.compose.material.icons.Icons

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo
import android.content.pm.PackageInfo import android.content.pm.PackageInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.app.settings.SettingsEnums import android.app.settings.SettingsEnums
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo
import android.os.Bundle import android.os.Bundle
@@ -42,8 +42,8 @@ import com.android.settingslib.spaprivileged.template.app.AppInfoProvider
private const val PACKAGE_NAME = "packageName" private const val PACKAGE_NAME = "packageName"
private const val USER_ID = "userId" private const val USER_ID = "userId"
object AppSettingsProvider : SettingsPageProvider { object AppInfoSettingsProvider : SettingsPageProvider {
override val name = "AppSettings" override val name = "AppInfoSettings"
override val parameter = listOf( override val parameter = listOf(
navArgument(PACKAGE_NAME) { type = NavType.StringType }, navArgument(PACKAGE_NAME) { type = NavType.StringType },
@@ -59,7 +59,7 @@ object AppSettingsProvider : SettingsPageProvider {
val packageInfoPresenter = remember { val packageInfoPresenter = remember {
PackageInfoPresenter(context, packageName, userId, coroutineScope) PackageInfoPresenter(context, packageName, userId, coroutineScope)
} }
AppSettings(packageInfoPresenter) AppInfoSettings(packageInfoPresenter)
packageInfoPresenter.PackageRemoveDetector() packageInfoPresenter.PackageRemoveDetector()
} }
@@ -67,7 +67,7 @@ object AppSettingsProvider : SettingsPageProvider {
fun navigator(app: ApplicationInfo) = navigator(route = "$name/${app.toRoute()}") fun navigator(app: ApplicationInfo) = navigator(route = "$name/${app.toRoute()}")
/** /**
* Gets the route to the App Settings page. * Gets the route to the App Info Settings page.
* *
* Expose route to enable enter from non-SPA pages. * Expose route to enable enter from non-SPA pages.
*/ */
@@ -75,13 +75,13 @@ object AppSettingsProvider : SettingsPageProvider {
} }
@Composable @Composable
private fun AppSettings(packageInfoPresenter: PackageInfoPresenter) { private fun AppInfoSettings(packageInfoPresenter: PackageInfoPresenter) {
val packageInfo = packageInfoPresenter.flow.collectAsState().value ?: return val packageInfo = packageInfoPresenter.flow.collectAsState().value ?: return
val app = packageInfo.applicationInfo val app = packageInfo.applicationInfo
RegularScaffold( RegularScaffold(
title = stringResource(R.string.application_info_label), title = stringResource(R.string.application_info_label),
actions = { actions = {
AppSettingsMoreOptions(packageInfoPresenter, app) AppInfoSettingsMoreOptions(packageInfoPresenter, app)
} }
) { ) {
val appInfoProvider = remember { AppInfoProvider(packageInfo) } val appInfoProvider = remember { AppInfoProvider(packageInfo) }

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.Context import android.content.Context
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo
@@ -34,7 +34,7 @@ import com.android.settingslib.spaprivileged.model.app.isDisallowControl
import com.android.settingslib.spaprivileged.model.app.userId import com.android.settingslib.spaprivileged.model.app.userId
@Composable @Composable
fun AppSettingsMoreOptions(packageInfoPresenter: PackageInfoPresenter, app: ApplicationInfo) { fun AppInfoSettingsMoreOptions(packageInfoPresenter: PackageInfoPresenter, app: ApplicationInfo) {
val context = LocalContext.current val context = LocalContext.current
// We don't allow uninstalling update for DO/PO if it's a system app, because it will clear data // We don't allow uninstalling update for DO/PO if it's a system app, because it will clear data
// on all users. We also don't allow uninstalling for all users if it's DO/PO for any user. // on all users. We also don't allow uninstalling for all users if it's DO/PO for any user.

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.Intent import android.content.Intent
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.Intent import android.content.Intent
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.ActivityNotFoundException import android.content.ActivityNotFoundException
import android.content.Context import android.content.Context

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.Context import android.content.Context
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.content.om.OverlayManager import android.content.om.OverlayManager
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo

View File

@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings package com.android.settings.spa.app.appinfo
import android.app.ActivityManager import android.app.ActivityManager
import android.app.settings.SettingsEnums import android.app.settings.SettingsEnums

View File

@@ -13,7 +13,8 @@
* See the License for the specific language governing permissions and * See the License for the specific language governing permissions and
* limitations under the License. * limitations under the License.
*/ */
package com.android.settings.spa.app.appsettings
package com.android.settings.spa.app.appinfo
import android.content.Context import android.content.Context
import android.content.pm.ApplicationInfo import android.content.pm.ApplicationInfo