Fixes #5531 Circular Progress bar keeps on moving when nomination for deletion is successful.

[neeldoshii]

Fixes

• Fixes [Bug]: Circular Progress bar keeps on moving even though nomination for deletion is successful #5531.

Description

This pull request addresses an issue where the circular progress bar remains active and keeps moving even after a nomination for deletion is successful. The fix ensures that the progress bar is hidden and the UI is updated correctly upon successful deletion also the Snackbar is shown when the nomination is done successfully.

Changes

• Modified the onDeleteClicked() & onDeleteClickeddialogtext & created a handleDeletionResult() for handling what to do on success, created a handleDeletionError() for handling what to do on failure.
• Ensured that UI updates are performed on the UI thread to prevent potential CalledFromWrongThreadException.

Test Plan

Steps to Test

1. Open the app.
2. Open any contribution.
3. Click on the nomination for deletion.
4. Observe the progress bar behavior after the nomination process completes.

Expected Results

• The circular progress bar should disappear, indicating that the process has completed.
• The user should see a confirmation message indicating the successful nomination for deletion.

Screenshots

Additional Information

• Tested on both emulators and physical devices to verify the fix.

[neeldoshii]

Special thanks to @kanahia1 for giving me the opportunity to contribute to this issue and for the keen observation in identifying this bug.

[neeldoshii]

Just a friendly ping @nicolas-raoul @RitikaPahwa4444 , can you please Review this pr and let me know if there is any change required.

[nicolas-raoul]

Very strangely, even though CI is running fine, I am getting this when building, any idea why?

> Task :app:compileProdDebugJavaWithJavac
The following annotation processors are not incremental: dagger-android-processor-2.23.jar (com.google.dagger:dagger-android-processor:2.23), dagger-compiler-2.23.jar (com.google.dagger:dagger-compiler:2.23).
Make sure all annotation processors are incremental to improve your build speed.
/Users/nicolasraoul/src/commons-app/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java:1235: error: cannot find symbol
            if (progressBarDeletion != null) {
                ^
  symbol:   variable progressBarDeletion
  location: class MediaDetailFragment
/Users/nicolasraoul/src/commons-app/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java:1236: error: cannot find symbol
                progressBarDeletion.setVisibility(GONE);
                ^
  symbol:   variable progressBarDeletion
  location: class MediaDetailFragment
/Users/nicolasraoul/src/commons-app/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java:1244: error: cannot find symbol
            delete.setText("Nominated for Deletion");
            ^
  symbol:   variable delete
  location: class MediaDetailFragment

[nicolas-raoul]

CI unfortunately fails after rebasing on the main branch.
Would you mind fixing?
Thanks a lot! :-)