lionheart / openradar-mirror

A mirror of radars pulled from http://openradar.me/.
246 stars 17 forks source link

27182847: Collection View Batch Update in SamplePhotosApp on iOS 9 #15270

Open openradar-mirror opened 8 years ago

openradar-mirror commented 8 years ago

Description

I've tested the same project with Xcode 8 beta 2 and I can't reproduce the crash. Great job. 👍

Summary: I am using your method called photoLibraryDidChange in my iOS 9 app. This makes it crash all the time when Photos Library is updated. Because your sample is iOS 10 only, I had to refactor it. Code attached.

Steps to Reproduce:

  1. Delete one photo in Simulator
  2. Open sample app

Expected Results: New photo appear

Actual Results: I have crash message: "attempt to delete and reload the same index path"

Version: Example app using Photos framework

https://developer.apple.com/library/ios/samplecode/UsingPhotosFramework/Introduction/Intro.html

Notes: I am attaching my app project so you can run it yourself. My app displays screenshots from Screenshots album in a collection view. Press command + S to make a screenshot and drag it into Simulator. It should appear in the Altershot.

I am attaching my refactored iOS 9 sample code. Related pre iOS 10 question http://stackoverflow.com/questions/29337765/crash-attempt-to-delete-and-reload-the-same-index-path#29370012

I will keep you updated. I will try to run your code as it is in iOS 10. Maybe this is bug in UIKit which you fixed in iOS 10.

Configuration: Xcode 7.3.1 iOS 9 iPhone 5s Simulator

Attachments: 'Altershot.zip' and 'update.swift' were successfully uploaded.

Product Version: Created: 2016-07-05T22:18:05.830010 Originated: 2016-07-09T00:00:00 Open Radar Link: http://www.openradar.me/27182847

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200

openradar-mirror commented 8 years ago

Modified: 2016-07-09T10:49:32.477200