User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Rename SysinfoModel.php
to SysInfoModel.php
.
Typo in Joomla\Component\Privacy\Site\Service\Router
.
Code review? CC @wilsonge
No.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_admin Front End |
typo looks correct to me
not sure about the camelcase name change - renaming this way always causes issues
Maybe just leave the file name as it is and only correct the typo?
@richard67 that won't fix the issue. We need to either rename the file or rename the class and change all instances of it in code.
Anyways, this is fine, I think, if updates between alphas aren't officially supported.
Hm, there are more instances of this:
\Joomla\Component\Menus\Administrator\Field\ComponentsCategoryField
\Joomla\Component\Menus\Administrator\Field\MenuOrderingField
Rename classes or rename files? Renaming files also requires updating XML files but it would be nice to have correctly cased classes at some point.
Labels |
Added:
?
|
Hmm, I don't remember right now how such renames were handled in past, was not involved in that, just reading some PR when passing by long time ago. George maybe remembers that better.
We had to rename them in two commits - to a different name and back again for git to be happy. Then that still doesn't work on shared hosts (but at this stage in j4 we don't guarentee b/c there so it's not a problem).
TL/DR - just rename the files and let's do this correctly and set the correct precedent :)
Category | Administration com_admin Front End | ⇒ | Administration com_admin com_menus Front End |
Category | Administration com_admin Front End com_menus | ⇒ | Administration com_admin com_categories com_menus Front End |
Hm, I think it's better to rename SysInfoModel
and revert the filename change. The view is called Sysinfo
so it tries to load SysinfoModel
by default anyways.
Category | Administration com_admin Front End com_menus com_categories | ⇒ | Administration com_categories com_menus Front End |
Category | Administration Front End com_menus com_categories | ⇒ | Administration com_admin com_categories com_menus Front End |
Unless we want to rename the view, which also means updating all links to it.
No I think keeping the existing view type makes sense
Category | Administration Front End com_menus com_categories com_admin | ⇒ | Administration com_admin com_categories com_menus Front End Libraries |
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-07-10 09:33:21 |
Closed_By | ⇒ | wilsonge |
Do these changes also work under Windows?
In my ticket (#23725) the developers also tried to rename the StringController and encountered problems with the update mechanism when changing a file name to upper or lower case under Windows.
Quote #23762:
I have tested this item unsuccessfully on a0538dc
In windows this PR breaks the whole deletion process, because
realpath
converts all slashes to\
, so it's always not true (and the JFile::delete will never be called).Additional there will not be 2 files in windows, so the deletion process (if it works) would delete the whole file without replacement.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23762.
@Orgoth this PR fixes only new files added in 4.0. Updates between 4.0 dev builds aren't supported, so if you try to update from older 4.0 version, you'll have to apply the filename changes manually or just use a new 4.0 installation.
This, however, won't be an issue when updating from 3.x because these files do not exist in 3.x.
Updates from Nightly > Nightly Build will be supported at Beta.
Lets just rename the class back to lowercase I think - i mean uppercase SQL is just a nice to have
The casing in the field type hasn't changed from 3.x so I'd rather not change it
Don't know if changing file casing causes issues at this point. If it does, we can rename the class instead?