? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
27 Feb 2020

Updates the SQLSrv extension to 5.8.0 which also supports PHP 7.4 https://pecl.php.net/package/sqlsrv/5.8.0

See appveyor tests pass on new version to test

avatar wilsonge wilsonge - open - 27 Feb 2020
avatar wilsonge wilsonge - change - 27 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Feb 2020
Category Unit Tests
avatar wilsonge
wilsonge - comment - 27 Feb 2020

@mbabker at a quick glance can you see why https://github.com/joomla/joomla-cms/blame/staging/tests/unit/core/helper.php#L55 is failing so hard in Appveyor PHP 7.4 - seems to be the same thing making travis fail - so don't think it's environment specific

avatar mbabker
mbabker - comment - 27 Feb 2020

If I had to guess, something to do with the deprecated $context parameter of the callable for error handlers being a null value, and PHPUnit 4.8 not being any kind of PHP 7 compatible expects it to always be an array as was the case before it was deprecated in PHP 7.2. So, you're probably going to need some handling in that Closure to deal with that.

avatar wilsonge
wilsonge - comment - 27 Feb 2020

I saw the context parameter had gone in the later versions of symfony and was afraid you were going to say that

avatar wilsonge wilsonge - change - 5 Mar 2020
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 28 Mar 2020

Merging for now so the tests run - as i believe they are the same failures on appveyor and they can be enabled then in the same set of commits

avatar wilsonge wilsonge - close - 28 Mar 2020
avatar wilsonge wilsonge - merge - 28 Mar 2020
avatar wilsonge wilsonge - change - 28 Mar 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-03-28 18:59:38
Closed_By wilsonge
Labels Added: ?
Removed: ?

Add a Comment

Login with GitHub to post a comment