Permission check for 'Not strict check' is wrong AFAIK about silverstripe-framework

[michalkleiner]

It's definitely the safer way how it currently is though I agree the code may not match the comment/the intended use.

[GuySartorelli]

Please avoid using images when referencing code - just the link to the code is sufficient, or use codeblocks if you really want to include the code in the issue itself.

[GuySartorelli]

Looks like in this commit (back in the SilverStripe 2 days) a refactor accidentally missed out the true from the return statement, and then later in this commit (very early Silverstripe 4) another refactor added false which probably seemed like the correct value to return without checking the PHPDoc.

Given it's been wrong since Silverstripe 2 I'm inclined to agree that the strict check should just be removed. It can be removed in 5.x without any BC changes since it ultimately returns false either way once you get to that part of the code. The parameter itself should be deprecated, to be removed in 6.

[sunnysideup]

will you make the changes?

[GuySartorelli]

There are over 2000 issues across the supported modules, and three total people on the CMS Squad. I can't commit to saying I or anyone on the CMS Squad will make changes for any given issue unless it has a high or critical impact.