[Capture location task] Prompted to Correct enable location task

[anandwana001]

Fixes #2855

@shobhitagarwal1612 PTAL?

[codecov]

Codecov Report

Attention: Patch coverage is 13.63636% with 19 lines in your changes missing coverage. Please review.

Project coverage is 61.19%. Comparing base (20d1540) to head (a18a479).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
...tion/tasks/location/CaptureLocationTaskFragment.kt	13.63%	18 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2885      +/-   ##
============================================
- Coverage     61.20%   61.19%   -0.01%     
+ Complexity     1185     1183       -2     
============================================
  Files           267      267              
  Lines          6400     6396       -4     
  Branches        914      914              
============================================
- Hits           3917     3914       -3     
+ Misses         1944     1943       -1     
  Partials        539      539              

Files with missing lines	Coverage Δ
...n/tasks/location/CaptureLocationTaskMapFragment.kt	60.00% <ø> (+23.41%)	⬆️
...tion/tasks/location/CaptureLocationTaskFragment.kt	59.61% <13.63%> (-33.72%)	⬇️

[shobhitagarwal1612]

Instead of moving the code to CaptureLocationTaskFragment, can you do something like:

  override fun onResume() {
    super.onResume()
    if (isVisible) {
       // add existing code for showing permission dialog.
    }
  }

[anandwana001]

Instead of moving the code to CaptureLocationTaskFragment, can you do something like:

  override fun onResume() {
    super.onResume()
    if (isVisible) {
       // add existing code for showing permission dialog.
    }
  }

it will not work, because as this fragment is a part of pagination where the code is getting executed when the index is +1.

[shobhitagarwal1612]

We care about the fragment being visible to the user. Can you try out the above code once?

[anandwana001]

We care about the fragment being visible to the user. Can you try out the above code once?

Tried, not working