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

Custom Property Mapping #8

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Custom Property Mapping #8

wants to merge 44 commits into from

Conversation

Buju77
Copy link
Collaborator

@Buju77 Buju77 commented Jul 15, 2015

New Features Since 0.2.0

  • Custom property name mapping: JAGPropertyMapping
  • Support converting NSData (eg. into Base64 strings): convertToData and convertFromData
  • Support converting enums: convertToEnum and convertFromEnum
  • Support for auto converting to/from snake_case: enableSnakeCaseSupport
  • Support for ingoring nil/null vales: shouldIngoreNullValues
  • Support to ignore weak properties for serialization: shouldConvertWeakProperties
  • Extracted and restructured some methods so JAGPropertyConverter can be subclassed: JAGPropertyConverter+Subclass.h
  • Enhanced identifyDict: it will now pass in the dictionary name (breaking change!)
  • Added isBoolean to JAGProperty

Yen-Chia Lin and others added 29 commits November 25, 2014 19:27
…concept)

Fixed convert to float unit tests
fixed error in converting to BOOL values (didn't work)
fixed float number converter unit tests (didn't work when iOS Simulator has a German or other region set)
fixed podspec
…SNull values

added new converter configuration: shouldConvertSnakeCaseToCamelCase to convert snake case JSON keys to camelcase properties
…ctionary

fixed a bug on iOS 8: class_copyPropertyList will return 4 additional properties defined in NSObject. -[RTJAGPropertyFinder propertiesForSubclass:] will now ignore those.
renamed shouldConvertSnakeCaseToCamelCase to enableSnakeCaseSupport
renamed JAGPropertyMappingProtocol methods
extended ExampleTest.m with additional features
added dictName property to the identifyDict block to identify different classes with the same property names
extended ExampleTest with additional features
ADD ignoreProperties protocol method
…ses of JAGPropertyConverter can simply override the public method (decomposeObject:) to do additional pre- and/or post-processing.
…e-mapping

2014 11 feature custom property name mapping
…implementing this protocol are now much simpler. No need to call super anymore. Combining the collection from all super classes are done directly by JAGPropertyConverter

added unit tests for inherited custom property mapping
…yFromAllInheritanceForObject: can now be used by subclasses of JAGPropertyConverter
…only 1 method to implement for both directions. JAGPropertyConverter will automatically swap the dictionary when converting from JSON into model

Added NSDictionary+JAGKeyValueSwapping.h
…to implement now). JAGPropertyConverter will automatically swap the dictionary when converting from JSON into model
Yen-Chia Lin and others added 9 commits September 17, 2015 11:10
ADD `NSDictionary+JsonString` category to get a pretty printed JSON string from the contents of a dictionary (only in test target)
#4)

ADD possibility to serialize `NSNull` values (opt-in) even though `converter.shouldIgnoreNullValues` is set to `YES`
ADD nullability and generics to `JAGPropertyMapping.h`
MOD converted `JAGOutputType` to `NS_ENUM`
MOD increased minimum deployment target to 7.0 (was 5.0) (otherwise we couldn't use Swift in unit tests)
Updated Xcode 8.1 recommended settings
Increased deployment target to 8.0
@jagill
Copy link
Owner

jagill commented Nov 15, 2016

Hi Buju77! I'm not doing much ObjC development these days, so I'm not able to properly maintain this package. Would you like push access to this repo?

@Buju77
Copy link
Collaborator Author

Buju77 commented Nov 15, 2016

@jagill sry to hear that you are not developing ObjC anymore ;)

If you don't mind, push access would be really cool :)

@jagill
Copy link
Owner

jagill commented Nov 17, 2016

Push access granted! Thanks for tending to this.

@Buju77
Copy link
Collaborator Author

Buju77 commented Nov 24, 2016

Cool thank you very much! When I have time in the upcoming days I'll update the Readme a little more and merge this PR then. 😄

Maximilian Landsmann and others added 6 commits January 24, 2017 16:54
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.

3 participants