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

fix: remove if-condition to prevent hiding top card and refactor code

Browse source
This commit is contained in:
Rohit Verma 2025-03-09 22:08:45 +05:30
parent 1e1e5d763f
commit 07a71cbef2
1 changed files with 22 additions and 25 deletions
Download patch file Download diff file Expand all files Collapse all files

7
app/src/main/java/fr/free/nrw/commons/upload/UploadPresenter.kt
View file

@ -146,11 +146,7 @@ class UploadPresenter @Inject internal constructor(
override fun deletePictureAtIndex(index: Int) { override fun deletePictureAtIndex(index: Int) {
val uploadableFiles = view.getUploadableFiles() val uploadableFiles = view.getUploadableFiles()
if (index == uploadableFiles!!.size - 1) { uploadableFiles?.let {
// If the next fragment to be shown is not one of the MediaDetailsFragment
// lets hide the top card so that it doesn't appear on the other fragments
view.showHideTopCard(false)
}
view.setImageCancelled(true) view.setImageCancelled(true)
repository.deletePicture(uploadableFiles[index].getFilePath()) repository.deletePicture(uploadableFiles[index].getFilePath())
if (uploadableFiles.size == 1) { if (uploadableFiles.size == 1) {
@ -175,6 +171,7 @@ class UploadPresenter @Inject internal constructor(
//In case lets update the number of uploadable media //In case lets update the number of uploadable media
view.updateTopCardTitle() view.updateTopCardTitle()
} }
}
override fun onAttachView(view: UploadContract.View) { override fun onAttachView(view: UploadContract.View) {
this.view = view this.view = view