Primary Key Predicate API's should Coerce Types #758

Closed
blakewatters opened this Issue May 23, 2012 · 0 comments

Projects

None yet

1 participant

@blakewatters
Member

A common issue we have run into on our team is related to initializing an NSPredicate that targets a numeric property with a string value. This results in odd behavior such as fetched results controllers that don't get notifications you would expect. Now that we have the NSEntity primary key extensions for properties, we have a nice choke point to provide a coercion to the correct type for the primary key attribute.

@blakewatters blakewatters was assigned May 23, 2012
@s2m-updater s2m-updater pushed a commit to SinnerSchraderMobileMirrors/RestKit that referenced this issue May 24, 2012
@blakewatters blakewatters Implemented support for type coercions in primaryKeyAttribute API's. …
…closes #758
98c8780
@signalsandstuff signalsandstuff added a commit to signalsandstuff/RestKit that referenced this issue May 31, 2012
@signalsandstuff signalsandstuff Merge branch 'master' of git://github.com/RestKit/RestKit
* 'master' of git://github.com/RestKit/RestKit: (360 commits)
  Fix issue where grouping objects by section in RKTableController and loading an empty collection would fail to refresh table view.
  Fix type mismatch assertion in cases where JSON/XML payload contains a string value for an integer primary key. fixes #754
  Fix static analyzer warning
  Cleanups to and test coverage expansion of RKTableController API's. refs #765
  Update RKTestFactory to silence logging output during setup/tear down operations. closes #764
  Reworked table controller state into a bit mask to coalesce state change into a single observation. fixes #753
  Implemented support for type coercions in primaryKeyAttribute API's. closes #758
  Cancel current object loader when loading a table view
  Clean up inconsistencies with 'cancelled' and 'cancel' to match Cocoa (isCancelled)
  Ensure managed object connections are established across appropriate managed object contexts
  Add RKLog helper for displaying detailed key-value validation failure information. closes #750
  Bump version to 0.10.1
  Fix for RestKit#709 - [RKManagedObjectThreadSafeInvocation serializeManagedObjectsForArgument:withKeyPaths:] is not properly serializing managed objects
  Ensure creation of Application Data directory when initializing Managed Object Store. fixes #657
  Add tests verifying expected behavior for availability of RKRequest response property. closes #527
  Eliminate duplication of request method to string logic in [RKRequest HTTPMethod]
  Migrate RKResponse property from RKObjectLoader to RKRequest. refs #527
  Fix static analyzer issues
  Replace v0.9.4 references with 0.10.0
  Rework intermittently failing tests
  ...

Conflicts:
	Code/Network/RKRequest.m
6b46630
@smclelland smclelland pushed a commit to smclelland/RestKit that referenced this issue Jun 7, 2012
smclelland Merge branch 'master' of git://github.com/RestKit/RestKit
* 'master' of git://github.com/RestKit/RestKit: (109 commits)
  Fix issue where grouping objects by section in RKTableController and loading an empty collection would fail to refresh table view.
  Fix type mismatch assertion in cases where JSON/XML payload contains a string value for an integer primary key. fixes #754
  Fix static analyzer warning
  Cleanups to and test coverage expansion of RKTableController API's. refs #765
  Update RKTestFactory to silence logging output during setup/tear down operations. closes #764
  Reworked table controller state into a bit mask to coalesce state change into a single observation. fixes #753
  Implemented support for type coercions in primaryKeyAttribute API's. closes #758
  Cancel current object loader when loading a table view
  Clean up inconsistencies with 'cancelled' and 'cancel' to match Cocoa (isCancelled)
  Ensure managed object connections are established across appropriate managed object contexts
  Add RKLog helper for displaying detailed key-value validation failure information. closes #750
  Bump version to 0.10.1
  Fix for RestKit#709 - [RKManagedObjectThreadSafeInvocation serializeManagedObjectsForArgument:withKeyPaths:] is not properly serializing managed objects
  Ensure creation of Application Data directory when initializing Managed Object Store. fixes #657
  Add tests verifying expected behavior for availability of RKRequest response property. closes #527
  Eliminate duplication of request method to string logic in [RKRequest HTTPMethod]
  Migrate RKResponse property from RKObjectLoader to RKRequest. refs #527
  Fix static analyzer issues
  Replace v0.9.4 references with 0.10.0
  Rework intermittently failing tests
  ...

Conflicts:
	Code/Support/Parsers/JSON/RKJSONParserJSONKit.m
c126532
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment