Events are not being named correctly.

[mikebronner]

Recently tracked events started being named like {u'1': {u'Status': u'Login Attempt Failed'}, u'0': u'Session', u'time': 1497541309}. /cc @richardbenson

[mikebronner]

Fixed by 254b56c

[mikebronner]

@richardbenson give it a shot now.

[richardbenson]

@mikebronner

FatalThrowableError in Guard.php line 273:
Type error: Argument 1 passed to Illuminate\Auth\Guard::validate() must be of the type array, null given, called in /home/vagrant/uvol-api/source/vendor/genealabs/laravel-mixpanel/src/Listeners/LaravelMixpanelEventHandler.php on line 28

I'm fiddling around in there to try and get the right order of things, will update if I crack it

Edit:

seems inelegant to me but solves it:

    public function onUserLoginAttempt($event)
    {
        $email = $event->credentials['email'] ?? '';
        $password = $event->credentials['password'] ?? '';

        if (starts_with(app()->version(), '5.1.')) {
            $email = $event['email'] ?? '';
            $password = $event['password'] ?? '';
        }

        $authModel = config('auth.providers.users.model') ?? config('auth.model');
        $user = app($authModel)
            ->where('email', $email)
            ->first();
        $eventName = 'Login Attempt Succeeded';

        if (starts_with(app()->version(), '5.1.')) {
            $validated = auth()->getProvider()->validateCredentials(
                $user,
                ['email' => $email, 'password' => $password]
            );
        } else {
            $validated = auth()->validate($event->credentials);
        }

        if ($user && ! $validated) {
            $eventName = 'Login Attempt Failed';
        }

        event(new MixpanelEvent($user, $eventName));
    }

[mikebronner]

Figures! :) Ha, made the fix in the 5.4 test platform, and not in 5.1. Testing there now, should have a fix shortly. Sorry about that.

[mikebronner]

@richardbenson Pushed release 0.6.11 which should alleviate the issue. Tested in both 5.1 and 5.4. Please note the release notes for some minor tracking changes made.