? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
21 Mar 2021

s/compatability/compatibility

avatar PhilETaylor PhilETaylor - open - 21 Mar 2021
avatar PhilETaylor PhilETaylor - change - 21 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2021
Category External Library Libraries Composer Change
avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

upstream submitted too joomla-framework/application#94

avatar richard67 richard67 - test_item - 21 Mar 2021 - Tested successfully
avatar richard67
richard67 - comment - 21 Mar 2021

I have tested this item successfully on 9bf6767

Code review.


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

avatar SharkyKZ SharkyKZ - test_item - 21 Mar 2021 - Tested unsuccessfully
avatar SharkyKZ
SharkyKZ - comment - 21 Mar 2021

I have tested this item ? unsuccessfully on 9bf6767

Code review.


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

avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

@SharkyKZ ????

avatar Quy
Quy - comment - 21 Mar 2021

It is my understanding that files in vendor directory should be made upstream and then migrated downstream.

avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

for those installable by composer maybe.... but are not these forked versions that are in the 4.0-dev?

upstream submitted too joomla-framework/application#94

avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

and looking at the last modified commits of the files in https://github.com/joomla/joomla-cms/tree/4.0-dev/libraries/src would suggest that Im right :)

avatar richard67
richard67 - comment - 21 Mar 2021

Last time this file was fetched from the framework was in 2019 or earlier. This commit in the framework hasn't arrived in the CMS yet: joomla-framework/application@134f298 .

But that's the only difference between the CMS and the framework version.

So here it could indeed make sense to do the change in the framework and later pull it into the CMS.

We have other framework stuff were we have more differences between framework and CMS, like e.g. database, and where we changed things in the CMS and later pulled it up into the framework, and I thought that's the case here, too, that's why I haven't insisted in doing it upstream in the framework.

All that said only for J3 (staging, 3.10-dev). For J4 it's definitely different.

avatar PhilETaylor
PhilETaylor - comment - 21 Mar 2021

Im not going to argue for a typo.

avatar PhilETaylor PhilETaylor - change - 21 Mar 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-03-21 21:39:24
Closed_By PhilETaylor
Labels Added: ? ?
avatar PhilETaylor PhilETaylor - close - 21 Mar 2021
avatar richard67
richard67 - comment - 21 Mar 2021

See also joomla-framework/application#94 and joomla-framework/application#95 ... we gotta pull the changes from the framework as soon as the PR's there got merged.

Thanks @PhilETaylor for reporting and making the PR's there.

Add a Comment

Login with GitHub to post a comment