User tests: Successful: Unsuccessful:
Factory::getSession() has been deprecated for a long time. This PR removes it and also all usages in our core code.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_finder com_joomlaupdate com_postinstall Front End com_contact Installation Libraries Plugins |
| Labels |
Added:
PR-6.0-dev
|
||
| Labels |
Added:
bug
|
||
I have tested this item ✅ successfully on bcb2dd6
| Status | Pending | ⇒ | Ready to Commit |
RTC
Can we replace the usages in our core code for now and remove the method in the next major release? The method is still use in our core code until now, so it is very likely it is also used by third party extensions and these extensions will be broken in the upcoming 6.0 release.
| Labels |
Added:
RTC
|
||
I moved it to the b/c plugin, I didn't restored Factory::$session since it has not been initialized, so it's pretty useless anyway.
| Title |
|
||||||
| Labels |
Added:
b/c break
PR-6.1-dev
|
||
I removed the removal of the getSession() method so that no deprecated functions get called anymore. The removal can and should be done in 7.0. If the tests are green it can be merged.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-01-09 10:24:08 |
| Closed_By | ⇒ | HLeithner |
thanks
| Title |
|
||||||
I have tested this item ✅ successfully on bcb2dd6
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45869.