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

Attempt to delete and reload the same index path #67

Merged
merged 9 commits into from
Feb 10, 2016

Conversation

polqf
Copy link
Owner

@polqf polqf commented Feb 5, 2016

🎩 What? Why?

As #66 pointed out, there is a case where the RRC is provoking a crash on the tableView.endUpdates()

This seems that is happening due to a deletion + an insertion of the same object in a background thread.

👻 GIF

Closes #66

@polqf polqf added the bug label Feb 5, 2016
let dummy = Task()
value = dummy.primaryKeyValue()
}
it("should be nil") {
Copy link
Collaborator

Choose a reason for hiding this comment

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

should be 0

@fernandogcat
Copy link
Collaborator

👍

@polqf polqf force-pushed the bug/deleteAndReloadSameIndex branch from eaf6db2 to 2638758 Compare February 10, 2016 11:28
let dummy2 = Task()
value = dummy1.hasSamePrimaryKeyValue(dummy2)
}
it("should return false") {
Copy link
Contributor

Choose a reason for hiding this comment

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

should return true

@mariabernis
Copy link
Contributor

🍏 👍

@polqf polqf force-pushed the bug/deleteAndReloadSameIndex branch from b809bcc to 15da28f Compare February 10, 2016 14:48
polqf added a commit that referenced this pull request Feb 10, 2016
Attempt to delete and reload the same index path
@polqf polqf merged commit 36bb736 into master Feb 10, 2016
@polqf polqf deleted the bug/deleteAndReloadSameIndex branch February 10, 2016 14:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants