Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retain cycle fix between a parent NSOperation and the entity deletion operation #2556

Open
wants to merge 8 commits into
base: development
Choose a base branch
from

Conversation

AgentFeeble
Copy link
Contributor

This pull request fixes a retain cycle when using tombstone deletion with a predicate. The parent operation strongly retains the deletion operation using associated objects, and the parent operation is added to the dependencies of the deletion operation so that the deletion operation only starts after the parent completes, which causes the deletion operation to strongly reference the parent as well.

segiddins and others added 8 commits November 13, 2015 07:21
* development:
  - ios 12 crash entity is not visible to managedObject context
  fixed typo
  Fixed xcode 9 warnings
  Add test that represents the issue
  Perform access to NSManagedObjectContext's properties only on its queue
  Fixes RestKit#2531. Fixed namespace collision with the official AFNetworking library's implementation of the UIImageView helper category for setting an image from a URL. This will be a breaking change, but should help make people aware of potential unwanted behavior if their project includes both AFNetworking and RestKit.
  Support for previous commit. Extra bonus for RestKit#2529. Implemented tests for the rest of the parameters in the 'success' block.
  Fixes RestKit#2529. Fixed bug where the NSURLRequest passed into 'setImageWithURLRequest:placeholderImage:success:failure:' wasn't being returned in the 'success' completion block when the image was fetched from the cache. The unit test is passing now.
  Adopted library testing syntax and stubbed out a couple of extra tests. The test is still failing at this point since I haven't yet fixed the bug.
  Added unit test for making sure the URLRequest is returned in the completion block of the 'setImageWithURLRequest:placeholderImage:success:failure:' method on the AFNetworking UIImageView extension.
  Remove duplicate word in README
  Method signature update
  Changed RKStringTokenizer to allow tokenization of symbol characters (RestKit#2516)
* development:
  bump version 0.27.3
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.

None yet

3 participants