User tests: Successful: Unsuccessful:
Removes fetchExtensionCompatibility
in UpdateController
class as it is not used since
Install test sample data and browse around the front end and back end.
All is working.
All is working.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org: joomla/Manual#460
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_joomlaupdate |
I'm not sure if it is like that because of the task parameter in the url? Did you test it with the camelCase name?
As this function is not used in core at all, there is nothing to test.
The closest you can do is to case insensitive search for the function name.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-05-26 07:25:01 |
Closed_By | ⇒ | laoneo | |
Labels |
Added:
b/c break
PR-6.0-dev
|
Status | Closed | ⇒ | New |
Closed_Date | 2025-05-26 07:25:01 | ⇒ | |
Closed_By | laoneo | ⇒ |
Status | New | ⇒ | Pending |
I have tested this item ✅ successfully on 2e596a8
Searched code for removed function and poked around in front end and back end. No problems.
I have tested this item ✅ successfully on 2e596a8
Verified that the method is not used:
git clone https://github.com/joomla/joomla-cms -b 6.0-dev 60
# test before the PR
find 60 -name \*.php | xargs grep -i fetchExtensionCompatibility
60/administrator/components/com_joomlaupdate/src/Controller/UpdateController.php: public function fetchExtensionCompatibility()
60/administrator/components/com_joomlaupdate/src/Model/UpdateModel.php: * Called by controller's fetchExtensionCompatibility, which is called via AJAX.
# Apply the PR
gh pr checkout 45436
# Retest
find 60 -name \*.php | xargs grep -i fetchExtensionCompatibility
# nothing found :)
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
RTC
|
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-06-12 07:27:04 |
Closed_By | ⇒ | Bodge-IT |
can we please fix the camelCase for the batch function... it's not a b/c break and makes it constant to our code base and would be a good point in time before documentation
https://3v4l.org/bQHFu