joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[6.0] Remove CMS DB aware trait

Open laoneo opened this issue 8 months ago • 5 comments

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:

  • [x] No documentation changes for docs.joomla.org needed

  • [x] Pull Request link for manual.joomla.org: https://github.com/joomla/Manual/pull/441

  • [ ] No documentation changes for manual.joomla.org needed

laoneo avatar Apr 17 '25 06:04 laoneo

@wilsonge while you on it, might want to have a look here as well.

laoneo avatar May 05 '25 06:05 laoneo

I have tested this item :white_check_mark: successfully on 4b27ea02da1939b024155fef5806bb797f544834

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.

ceford avatar May 30 '25 13:05 ceford

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.

peterpeter avatar May 31 '25 21:05 peterpeter

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

brianteeman avatar Jun 01 '25 04:06 brianteeman

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

brianteeman avatar Jun 01 '25 04:06 brianteeman