Fixed: Application crashing below API level 29 when we select the external storage.

[MohitMaliFtechiz]

Parent Issue #3638

See #3638 (comment)

• The issue was with the FileObserver class constructor we used, which is introduced in SDK 29. As a result, it is not available for older versions, leading to a NoSuchMethodError when attempting to access it. This caused the application to crash below API level 29 when selecting external storage.

[kelson42]

@MohitMaliFtechiz Which apps are impacted (kiwix, custom apps)? Why this bug has not been detected for other (custom) apps? Earlier? Why custom apps - where you can not choose any storage - are impacted? How often such a crash will happen?

[MohitMaliFtechiz]

@MohitMaliFtechiz Which apps are impacted (kiwix, custom apps)?

@kelson42 It will impact the kiwix app.

Why this bug has not been detected for other (custom) apps? Earlier? Why custom apps - where you can not choose any storage - are impacted?

This bug does not exist in the custom apps, which is why it has not been recognized yet in custom apps, for kiwix app, it occurs below Android 9 when external storage has very little storage in it.

[codecov]

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (492e8ee) 48.61% compared to head (e775447) 48.63%.
Report is 1 commits behind head on main.

Files	Patch %	Lines
...a/org/kiwix/kiwixmobile/zimManager/Fat32Checker.kt	0.00%	10 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #3642      +/-   ##
============================================
+ Coverage     48.61%   48.63%   +0.01%     
- Complexity     1090     1092       +2     
============================================
  Files           285      285              
  Lines         10582    10588       +6     
  Branches       1418     1418              
============================================
+ Hits           5144     5149       +5     
- Misses         4594     4598       +4     
+ Partials        844      841       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MohitMaliFtechiz]

For the lint issue, we just merged the #3626, and the string file has not been updated on translate-wiki, so it is not allowing me to correct the translations. Probably it will take a few hours to update.

[kelson42]

@MohitMaliFtechiz Why this PR had no reviewer?

[MohitMaliFtechiz]

@kelson42 We had encountered the lint-related issue so I did not request a review at that time, and after the lint issue was resolved I was busy with CI failure and libkiwix bookmarks issues so I forgot to request a review on this PR, sorry for that.

[gouri-panda]

Thanks @MohitMaliFtechiz ! @kelson42 LGTM!