Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[image_picker] Fix check for iOS 14+ authorization status #6845

Merged
merged 5 commits into from
Dec 23, 2022
Merged

[image_picker] Fix check for iOS 14+ authorization status #6845

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
6d50058
fix check for authorization status
vashworth Dec 14, 2022
2da5b91
Merge remote-tracking branch 'upstream/main' into image_picker_permis…
vashworth Dec 16, 2022
1d95a0a
add unit tests and update release info
vashworth Dec 16, 2022
c3f82e7
Merge remote-tracking branch 'upstream/main' into image_picker_permis…
vashworth Dec 21, 2022
e9ae12f
update release info
vashworth Dec 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions packages/image_picker/image_picker_ios/ios/Classes/FLTImagePickerPlugin.m
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -368,11 +368,13 @@ - (void)checkPhotoAuthorizationWithImagePicker:(UIImagePickerController *)imageP
}

- (void)checkPhotoAuthorizationForAccessLevel API_AVAILABLE(ios(14)) {
PHAuthorizationStatus status = [PHPhotoLibrary authorizationStatus];
PHAccessLevel requestedAccessLevel = PHAccessLevelReadWrite;
PHAuthorizationStatus status =
[PHPhotoLibrary authorizationStatusForAccessLevel:requestedAccessLevel];
Comment on lines +368 to +370
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Up to you, but it might be clearer to get rid of the local variable and pass in PHAccessLevelReadWrite in two places. Accessing an enum is so cheap there's no reason to cache it.

Suggested change
PHAccessLevel requestedAccessLevel = PHAccessLevelReadWrite;
PHAuthorizationStatus status =
[PHPhotoLibrary authorizationStatusForAccessLevel:requestedAccessLevel];
PHAuthorizationStatus status =
[PHPhotoLibrary authorizationStatusForAccessLevel:PHAccessLevelReadWrite];

switch (status) {
case PHAuthorizationStatusNotDetermined: {
[PHPhotoLibrary
requestAuthorizationForAccessLevel:PHAccessLevelReadWrite
requestAuthorizationForAccessLevel:requestedAccessLevel
handler:^(PHAuthorizationStatus status) {
dispatch_async(dispatch_get_main_queue(), ^{
if (status == PHAuthorizationStatusAuthorized) {
Expand Down