Fixes commons-app#3923 (Back" does nothing in Nearby in particular circumstance) (commons-app#3961)

* Fixes commons-app#3923
* Handled possible missing backpress cases in MainActivity-nearby

* Fixes commons-app#3923
* Handled possible missing backpress cases in MainActivity-nearby

Author: ashishkumar468

app/src/main/java/fr/free/nrw/commons/contributions/MainActivity.java

@@ -242,7 +242,7 @@ public void onBackPressed() {
             // Meas that contribution fragment is visible (not nearby fragment)
             ContributionsFragment contributionsFragment = (ContributionsFragment) getSupportFragmentManager().findFragmentByTag(contributionsFragmentTag);
 
-            if (contributionsFragment.getChildFragmentManager().findFragmentByTag(ContributionsFragment.MEDIA_DETAIL_PAGER_FRAGMENT_TAG) != null) {
+            if (contributionsFragment.getChildFragmentManager().getBackStackEntryCount()>0 ) {
                 // Means that media details fragment is visible to uer instead of contributions list fragment (As chils fragment)
                 // Then we want to go back to contributions list fragment on backbutton pressed from media detail fragment
                 contributionsFragment.getChildFragmentManager().popBackStack();
@@ -257,12 +257,13 @@ public void onBackPressed() {
                     contributionsFragment.nearbyNotificationCardView.setVisibility(View.GONE);
                 }
             } else {
-                finish();
+                super.onBackPressed();
             }
-        } else if (getSupportFragmentManager().findFragmentByTag(nearbyFragmentTag) != null && !isContributionsFragmentVisible) {
+        } else if (getSupportFragmentManager().findFragmentByTag(nearbyFragmentTag) != null
+            && !isContributionsFragmentVisible) {
             // Means that nearby fragment is visible (not contributions fragment)
-            if (null != nearbyParentFragment) {
-                nearbyParentFragment.backButtonClicked();
+            if (null == nearbyParentFragment || !nearbyParentFragment.backButtonClicked()) {
+                super.onBackPressed();
             }
         } else {
             super.onBackPressed();

app/src/main/java/fr/free/nrw/commons/nearby/contract/NearbyParentFragmentContract.java

@@ -89,7 +89,7 @@ interface UserActions {
         void detachView();
 
         void setActionListeners(JsonKvStore applicationKvStore);
-        void backButtonClicked();
+        boolean backButtonClicked();
         void onCameraMove(com.mapbox.mapboxsdk.geometry.LatLng latLng);
         void filterByMarkerType(List<Label> selectedLabels, int state, boolean filterForPlaceState, boolean filterForAllNoneType);
 

app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java

@@ -980,8 +980,8 @@ public void onLocationChangedMedium(final fr.free.nrw.commons.location.LatLng la
         }
     }
 
-    public void backButtonClicked() {
-        presenter.backButtonClicked();
+    public boolean backButtonClicked() {
+        return presenter.backButtonClicked();
     }
 
     /**

app/src/main/java/fr/free/nrw/commons/nearby/presenter/NearbyParentFragmentPresenter.java

@@ -111,16 +111,16 @@ public void setActionListeners(JsonKvStore applicationKvStore) {
     }
 
     @Override
-    public void backButtonClicked() {
+    public boolean backButtonClicked() {
         if(nearbyParentFragmentView.isListBottomSheetExpanded()) {
             // Back should first hide the bottom sheet if it is expanded
             nearbyParentFragmentView.listOptionMenuItemClicked();
+            return true;
         } else if (nearbyParentFragmentView.isDetailsBottomSheetVisible()) {
             nearbyParentFragmentView.setBottomSheetDetailsSmaller();
-        } else {
-            // Otherwise go back to contributions fragment
-            nearbyParentFragmentView.setTabItemContributions();
+            return true;
         }
+        return false;
     }
 
     public void markerUnselected() {

app/src/test/kotlin/fr/free/nrw/commons/nearby/NearbyParentFragmentPresenterTest.kt

@@ -374,8 +374,8 @@ class NearbyParentFragmentPresenterTest {
     fun testBackButtonClickedWhenNoSheetVisible() {
         whenever(nearbyParentFragmentView.isListBottomSheetExpanded()).thenReturn(false)
         whenever(nearbyParentFragmentView.isDetailsBottomSheetVisible()).thenReturn(false)
-        nearbyPresenter.backButtonClicked()
-        verify(nearbyParentFragmentView).setTabItemContributions()
+        val hasNearbyHandledBackPress = nearbyPresenter.backButtonClicked()
+        assertFalse(hasNearbyHandledBackPress)
     }
 
     @Test