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

Default transformer use #54

Merged
merged 2 commits into from
Jan 29, 2014
Merged

Default transformer use #54

merged 2 commits into from
Jan 29, 2014

Conversation

patrickhartling
Copy link
Contributor

The Core Data documentation on transformable attributes has this caveat:

Important: Although the default transformer is the transformer specified by NSKeyedUnarchiveFromDataTransformerName, this transformer is actually used in reverse. If you specify the default transformer explicitly, Core Data would use it “in the wrong direction.”

I made adjustments to the use of value transformers to account for the default case. Test cases have been added to cover the cases of using the default transformer and a custom transformer.

According to the Core Data documentation, the default transformer is
used in reverse.
@ghost ghost assigned gavin-black Jan 28, 2014
gavin-black added a commit that referenced this pull request Jan 29, 2014
@gavin-black gavin-black merged commit ad8fa6b into project-imas:master Jan 29, 2014
@gavin-black
Copy link
Member

Thanks as always for the contribution, will check out the uniq-cols issues you mentioned this week.

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.

2 participants