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 #76

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
Required by ibexa/admin-ui#1212
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 and introduces strict getters for UserSetting Value Object, needed by ibexa/admin-ui#1212. 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
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

Co-Authored-By: Konrad Oboza <konradoboza@users.noreply.github.com>
Copy link

sonarcloud bot commented May 24, 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 requested a review from konradoboza May 24, 2024 14:03
@alongosz alongosz merged commit 36ee5cd into 4.6 Jun 5, 2024
16 checks passed
@alongosz alongosz deleted the ibx-7911-drop-magic-getters branch June 5, 2024 15:54
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.

7 participants