Lookup asynchronously and use PHAssets instead of UIImages

[zenangst]

No description provided.

[zenangst]

This is a bit dirty but it helps it from not calling the completion twice. This has to do with how requestImageForAsset resolves assets.

For an asynchronous request, Photos may call your result handler block more than once. Photos first calls the block to provide a low-quality image suitable for displaying temporarily while it prepares a high-quality image. (If low-quality image data is immediately available, the first call may occur before the method returns.)

http://stackoverflow.com/questions/27670325/photos-framework-requestimageforasset-returning-two-results-cant-set-image-vi

@hyperoslo/mystore-ios

[vadymmarkov]

I think should also work without resultHandler 😃

[vadymmarkov]

@zenangst Made some comments, except of that looks good for me.

[RamonGilabert]

It looks good to me!

[zenangst]

@vadymmarkov fixed your comments

[vadymmarkov]

@zenangst 🌟

[RamonGilabert]

Why is this branch restored?

[zenangst]

@RamonGilabert I think I push to it by accident

[ghost]

Does this mean that we can have [PHAsset] instead of [UIImage] for wrapperDidPress and doneButtonDidPress ?

[onmyway133]

@mosn As of https://github.com/hyperoslo/ImagePicker/blob/master/Source/ImagePickerController.swift

You can see that it returns UIImage

public protocol ImagePickerDelegate: class {

  func wrapperDidPress(imagePicker: ImagePickerController, images: [UIImage])
  func doneButtonDidPress(imagePicker: ImagePickerController, images: [UIImage])
  func cancelButtonDidPress(imagePicker: ImagePickerController)
}