Feeling Lucky
PR-4.3-dev
?
[#34892] - Simplify com_content params model state
- Fixed in Code Base
- 21 Oct 2022
- Medium
- Build: 4.0-dev
- # 34892
- Diff
- joomdonation:category_model_params_state
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
The $params variable returned from $app->getParams(); already contains component parameters merged with active menu item parameters, so no need to merge it with active menu item parameters again.
For reference, we do it right in Categories model https://github.com/joomla/joomla-cms/blob/4.0-dev/components/com_content/src/Model/CategoriesModel.php#L69-L70
Testing Instructions
Code review only.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
| Title |
|
||||||
| Labels |
Added:
?
|
||
joomla-bot
- comment
- 27 Jun 2022
This pull requests has been automatically converted to the PSR-12 coding standard.
rdeutz
- | Title |
|
||||||
| Labels |
Added:
?
PR-4.3-dev
?
Removed: ? |
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-10-21 16:10:39 |
| Closed_By | ⇒ | bembelimen | |
| Labels |
Removed:
?
|
||
bembelimen
- comment
- 21 Oct 2022
Thx
This pull request has automatically rebased to 4.2-dev.