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

IBX-7911: Replaced usage of magic getters for load subtree code paths #1212

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

alongosz
Copy link
Member

@alongosz alongosz commented Mar 14, 2024

Question Answer
JIRA issue IBX-7911
Requires ibexa/core#347, ibexa/user#76
Type bug
Target Ibexa version v4.6
BC breaks no

This PR replaces usages of magic getters with strict ones introduced via ibexa/core#347. See the ibexa/core PR for the reasoning behind this change. The scope of the changes is limited to code paths executed when calling ContentTreeController::loadSubreeAction controller.

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly.
  • Asked for a review.

Copy link

sonarcloud bot commented Mar 14, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

src/lib/UI/Module/ContentTree/NodeFactory.php Outdated Show resolved Hide resolved
@alongosz alongosz force-pushed the ibx-7911-drop-magic-getters branch from 69e2b98 to ba0073c Compare June 5, 2024 15:56
Copy link

sonarcloud bot commented Jun 5, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@alongosz alongosz merged commit 2232a88 into 4.6 Jun 5, 2024
23 checks passed
@alongosz alongosz deleted the ibx-7911-drop-magic-getters branch June 5, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants