mnalis/apps-android-commons
1
0
Fork 0
mirror of https://github.com/commons-app/apps-android-commons.git synced 2025-10-27 04:43:54 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

make prop nullable (#6073)

Browse source 
Signed-off-by: parneet-guraya <gurayaparneet@gmail.com>
This commit is contained in:
Parneet Singh 2024-12-24 21:25:12 +05:30 • committed by GitHub
parent e7864ac1dd
commit 86cdf96f3d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 17 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

2
app/src/main/java/fr/free/nrw/commons/upload/UploadBaseFragment.kt
View file

@ -6,7 +6,7 @@ import fr.free.nrw.commons.di.CommonsDaggerSupportFragment
* The base fragment of the fragments in upload * The base fragment of the fragments in upload
*/ */
abstract class UploadBaseFragment : CommonsDaggerSupportFragment() { abstract class UploadBaseFragment : CommonsDaggerSupportFragment() {
lateinit var callback: Callback var callback: Callback? = null
protected open fun onBecameVisible() = Unit protected open fun onBecameVisible() = Unit

8
app/src/main/java/fr/free/nrw/commons/upload/categories/UploadCategoriesFragment.kt
View file

@ -96,10 +96,10 @@ class UploadCategoriesFragment : UploadBaseFragment(), CategoriesContract.View {
if (media == null) { if (media == null) {
if (callback != null) { if (callback != null) {
binding!!.tvTitle.text = getString( binding!!.tvTitle.text = getString(
R.string.step_count, callback.getIndexInViewFlipper( R.string.step_count, callback!!.getIndexInViewFlipper(
this this
) + 1, ) + 1,
callback.totalNumberOfSteps, getString(R.string.categories_activity_title) callback!!.totalNumberOfSteps, getString(R.string.categories_activity_title)
) )
} }
} else { } else {
@ -220,7 +220,7 @@ class UploadCategoriesFragment : UploadBaseFragment(), CategoriesContract.View {
} }
override fun goToNextScreen() { override fun goToNextScreen() {
callback.onNextButtonClicked(callback.getIndexInViewFlipper(this)) callback!!.onNextButtonClicked(callback!!.getIndexInViewFlipper(this))
} }
override fun showNoCategorySelected() { override fun showNoCategorySelected() {
@ -322,7 +322,7 @@ class UploadCategoriesFragment : UploadBaseFragment(), CategoriesContract.View {
mediaDetailFragment.onResume() mediaDetailFragment.onResume()
goBackToPreviousScreen() goBackToPreviousScreen()
} else { } else {
callback.onPreviousButtonClicked(callback.getIndexInViewFlipper(this)) callback!!.onPreviousButtonClicked(callback!!.getIndexInViewFlipper(this))
} }
} }

10
app/src/main/java/fr/free/nrw/commons/upload/depicts/DepictsFragment.kt
View file

@ -97,10 +97,10 @@ class DepictsFragment : UploadBaseFragment(), DepictsContract.View {
if (media == null) { if (media == null) {
binding.depictsTitle.text = binding.depictsTitle.text =
String.format( String.format(
getString(R.string.step_count), callback.getIndexInViewFlipper( getString(R.string.step_count), callback!!.getIndexInViewFlipper(
this this
) + 1, ) + 1,
callback.totalNumberOfSteps, getString(R.string.depicts_step_title) callback!!.totalNumberOfSteps, getString(R.string.depicts_step_title)
) )
} else { } else {
binding.depictsTitle.setText(R.string.edit_depictions) binding.depictsTitle.setText(R.string.edit_depictions)
@ -173,11 +173,11 @@ class DepictsFragment : UploadBaseFragment(), DepictsContract.View {
} }
override fun goToNextScreen() { override fun goToNextScreen() {
callback.onNextButtonClicked(callback.getIndexInViewFlipper(this)) callback!!.onNextButtonClicked(callback!!.getIndexInViewFlipper(this))
} }
override fun goToPreviousScreen() { override fun goToPreviousScreen() {
callback.onPreviousButtonClicked(callback.getIndexInViewFlipper(this)) callback!!.onPreviousButtonClicked(callback!!.getIndexInViewFlipper(this))
} }
override fun noDepictionSelected() { override fun noDepictionSelected() {
@ -338,7 +338,7 @@ class DepictsFragment : UploadBaseFragment(), DepictsContract.View {
updateDepicts() updateDepicts()
goBackToPreviousScreen() goBackToPreviousScreen()
} else { } else {
callback.onPreviousButtonClicked(callback.getIndexInViewFlipper(this)) callback!!.onPreviousButtonClicked(callback!!.getIndexInViewFlipper(this))
} }
} }

14
app/src/main/java/fr/free/nrw/commons/upload/license/MediaLicenseFragment.kt
View file

@ -47,20 +47,20 @@ class MediaLicenseFragment : UploadBaseFragment(), MediaLicenseContract.View {
binding.tvTitle.text = getString( binding.tvTitle.text = getString(
R.string.step_count, R.string.step_count,
callback.getIndexInViewFlipper(this) + 1, callback!!.getIndexInViewFlipper(this) + 1,
callback.totalNumberOfSteps, callback!!.totalNumberOfSteps,
getString(R.string.license_step_title) getString(R.string.license_step_title)
) )
setTvSubTitle() setTvSubTitle()
binding.btnPrevious.setOnClickListener { binding.btnPrevious.setOnClickListener {
callback.onPreviousButtonClicked( callback!!.onPreviousButtonClicked(
callback.getIndexInViewFlipper(this) callback!!.getIndexInViewFlipper(this)
) )
} }
binding.btnSubmit.setOnClickListener { binding.btnSubmit.setOnClickListener {
callback.onNextButtonClicked( callback!!.onNextButtonClicked(
callback.getIndexInViewFlipper(this) callback!!.getIndexInViewFlipper(this)
) )
} }
@ -201,6 +201,6 @@ class MediaLicenseFragment : UploadBaseFragment(), MediaLicenseContract.View {
* Show the wlm info message if the upload is a WLM upload * Show the wlm info message if the upload is a WLM upload
*/ */
binding.llInfoMonumentUpload.visibility = binding.llInfoMonumentUpload.visibility =
if (callback.isWLMUpload && presenter.isWLMSupportedForThisPlace()) View.VISIBLE else View.GONE if (callback!!.isWLMUpload && presenter.isWLMSupportedForThisPlace()) View.VISIBLE else View.GONE
} }
} }