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.

Sign up for GitHub

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

Jump to bottom

Null handling #222

Closed
wants to merge 4 commits into from
Closed

Null handling #222

wants to merge 4 commits into from

Commits on Nov 8, 2012

  1. Add InitializedObjectConstructor as a test double for DoctrineObjectC… 

    …onstructor.

Add tests to BaseSerializationTest:
* testNull when expecting types
* testNumerics when expecting numeric types
* testDateTime
* testDeserializingNull when using InitializedObjectConstructor

Add content for blog_post_unauthored and date_time:
* JsonSerializationTest
* xml/blog_post_unauthored.xml
* xml/date_time.xml
* yml/blog_post_unauthored.yml
* yml/date_time.yml

So, from the phpunit test results, we see:
* fails when serializing a simple DateTime
* error when deserialing a null when expecting a DateTime
* error: no support for the "float" alias for "double"
* error when deserializing: a null value will not overwrite a non-null property in the initialized object
    @robocoder
    robocoder committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    97ec3b5 View commit details
    Browse the repository at this point in the history

  2. GraphNavigator support for "float" alias.

    @robocoder
    robocoder committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    87c232c View commit details
    Browse the repository at this point in the history

  3. DateTimeHandler: 

    * fix serialization/deserialization of DateTime
* there's no change to parseDateTime() other than removing DOS carriage returns
    @robocoder
    robocoder committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    cc94941 View commit details
    Browse the repository at this point in the history

  4. when deserializing, null values should overwrite properties in the in… 

    …itialized object
    @robocoder
    robocoder committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    e1ea08d View commit details
    Browse the repository at this point in the history