Scrutinizer dependency check fails on cake3 crud about crud HOT 12 CLOSED

Where is cake\event\event used without property capitalization? Looks like its related to something like that.

from crud.

I will check my plugin for any lower casing in my plugin tomorrow. Would be akward to say the least 😏

from crud.

Just checked and there are only two references to Cake\Event\Event in my (almost empty) plugin and they are using the correct casing, just like found in the CRUD plugin:

use Cake\Event\Event;

    /**
     * beforeFilter
     *
     * @param Cake\Event\Event $event Event instance
     * @return void
     */
    public function beforeFilter(Event $event)
    {
        pr("AppPlusPlus component beforeFilter()");
    }

Could it be related to the CRUD plugin and not my code? No biggie if this isn't solved, I can just disable Scrutinizer for the time being.

from crud.

Spotted one difference though, CRUD uses a leading , testing right now.

* @param \Cake\Event\Event $event Event instance

from crud.

feel free to PR a fix for it :)

from crud.

Sorry guys, I have triple-checked my sources and even disabled everything remotely related to this but the outcome is still the same and because the Scrutinizer information is so limited I honestly have no idea what to investigate next.

from crud.

I have stripped my app to the bare minimum but the issue persists. Since this plugin is pretty rock-solid the issue could very well be caused by some configuration f*** up on my behalf so... feel free to close if you guys think this is one of those cases.

from crud.

For the record, I am getting the exact same issue.

from crud.

Asked Scrutinizer support, and they said:

it seems like in that dependency, there is somewhere a reference to a class Évent (note the apostrophe on the first E) which is causing problems with the analyzer. We will release a fix for this shortly, but it's probably also a bug in that dependency which you might want to fix/report.

Although, can't actually find any such reference.

from crud.

@dakota thanks for looking into this, very interesting to say the least

from crud.

@bravo-kernel Well, it's affecting me as well :)

from crud.

Closing as PR has been opened.

from crud.