EZP-31837: Creating an additional Content Location does not respect Content visibility

[pspanja]

Question	Answer
JIRA issue	EZP-31837
Type	bug
Target eZ Platform version	v3.1.3
BC breaks	no
Tests pass	yes
Doc needed	no

Checklist:

• PR description is updated.
• Tests are implemented.
• Added code follows Coding Standards (use $ composer fix-cs).
• PR is ready for a review.

[ViniTou]

Should it not target 1.1 at least?
@adamwojs

[adamwojs]

Should it not target 1.1 at least?

You are right. AFAIR content hide/reveal was part of the eZ Platform 2.5 so probably bug is there as well.

[pspanja]

OK, do let me know where exactly you want this.

[lserwatka]

Could you open this against ezpublish-kernel/7.5 branch. We will run QA on 2.5.x and 3.1.x.

[pspanja]

OK, will do. I also added a fix for incorrect docs of Location::$invisible property in 4acc1d4, as tested here:

ezplatform-kernel/eZ/Publish/API/Repository/Tests/ContentServiceTest.php

Line 6324 in 2b64f12

$this->assertTrue($locations[0]->invisible);

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
2 Code Smells

No Coverage information
0.0% Duplication

[lserwatka]

OK, will do. I also added a fix for incorrect docs of Location::$invisible property in 4acc1d4, as tested here:

ezplatform-kernel/eZ/Publish/API/Repository/Tests/ContentServiceTest.php

Line 6324 in 2b64f12

$this->assertTrue($locations[0]->invisible);

Looks good. Thank you

[pspanja]

Opened ezsystems/ezpublish-kernel#3066

[adamwojs]

Closed in favor of ezsystems/ezpublish-kernel#3066 (changes will be merged up)