diff --git a/app/src/main/java/org/kiwix/kiwixmobile/language/LanguageActivity.kt b/app/src/main/java/org/kiwix/kiwixmobile/language/LanguageActivity.kt index 387c5c401..43ebcce67 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/language/LanguageActivity.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/language/LanguageActivity.kt @@ -34,7 +34,7 @@ class LanguageActivity : BaseActivity() { @Inject lateinit var viewModelFactory: ViewModelProvider.Factory - val compositeDisposable = CompositeDisposable() + private val compositeDisposable = CompositeDisposable() private val languageAdapter = LanguageAdapter( @@ -75,7 +75,7 @@ class LanguageActivity : BaseActivity() { compositeDisposable.clear() } - fun render(state: State) = when (state) { + private fun render(state: State) = when (state) { Loading -> language_progressbar.show() is Content -> { language_progressbar.hide() diff --git a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageDelegate.kt b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageDelegate.kt index 08cbde435..8997a0ab1 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageDelegate.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageDelegate.kt @@ -20,7 +20,7 @@ sealed class LanguageDelegate Unit) : LanguageDelegate() { + class LanguageItemDelegate(private val clickAction: (LanguageItem) -> Unit) : LanguageDelegate() { override val itemClass = LanguageItem::class.java override fun createViewHolder(parent: ViewGroup) = diff --git a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListItem.kt b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListItem.kt index 7842fe4bf..3450d467d 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListItem.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListItem.kt @@ -8,7 +8,7 @@ sealed class LanguageListItem { data class HeaderItem constructor( override val id: Long ) : LanguageListItem() { - companion object{ + companion object { const val SELECTED = Long.MAX_VALUE const val OTHER = Long.MIN_VALUE } diff --git a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListViewHolder.kt b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListViewHolder.kt index 1c4441880..f8ac59493 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListViewHolder.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/language/adapter/LanguageListViewHolder.kt @@ -34,7 +34,7 @@ sealed class LanguageListViewHolder(override val container item_language_books_count.text = containerView.resources.getQuantityString( R.plurals.books_count, language.occurencesOfLanguage, language.occurencesOfLanguage ) - item_language_checkbox.isChecked = language.active; + item_language_checkbox.isChecked = language.active item_language_clickable_area.setOnClickListener { clickAction(item) } } diff --git a/app/src/main/java/org/kiwix/kiwixmobile/language/viewmodel/State.kt b/app/src/main/java/org/kiwix/kiwixmobile/language/viewmodel/State.kt index a49febd60..02be4c9e0 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/language/viewmodel/State.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/language/viewmodel/State.kt @@ -5,7 +5,7 @@ import org.kiwix.kiwixmobile.language.adapter.LanguageListItem.HeaderItem import org.kiwix.kiwixmobile.language.adapter.LanguageListItem.LanguageItem import org.kiwix.kiwixmobile.zim_manager.Language -sealed class State() { +sealed class State { object Loading : State() object Saving : State() data class Content(