[#42717] - Change the getItem return type to stdClass and make the CMSObject inherit it
- Fixed in Code Base
- 27 Feb 2024
- Medium
- Build: 5.1-dev
- # 42717
- Diff
- Digital-Peak:model/admin/getitem2
User tests: Successful: Unsuccessful:
Summary of Changes
To get away from the CMSObject class we need a step by step process in the getItem function of the admin model. Alternative to #42714 which makes the CMSObject a stdClass and changes the return type to it. It is a more lightweight solution as no new deprecated flag is introduced. It has basically the same effect as when we change the flag in 6.0.
Testing Instructions
Create an article in the back end. With as many content plugins as possible.
Actual result BEFORE applying this Pull Request
All works.
Expected result AFTER applying this Pull Request
All works.
Link to documentations
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#232
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_installer Libraries |
| Labels |
Added:
PR-5.1-dev
|
||
| Title |
|
||||||
I have tested this item ✅ successfully on 080cca9
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42717.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42717.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-02-27 10:45:34 |
| Closed_By | ⇒ | LadySolveig | |
| Labels |
Added:
RTC
|
||
Thank you @laoneo and also for testing @crommie and @viocassel
I have tested this item ✅ successfully on 080cca9
I've created an article with a number of custom fields and schema stuff. No difference before / after patch so I guess it works.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42717.