Do not use erroneous thumb url util function

[maskaravivek]

Description (required)

Fixes #2899
Fixes #2900

[codecov-io]

Codecov Report

Merging #2901 into master will decrease coverage by 0.03%.
The diff coverage is 25%.

@@            Coverage Diff            @@
##           master   #2901      +/-   ##
=========================================
- Coverage    3.42%   3.39%   -0.04%     
=========================================
  Files         246     246              
  Lines       12252   12212      -40     
  Branches     1080    1080              
=========================================
- Hits          420     414       -6     
+ Misses      11802   11768      -34     
  Partials       30      30

Impacted Files	Coverage Δ
app/src/main/java/fr/free/nrw/commons/Media.java	0.68% <ø> (-5.44%)	⬇️
...free/nrw/commons/utils/MediaDataExtractorUtil.java	88.88% <ø> (+41.83%)	⬆️
app/src/main/java/fr/free/nrw/commons/Utils.java	18.03% <ø> (+0.84%)	⬆️
...ava/fr/free/nrw/commons/review/ReviewActivity.java	0% <0%> (ø)	⬆️
...ommons/contributions/ContributionsSyncAdapter.java	0% <0%> (ø)	⬆️
.../java/fr/free/nrw/commons/review/ReviewHelper.java	86.48% <100%> (+0.77%)	⬆️
.../fr/free/nrw/commons/nearby/NearbyMapFragment.java	0% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update accb880...293a939. Read the comment docs.

[maskaravivek]

The database no longer saves invalid URLs. The URLs saved are directly taken from the API. Otherwise null is stored. In a different PR, we can have logic for fallback images.

Tested the following:

Image loading in peer review:

Category loading in peer review:

Image upload:

[ashishkumar468]

I know this is beyond the scope of this PR, but can you clear the composite disposable in the onStop/ onDestroy of the activity, ?

[maskaravivek]

Thanks for merging @ashishkumar468