Page MenuHomeDevCentral

Bump jsonmapper to v1.1.1
ClosedPublic

Authored by dereckson on Jan 31 2017, 01:26.
Tags
None
Referenced Files
F12390569: D896.id2276.diff
Sat, Nov 1, 11:14
F12383889: D896.id2273.diff
Fri, Oct 31, 21:34
Unknown Object (File)
Fri, Oct 31, 14:26
Unknown Object (File)
Fri, Oct 31, 01:09
Unknown Object (File)
Thu, Oct 30, 16:45
Unknown Object (File)
Thu, Oct 30, 16:13
Unknown Object (File)
Tue, Oct 28, 05:10
Unknown Object (File)
Mon, Oct 27, 18:30
Subscribers
None

Details

Summary

Composer installed 0.1.3, as the package version constraint was ~0.1.0.

This version can't correctly handle a property documented as mixed:

Notification
/**
 * The notification's source payload, data or message
 *
 * @var mixed
 */
public $rawContent;

With this version, JsonMapper raises the following issue:

Class '\Nasqueron\Notifications\Notifications\mixed' not found

This is a known bug, fixed upstream:
https://github.com/cweiske/jsonmapper/issues/33

The version is bumped to get this fix.

Test Plan

Run tests to ensure current mappings still work.

Test this code works:

private function getNotification () : Notification {
    $mapper = new \JsonMapper();
    return $mapper->map(
        json_decode($this->rawRequestContent),
        new Notification
    );
}

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
No Test Coverage
Branch
bump-jsonmapper (branched from master)
Build Status
Buildable 1381
Build 1629: arc lint + arc unit

Event Timeline

dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
This revision is now accepted and ready to land.Jan 31 2017, 01:34
This revision was automatically updated to reflect the committed changes.