[#35888] - Getdbo patch fixes #35884
- Closed
- 31 Dec 2021
- Medium
- Build: 4.0-dev
- # 35888
- Diff
- basd82:getdbo-patch
User tests: Successful: Unsuccessful:
Made pull request like sugested by Richard Fath
I made this change to 4.0-dev branch becouse they are fixes that are only valid voor 4.x
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content com_finder com_users |
| Title |
|
||||||
Pull Request for Issue # 35884
Summary of Changes
Replaced Getdbo() becouse it is deprecated
Testing Instructions
Test if the following component's in the frontend are still working like before:
com_users
com_content
com_finder
Actual result BEFORE applying this Pull Request
Messages about depricated calls to getDbo() in adminstrator/log/depricated.php
Expected result AFTER applying this Pull Request
No more log messages for these components
Documentation Changes Required
None that i can think off
Pull Request for Issue # 35884
Summary of Changes
Replaced Getdbo() becouse it is deprecated
Testing Instructions
Test if the following component's in the frontend are still working like before:
com_users
com_content
com_finder
Actual result BEFORE applying this Pull Request
Messages about depricated calls to getDbo() in adminstrator/log/depricated.php
Expected result AFTER applying this Pull Request
No more log messages for these components
Documentation Changes Required
None that i can think off
This PR needs a description and testing instructions as in all other properly documented PRs.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35888.
Added the request pr
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-31 17:00:27 |
| Closed_By | ⇒ | alikon | |
| Labels |
Added:
?
|
||
| Status | Closed | ⇒ | New |
| Closed_Date | 2021-12-31 17:00:27 | ⇒ | |
| Closed_By | alikon | ⇒ |
| Status | New | ⇒ | Pending |
There is no point reopening this after you threw your toys out your cot after being asked to make 12 lines of copy and paste changes - it’s no better than the PR you proposed.
I’ll make a PR later tonight that does it correctly, completely, in all places
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-31 17:14:27 |
| Closed_By | ⇒ | alikon |
looking forward for your pr
This PR needs a description and testing instructions as in all other properly documented PRs.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35888.