Skip to content

Move getGrantsForTarget method to helper class#26082

Open
kingzacko1 wants to merge 6 commits into
masterfrom
refactor-grantsForTarget-to-entitySharesService
Open

Move getGrantsForTarget method to helper class#26082
kingzacko1 wants to merge 6 commits into
masterfrom
refactor-grantsForTarget-to-entitySharesService

Conversation

@kingzacko1
Copy link
Copy Markdown
Contributor

Description

Refactors getGrantsForTarget from EntityRegistrar to more appropriate EntitySharesService.
/nocl internal refactoring
/prd Graylog2/graylog-plugin-enterprise#14105

Motivation and Context

Needed to use the method for feature work, saw the TODO, figured I'd knock it out.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have requested a documentation update.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@kingzacko1 kingzacko1 requested review from danotorrey and thll May 20, 2026 17:39
@kingzacko1 kingzacko1 changed the title Move getGrantsForTarget method to EntitySharesService Move getGrantsForTarget method to helper class May 21, 2026
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.

1 participant