Skip to content

Conversation

@gklka
Copy link

@gklka gklka commented Oct 13, 2017

If the user uses iCloud Photo Library in "Optimize Storage" mode, photos might have not been downloaded to the device yet (only the metadata). I added the corresponding options which let the system use network for download the image assets which the user have selected. iOS tries to give you a thumbnail until the network access happens, this way the callback could have been invoked multiple times without the necessary data to indicate it is a thumbnail or not. I also added a check which invokes the callback only once, when the final result comes in.

Copy link
Owner

@d0ping d0ping left a comment

Choose a reason for hiding this comment

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

Please fix commented places

TargetAttributes = {
C3134EAE1C9C30990023B9E6 = {
CreatedOnToolsVersion = 7.2.1;
DevelopmentTeam = 9L2H2A796T;
Copy link
Owner

Choose a reason for hiding this comment

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

Please don't specified private developer team in the example project.

PHFetchResult *userAlbums = [PHAssetCollection fetchAssetCollectionsWithType:PHAssetCollectionTypeAlbum subtype:PHAssetCollectionSubtypeAny options:nil];
self.fetchResults = @[smartAlbums, userAlbums];

// self.fetchResults = @[userAlbums];
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove unnecessary code

INFOPLIST_FILE = DBAttachmentPickerControllerExample/Info.plist;
IPHONEOS_DEPLOYMENT_TARGET = 8.0;
LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
PRODUCT_BUNDLE_IDENTIFIER = DB.DBAttachmentPickerControllerExample;
Copy link
Owner

Choose a reason for hiding this comment

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

Shouldn't change bundle identifier of the example project

@gklka
Copy link
Author

gklka commented Jul 6, 2019

Corrected my repo according your requests. You could now merge my pull request if you like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants