fix: user manager fix

[Shivam-nagar23]

Description

Fixes https://github.com/devtron-labs/sprint-tasks/issues/2648

Checklist:

• The title of the PR states what changed and the related issues number (used for the release note).
• Does this PR requires documentation updates?
• I've updated documentation as required by this PR.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have tested it for all user roles.
• I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?

Summary by Bito

This pull request fixes a bug in the user authorization logic within the UserRestHandler, refining access conditions for chart group entities. The enhancements improve security and functionality in user management, particularly with multiple role filters.

[github-actions]

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n

[Copilot]

Pull Request Overview

This PR fixes user authorization logic for chart group entities in the user management system. The fix addresses a scenario where chart group authorization was not properly handled when role groups were also being assigned.

• Updated the authorization logic for chart group entities to properly handle cases with role groups
• Added conditional logic to defer chart group authorization to role group validation when both are present
• Improved code comments to clarify the authorization flow

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[github-actions]

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud