Fix photos permission issue on Android 12 and below

mobile/lib/pages/image_picker_page.dart

@@ -8,7 +8,9 @@ import 'package:flutter/material.dart';
 import 'package:mapbox_gl/mapbox_gl.dart' as maps;
 import 'package:image_picker/image_picker.dart';
 import 'package:mobile/time_manager.dart';
+import 'package:mobile/wrappers/device_info_wrapper.dart';
 import 'package:mobile/wrappers/exif_wrapper.dart';
+import 'package:mobile/wrappers/io_wrapper.dart';
 import 'package:photo_manager/photo_manager.dart';
 import 'package:quiver/strings.dart';
 import 'package:timezone/timezone.dart';
@@ -188,10 +190,14 @@ class ImagePickerPageState extends State<ImagePickerPage> {
 
   late Future<bool> _isPermissionGrantedFuture;
 
+  DeviceInfoWrapper get _deviceInfoWrapper => DeviceInfoWrapper.of(context);
+
   FilePickerWrapper get _filePicker => FilePickerWrapper.of(context);
 
   ImagePickerWrapper get _imagePicker => ImagePickerWrapper.of(context);
 
+  IoWrapper get _ioWrapper => IoWrapper.of(context);
+
   PermissionHandlerWrapper get _permissionHandlerWrapper =>
       PermissionHandlerWrapper.of(context);
 
@@ -202,7 +208,8 @@ class ImagePickerPageState extends State<ImagePickerPage> {
   @override
   void initState() {
     super.initState();
-    _isPermissionGrantedFuture = _permissionHandlerWrapper.requestPhotos();
+    _isPermissionGrantedFuture =
+        _permissionHandlerWrapper.requestPhotos(_deviceInfoWrapper, _ioWrapper);
   }
 
   @override

mobile/lib/wrappers/permission_handler_wrapper.dart

@@ -1,8 +1,10 @@
 import 'package:flutter/material.dart';
+import 'package:mobile/wrappers/device_info_wrapper.dart';
 import 'package:permission_handler/permission_handler.dart';
 import 'package:provider/provider.dart';
 
 import '../app_manager.dart';
+import 'io_wrapper.dart';
 
 class PermissionHandlerWrapper {
   static PermissionHandlerWrapper of(BuildContext context) =>
@@ -19,8 +21,19 @@ class PermissionHandlerWrapper {
   Future<bool> get isLocationAlwaysGranted async =>
       Permission.locationAlways.isGranted;
 
-  Future<bool> requestPhotos() async =>
-      (await Permission.photos.request()).isGranted;
+  Future<bool> requestPhotos(
+      DeviceInfoWrapper deviceInfo, IoWrapper ioWrapper) async {
+    // TODO: Necessary until
+    //  https://github.com/Baseflow/flutter-permission-handler/issues/944 is
+    //  fixed. Permission.storage.request() always returns denied on Android 12
+    //  and below.
+    if (ioWrapper.isAndroid &&
+        (await deviceInfo.androidInfo).version.sdkInt <= 32) {
+      return (await Permission.storage.request()).isGranted;
+    } else {
+      return (await Permission.photos.request()).isGranted;
+    }
+  }
 
   Future<bool> openSettings() => openAppSettings();
 }

mobile/test/mocks/mocks.mocks.dart

@@ -1336,9 +1336,8 @@ class _FakeRequestType_116 extends _i1.SmartFake implements _i68.RequestType {
         );
 }
 
-class _FakeFilterOptionGroup_117 extends _i1.SmartFake
-    implements _i68.FilterOptionGroup {
-  _FakeFilterOptionGroup_117(
+class _FakePMFilter_117 extends _i1.SmartFake implements _i68.PMFilter {
+  _FakePMFilter_117(
     Object parent,
     Invocation parentInvocation,
   ) : super(
@@ -12839,10 +12838,17 @@ class MockPermissionHandlerWrapper extends _i1.Mock
         returnValue: _i2.Future<bool>.value(false),
       ) as _i2.Future<bool>);
   @override
-  _i2.Future<bool> requestPhotos() => (super.noSuchMethod(
+  _i2.Future<bool> requestPhotos(
+    _i29.DeviceInfoWrapper? deviceInfo,
+    _i38.IoWrapper? ioWrapper,
+  ) =>
+      (super.noSuchMethod(
         Invocation.method(
           #requestPhotos,
-          [],
+          [
+            deviceInfo,
+            ioWrapper,
+          ],
         ),
         returnValue: _i2.Future<bool>.value(false),
       ) as _i2.Future<bool>);
@@ -13180,13 +13186,13 @@ class MockAssetPathEntity extends _i1.Mock implements _i68.AssetPathEntity {
         returnValue: false,
       ) as bool);
   @override
-  _i68.FilterOptionGroup get filterOption => (super.noSuchMethod(
+  _i68.PMFilter get filterOption => (super.noSuchMethod(
         Invocation.getter(#filterOption),
-        returnValue: _FakeFilterOptionGroup_117(
+        returnValue: _FakePMFilter_117(
           this,
           Invocation.getter(#filterOption),
         ),
-      ) as _i68.FilterOptionGroup);
+      ) as _i68.PMFilter);
   @override
   _i2.Future<int> get assetCountAsync => (super.noSuchMethod(
         Invocation.getter(#assetCountAsync),

mobile/test/pages/add_catch_journey_test.dart

@@ -101,7 +101,7 @@ void main() {
       page: anyNamed("page"),
       size: anyNamed("size"),
     )).thenAnswer((_) => Future.value(mockAssets));
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(true));
     when(appManager.photoManagerWrapper.getAllAssetPathEntity(any))
         .thenAnswer((_) => Future.value(allAlbum));

mobile/test/pages/image_picker_page_test.dart

@@ -44,7 +44,7 @@ void main() {
         .thenAnswer((_) => Future.value(null));
     when(appManager.photoManagerWrapper.getAllAssetPathEntity(any))
         .thenAnswer((_) => Future.value(allAlbum));
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(true));
 
     var exif = MockExif();
@@ -549,7 +549,7 @@ void main() {
 
   testWidgets("Placeholder grid shown when waiting for permission future",
       (tester) async {
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(false));
 
     await tester.pumpWidget(Testable(
@@ -605,7 +605,7 @@ void main() {
   });
 
   testWidgets("No permission placeholder shown", (tester) async {
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(false));
 
     await tester.pumpWidget(Testable(

mobile/test/widget/fishing_spot_details_test.dart

@@ -25,7 +25,7 @@ void main() {
     when(appManager.fishingSpotManager.entity(any)).thenReturn(null);
     when(appManager.fishingSpotManager.entityExists(any)).thenReturn(false);
 
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(false));
 
     when(appManager.userPreferenceManager.catchFieldIds).thenReturn([]);

mobile/test/widget/image_input_test.dart

@@ -37,7 +37,7 @@ void main() {
         .thenAnswer((_) => Future.value(null));
     when(appManager.photoManagerWrapper.getAllAssetPathEntity(any))
         .thenAnswer((_) => Future.value(allAlbum));
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(true));
 
     var exif = MockExif();

mobile/test/widget/image_picker_test.dart

@@ -16,7 +16,7 @@ void main() {
   setUp(() {
     appManager = StubbedAppManager();
 
-    when(appManager.permissionHandlerWrapper.requestPhotos())
+    when(appManager.permissionHandlerWrapper.requestPhotos(any, any))
         .thenAnswer((_) => Future.value(true));
 
     when(appManager.photoManagerWrapper.getAllAssetPathEntity(any))