? ? Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
16 Jun 2019

Hopefully makeing Appveyor happy again.

avatar Hackwar Hackwar - open - 16 Jun 2019
avatar Hackwar Hackwar - change - 16 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jun 2019
Category External Library Libraries Composer Change
avatar zero-24
zero-24 - comment - 16 Jun 2019

When this helps we should also patch this upstream in the string package: https://github.com/joomla-framework/string/blob/master/src/phputf8/utf8.php#L45 it seams some other changes have been done there in the meantime too.

avatar richard67
richard67 - comment - 16 Jun 2019

It seems before making Appveyor happy it needs to wake him up.

avatar richard67
richard67 - comment - 16 Jun 2019

Hooray, appveyor passed with success!!!

avatar richard67
richard67 - comment - 16 Jun 2019

I have tested this item successfully on 264fd78

Before this PR, Appveyor tests failed here:
https://github.com/joomla/joomla-cms/blob/staging/libraries/vendor/joomla/string/src/phputf8/utf8.php#L38
with error

Warning: A non-numeric value encountered in C:\projects\joomla-cms\libraries\vendor\joomla\string\src\phputf8\utf8.php on line 38

With this PR is passes.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25234.

avatar richard67 richard67 - test_item - 16 Jun 2019 - Tested successfully
avatar richard67
richard67 - comment - 16 Jun 2019

@zero-24 PR for framwork package see joomla-framework/string#22.

avatar richard67
richard67 - comment - 16 Jun 2019

@zero-24 Does it need another PR in the framework package for the 2.0-dev branch? No, clarified.

avatar Hackwar
Hackwar - comment - 16 Jun 2019

Since this is the wrong way to fix this, I'm closing this one.

avatar Hackwar Hackwar - change - 16 Jun 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-06-16 21:01:44
Closed_By Hackwar
Labels Added: ? ?
avatar Hackwar Hackwar - close - 16 Jun 2019
avatar richard67
richard67 - comment - 16 Jun 2019

@Hackwar Please check PR #25243 .

Add a Comment

Login with GitHub to post a comment