Refactor Binding Logic in AppViewHolder of PerAppProxyAdapter (#3772)
Simplified the binding logic in AppViewHolder by improving readability and removing redundant code. Consolidated logic for setting the app name and handling system apps. This refactor improves the maintainability of the PerAppProxyAdapter class.
This commit is contained in:
1 parent
90f89de957
commit
f4db6bcf63
1 file changed
+10
-9
@@ -59,22 +59,23 @@ class PerAppProxyAdapter(val activity: BaseActivity, val apps: List<AppInfo>, bl
|
||||
fun bind(appInfo: AppInfo) {
|
||||
this.appInfo = appInfo
|
||||
|
||||
// Set app icon and name
|
||||
itemBypassBinding.icon.setImageDrawable(appInfo.appIcon)
|
||||
// name.text = appInfo.appName
|
||||
|
||||
itemBypassBinding.checkBox.isChecked = inBlacklist
|
||||
itemBypassBinding.packageName.text = appInfo.packageName
|
||||
if (appInfo.isSystemApp) {
|
||||
itemBypassBinding.name.text = String.format("** %1s", appInfo.appName)
|
||||
//name.textColor = Color.RED
|
||||
itemBypassBinding.name.text = if (appInfo.isSystemApp) {
|
||||
String.format("** %s", appInfo.appName)
|
||||
} else {
|
||||
itemBypassBinding.name.text = appInfo.appName
|
||||
//name.textColor = Color.DKGRAY
|
||||
appInfo.appName
|
||||
}
|
||||
|
||||
// Set package name and checkbox state
|
||||
itemBypassBinding.packageName.text = appInfo.packageName
|
||||
itemBypassBinding.checkBox.isChecked = inBlacklist
|
||||
|
||||
// Handle item click to toggle blacklist status
|
||||
itemView.setOnClickListener(this)
|
||||
}
|
||||
|
||||
|
||||
override fun onClick(v: View?) {
|
||||
if (inBlacklist) {
|
||||
blacklist.remove(appInfo.packageName)
|
||||
|
||||
Reference in new issue
Block a user