? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
5 Feb 2017

Summary of Changes

This PR makes the JHttp class set extend the Framework's HTTP package for 4.0. This should improve interoperability with the Framework packages as now the CMS' HTTP adapters will be compatible with the Framework (mainly affects the OAuth and social packages, the latter having been removed from the CMS).

This backports some structural changes from the Framework's package as well. First, the requirement that the options parameter be a Registry object being loosened to an array or ArrayAccess implementation (Registry does implement this). Next, transports aren't typehinted to JUri now but the Joomla\Uri\UriInterface. Lastly, this now results in PSR-7 compatible response objects being used.

Testing Instructions

The HTTP API should continue to function correctly.

Documentation Changes Required

There is a B/C break in the transport implementations, the signature for the request method now uses the Framework's Joomla\Http\TransportInterface::request() signature (the typehint of the URI object is changed from JUri to Joomla\Uri\UriInterface).

avatar mbabker mbabker - open - 5 Feb 2017
avatar mbabker mbabker - change - 5 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Feb 2017
Category Repository External Library Libraries
avatar wilsonge wilsonge - change - 5 Feb 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-05 17:40:45
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 5 Feb 2017
avatar wilsonge wilsonge - merge - 5 Feb 2017
avatar wilsonge
wilsonge - comment - 5 Feb 2017

Nice work :)

Add a Comment

Login with GitHub to post a comment