[#45340] - [6.0] Remove CMS DB aware trait
- Pending
- Medium
- Build: 6.0-dev
- # 45340
- Diff
- Digital-Peak:mvcdbtrait
User tests: Successful: Unsuccessful:
Summary of Changes
Removes the deprecated CMS DatabaseAwareTrait class, as the one from the framework should be used instead.
Testing Instructions
Install test sample data and browse around the front end and back end.
Actual result BEFORE applying this Pull Request
All is working.
Expected result AFTER applying this Pull Request
All is working.
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#441
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Repository Libraries Unit Tests |
| Labels |
Added:
Unit/System Tests
b/c break
PR-6.0-dev
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-05-26 07:24:34 |
| Closed_By | ⇒ | laoneo |
rdeutz
- | Status | Closed | ⇒ | New |
| Closed_Date | 2025-05-26 07:24:34 | ⇒ | |
| Closed_By | laoneo | ⇒ |
| Status | New | ⇒ | Pending |
I have tested this item ✅ successfully on 4b27ea0
Applied the patch and poked around - no issues. But I am not sure I know enough to test this patch!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45340.
I wont to test this patch. Patchtester 'says':
"The file marked for modification does not exist: ruleset.xml"
https://prnt.sc/f6pssNowfkNy
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45340.
Unfortunately, some pull requests cannot be tested with the patch tester. This is one of them. If you still want to test this, you can use one of the pre-built packages for this pull request
Unfortunately, some pull requests cannot be tested with the patch tester. This is one of them. If you still want to test this, you can use one of the pre-built packages for this pull request
@wilsonge while you on it, might want to have a look here as well.