HomeDevCentral

Bump jsonmapper to v1.1.1

Description

Bump jsonmapper to v1.1.1

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
    );
}

Reviewers: dereckson

Reviewed By: dereckson

Differential Revision: https://devcentral.nasqueron.org/D896

Details

Provenance
derecksonAuthored on Jan 31 2017, 01:21
Reviewer
dereckson
Differential Revision
D896: Bump jsonmapper to v1.1.1
Parents
rNOTIF5760d2f2f0f5: Prune unused use statement
Branches
Unknown
Tags
Unknown
Tasks
T979: Gate for notifications in notifications format