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

#28896 Create Factory method to find categories in all levels #29021

Conversation

freddyDOTCMS
Copy link
Contributor

@freddyDOTCMS freddyDOTCMS commented Jun 26, 2024

This is the first PR for #28896

Later I am going to use this Factory to create a API Method

Proposed Changes

  • Create Method to find categories in all levels and apply a filter

https://github.com/dotCMS/core/pull/29021/files#diff-bf828c747c99cefe73af7cec527db7f7e64bd4a66ba54a55eb1170f6a2996333R844

  • Update the Cache with the result

https://github.com/dotCMS/core/pull/29021/files#diff-bf828c747c99cefe73af7cec527db7f7e64bd4a66ba54a55eb1170f6a2996333R863

We already have a method to find the ALL the categories and it update the cache after execute the query, so I created a Util method to share the code.

https://github.com/dotCMS/core/pull/29021/files#diff-bf828c747c99cefe73af7cec527db7f7e64bd4a66ba54a55eb1170f6a2996333R168

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (add notes if applicable)

Additional Info

** any additional useful context or info **

Screenshots

Original Updated
** original screenshot ** ** updated screenshot **

This PR fixes: #28896

@freddyDOTCMS freddyDOTCMS marked this pull request as draft June 26, 2024 18:47
@freddyDOTCMS freddyDOTCMS marked this pull request as ready for review June 26, 2024 20:51
@freddyDOTCMS freddyDOTCMS enabled auto-merge June 26, 2024 20:51
…Category' of https://github.com/dotCMS/core into issue-28896-Create-Factory-Method-to-filter-all-levels-Category
@freddyDOTCMS freddyDOTCMS marked this pull request as draft June 26, 2024 23:27
auto-merge was automatically disabled June 26, 2024 23:27

Pull request was converted to draft

@freddyDOTCMS freddyDOTCMS marked this pull request as ready for review June 26, 2024 23:28
@freddyDOTCMS freddyDOTCMS marked this pull request as draft June 26, 2024 23:33
@freddyDOTCMS freddyDOTCMS marked this pull request as ready for review July 1, 2024 20:07
@freddyDOTCMS freddyDOTCMS enabled auto-merge July 1, 2024 20:07
@freddyDOTCMS freddyDOTCMS added this pull request to the merge queue Jul 1, 2024
Merged via the queue into master with commit 7e09771 Jul 2, 2024
29 checks passed
@freddyDOTCMS freddyDOTCMS deleted the issue-28896-Create-Factory-Method-to-filter-all-levels-Category branch July 2, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Updating endpoint to search categories throughout the entire tree of children of a given parent
4 participants