Skip to content

[DO NOT MERGE][PREVIEW][CS] Bumped Ibexa Code Style to ^2.2.x-dev #666

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.

Sign up for GitHub

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

Draft
alongosz wants to merge 10 commits into
base: 5.0
Choose a base branch
from

[PHPStan] Removed resolved issues from the baseline

ecff830
Select commit
Loading
Failed to load commit list.
Draft

[DO NOT MERGE][PREVIEW][CS] Bumped Ibexa Code Style to ^2.2.x-dev #666

[PHPStan] Removed resolved issues from the baseline
ecff830
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 29, 2026 in 4m 19s

Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

Annotations

Check warning on line 75 in src/lib/FieldType/NullStorage.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$originalField". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3rd8470ZLMUbAfR&open=AZ3Yl3rd8470ZLMUbAfR&pullRequest=666

Check failure on line 909 in tests/lib/Repository/Service/Mock/PermissionTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2018470ZLMUbAel&open=AZ3Yl2018470ZLMUbAel&pullRequest=666

Check warning on line 31 in tests/lib/FieldType/EmailAddressTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$group". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl23V8470ZLMUbAeo&open=AZ3Yl23V8470ZLMUbAeo&pullRequest=666

Check warning on line 116 in src/lib/Limitation/SectionLimitationType.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This method has 5 returns, which is more than the 3 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3fr8470ZLMUbAe-&open=AZ3Yl3fr8470ZLMUbAe-&pullRequest=666

Check warning on line 28 in tests/bundle/Core/EventSubscriber/TrustedHeaderClientIpEventSubscriberTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure using this hardcoded IP address is safe here. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3HG8470ZLMUbAe3&open=AZ3Yl3HG8470ZLMUbAe3&pullRequest=666

Check failure on line 1640 in src/lib/Repository/ContentService.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3wi8470ZLMUbAfU&open=AZ3Yl3wi8470ZLMUbAfU&pullRequest=666

Check warning on line 496 in src/bundle/Core/DependencyInjection/IbexaCoreExtension.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$container". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4IE8470ZLMUbAf1&open=AZ3Yl4IE8470ZLMUbAf1&pullRequest=666

Check failure on line 81 in tests/lib/Limitation/NewObjectStateLimitationTypeTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3B28470ZLMUbAey&open=AZ3Yl3B28470ZLMUbAey&pullRequest=666

Check failure on line 27 in tests/lib/Base/Container/Compiler/Stubs/GatewayBasedStorageHandler.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an Exception or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3Cn8470ZLMUbAez&open=AZ3Yl3Cn8470ZLMUbAez&pullRequest=666

Check warning on line 993 in tests/integration/Core/Repository/SearchServiceTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "testFindContentInfoFiltered" on line 945. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2jA8470ZLMUbAeW&open=AZ3Yl2jA8470ZLMUbAeW&pullRequest=666

Check failure on line 246 in src/bundle/Core/Features/Context/UserContext.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4Ka8470ZLMUbAf8&open=AZ3Yl4Ka8470ZLMUbAf8&pullRequest=666

Check failure on line 82 in tests/lib/Limitation/ParentDepthLimitationTypeTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3Aq8470ZLMUbAeu&open=AZ3Yl3Aq8470ZLMUbAeu&pullRequest=666

Check warning on line 338 in src/bundle/Core/Features/Context/UserContext.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This method has 4 returns, which is more than the 3 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4Ka8470ZLMUbAf7&open=AZ3Yl4Ka8470ZLMUbAf7&pullRequest=666

Check failure on line 109 in src/lib/Repository/Permission/PermissionResolver.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 33 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3yU8470ZLMUbAfY&open=AZ3Yl3yU8470ZLMUbAfY&pullRequest=666

Check failure on line 135 in src/lib/Limitation/SubtreeLimitationType.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3i88470ZLMUbAfH&open=AZ3Yl3i88470ZLMUbAfH&pullRequest=666

Check warning on line 418 in src/bundle/Core/DependencyInjection/IbexaCoreExtension.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$container". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4IE8470ZLMUbAfy&open=AZ3Yl4IE8470ZLMUbAfy&pullRequest=666

Check warning on line 55 in src/lib/FieldType/Validator/ImageValidator.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This method has 4 returns, which is more than the 3 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3lN8470ZLMUbAfL&open=AZ3Yl3lN8470ZLMUbAfL&pullRequest=666

Check warning on line 34 in src/lib/QueryType/BuiltIn/RelatedToContentQueryType.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$options". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl32j8470ZLMUbAfg&open=AZ3Yl32j8470ZLMUbAfg&pullRequest=666

Check failure on line 121 in src/bundle/Core/DependencyInjection/Configuration/Parser/Image.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an Exception or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4FW8470ZLMUbAfp&open=AZ3Yl4FW8470ZLMUbAfp&pullRequest=666

Check warning on line 530 in src/bundle/Core/DependencyInjection/IbexaCoreExtension.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused function parameter "$container". 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl4IE8470ZLMUbAf3&open=AZ3Yl4IE8470ZLMUbAf3&pullRequest=666

Check failure on line 85 in tests/lib/Limitation/ContentTypeLimitationTypeTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2_b8470ZLMUbAes&open=AZ3Yl2_b8470ZLMUbAes&pullRequest=666

Check warning on line 88 in tests/lib/Repository/SiteAccessAware/SearchServiceTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "setLanguagesLookupExpectedArguments" on line 75. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2xY8470ZLMUbAeh&open=AZ3Yl2xY8470ZLMUbAeh&pullRequest=666

Check warning on line 117 in src/lib/Limitation/ContentTypeLimitationType.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This method has 4 returns, which is more than the 3 allowed. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl3gP8470ZLMUbAe_&open=AZ3Yl3gP8470ZLMUbAe_&pullRequest=666

Check warning on line 2835 in tests/integration/Core/Repository/SearchServiceTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "testFindContentInfoFiltered" on line 945. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2jA8470ZLMUbAeX&open=AZ3Yl2jA8470ZLMUbAeX&pullRequest=666

Check failure on line 222 in tests/integration/Core/Repository/FieldType/TimeIntegrationTest.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case. 

See more on https://sonarcloud.io/project/issues?id=ibexa_core&issues=AZ3Yl2Vb8470ZLMUbAeQ&open=AZ3Yl2Vb8470ZLMUbAeQ&pullRequest=666
View more details on SonarQubeCloud