Regression with subsites compatibility

[silverstripe]

Regression with subsites compatibility

[robbieaverill]

We've reverted the subsites change

[robbieaverill]

I've confirmed it as a regression, not just in the tests

[robbieaverill]

I've made a mistake in my thinking with that subsites pull request. In psuedo code, this is what I've changed it to do when you're asking which subsites can be accessed for a given permission code:

• Loop all subsites
• Set current subsite to each
• Ask Permission::checkMember whether the code is valid in the current state

I think the problem is that the permission check doesn't cross reference against the subsites a user's group can be used in, so they're all treated as global checks.

In practice the permission system seems to be working, but the API has clearly regressed since this report is broken.

I'll need to reintroduce some of the previous logic to run SQL queries against the member's groups in order to filter permissions available for each group the member is in, which is in turn filtered against the current subsite.

The Permission::checkMember() method won't work I don't think =(

[robbieaverill]

I've brain dumped at silverstripe/silverstripe-subsites#358 (comment)