User tests: Successful: Unsuccessful:
s/compatability/compatibility
Status | New | ⇒ | Pending |
Category | ⇒ | External Library Libraries Composer Change |
I have tested this item
Code review.
I have tested this item
Code review.
It is my understanding that files in vendor
directory should be made upstream and then migrated downstream.
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
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 :)
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.
Im not going to argue for a typo.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-03-21 21:39:24 |
Closed_By | ⇒ | PhilETaylor | |
Labels |
Added:
?
?
|
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.
upstream submitted too joomla-framework/application#94