Silverstripe-framework: Suspect comparison in handleRequest

[ntd]

Sometime I dig into the code to see how it works and I meet some "interesting" stuff: please, tell me if reporting here this kind of issues is not appropriate or if there is a less invasive way.

Quoting this code:

in_array(strtolower($action), ['run', 'doInit']))

There is something wrong: or doInit must not exist or it should be doinit.

CC @dhensby as this seems to be introduced by PR #3926.

[dhensby]

What a good spot! looks like that needs patching

[ScopeyNZ]

Sometime I dig into the code to see how it works and I meet some "interesting" stuff: please, tell me if reporting here this kind of issues is not appropriate or if there is a less invasive way.

Please do! Always happy to receive bug reports or discuss "interesting stuff"! 🙏

[kinglozzer]

Fixed in https://github.com/silverstripe/silverstripe-framework/pull/8843

[dhensby]

I've merged up, too