Skip to content

Ensure Compatibility to supported range of PHP Versions#32

Merged
nibra merged 18 commits into2.0-devfrom
php81
Jan 25, 2022
Merged

Ensure Compatibility to supported range of PHP Versions#32
nibra merged 18 commits into2.0-devfrom
php81

Conversation

@nibra
Copy link
Contributor

@nibra nibra commented Jan 25, 2022

Pull Request for Issue #

Summary of Changes

Testing Instructions

Documentation Changes Required

mbabker and others added 18 commits June 27, 2020 10:52
Switching from Travis to Drone
Correcting test-class name
strpos with exactly !== false check
Style - Rename class
Fix - Harden use of strpos()
This allows the tests to run on PHP 8.0+.

* Compatibility - Provide separate testsuite for PHP 7.1+
* Build - Update Drone configuration
* Tests - Fix reconstruction test
* Build - Disable PHP 5.4 and PHP 5.5. test environments
  Should be safe enough, if it works on PHP 5.3 and PHP 5.6.
# Conflicts:
#	.drone.jsonnet
#	.drone.yml
#	.gitignore
#	README.md
#	Tests/php53/UriHelperTest.php
#	Tests/php53/UriImmutableTest.php
#	Tests/php53/UriTest.php
#	composer.json
#	phpunit.xml.dist
#	src/UriHelper.php
#	src/UriInterface.php
The merge of from 1.x-dev introduced doubled tests to solve version problems, which are no longer needed.
@nibra nibra merged commit 5046ea5 into 2.0-dev Jan 25, 2022
@nibra nibra deleted the php81 branch January 25, 2022 13:05
@wilsonge
Copy link
Contributor

Removing the array typehint from the buildQuery isn't required - it still was fully compatible with our php version hitlist and additionally broke the cms (see joomla/joomla-cms#37589 )

@laoneo
Copy link
Contributor

laoneo commented Apr 20, 2022

@nibra do you revert that change, otherwise we need to patch the CMS?

@nibra
Copy link
Contributor Author

nibra commented Apr 20, 2022

Will do

@laoneo
Copy link
Contributor

laoneo commented Apr 21, 2022

Here we go #33

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.

9 participants