Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify params model state for com_contact, com_newsfeeds #34894

Merged

Conversation

joomdonation
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

Use $app->getParams() to get $params variable. Less code, consistent with other model classes.

Testing Instructions

Code review should be enough.

@wilsonge
Copy link
Contributor

https://github.com/joomla/joomla-cms/blob/staging/components/com_content/models/category.php#L117-L127 We have the same code in com_content at the moment.

Newsfeeds doesn't even merge the menu specific items :) https://github.com/joomla/joomla-cms/blob/staging/components/com_newsfeeds/models/category.php#L197

Overall I think this is right - we just need to get everything consistent at once.

@joomdonation
Copy link
Contributor Author

@wilsonge We handle it different in com_content, thus I made a separate PR #34892 for com_content.

For newsfeeds, yes, we have it in different way, too. So 3 components, should work in the same way but we handle it different in each component :D. I will add the same change to com_newsfeeds into this PR

@joomdonation joomdonation changed the title [4.0] Simplify params model state for com_contact [4.0] Simplify params model state for com_contact, com_newsfeeds Jul 26, 2021
@joomdonation
Copy link
Contributor Author

OK. I use similar code for com_newsfeeds. Assume it is accepted, we will have consistent code for the three components :).

@chmst chmst changed the base branch from 4.0-dev to 4.1-dev January 31, 2022 15:53
@chmst chmst removed the PR-4.0-dev label Jan 31, 2022
@HLeithner HLeithner changed the base branch from 4.1-dev to 4.2-dev June 27, 2022 13:07
@HLeithner
Copy link
Member

This pull request has automatically rebased to 4.2-dev.

@joomla-bot
Copy link
Contributor

This pull requests has been automatically converted to the PSR-12 coding standard.

@Quy Quy added PR-4.2-dev and removed PR-4.1-dev labels Jul 3, 2022
@rdeutz rdeutz changed the title [4.0] Simplify params model state for com_contact, com_newsfeeds Simplify params model state for com_contact, com_newsfeeds Oct 21, 2022
@rdeutz rdeutz changed the base branch from 4.2-dev to 4.3-dev October 21, 2022 14:06
@bembelimen bembelimen merged commit b25c0cd into joomla:4.3-dev Oct 21, 2022
@bembelimen
Copy link
Contributor

Thx

@bembelimen bembelimen added this to the Joomla! 4.3.0 milestone Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants