Using Middleware; redirect instead of throw exception about laratrust HOT 9 CLOSED

Actually i think it's a good idea.

from laratrust.

@Naoray , Thanks but i already solved it with 934f260

from laratrust.

I tried it with:

// config
'middleware_handling' => 'abort',
'middleware_params' => '403',


// middleware
if ($this->auth->guest() || !$request->user()->ability($roles, $permissions, [ 'validate_all' => $validateAll ]))
{
    return call_user_func(Config::get('laratrust.middleware_handling'), Config::get('laratrust.middleware_params'));
}

... it works, but there are some errors in the tests:

1) LaratrustAbilityTest::testHandle_IsGuestWithNoAbility_ShouldAbort403
Mockery\Exception\NoMatchingExpectationException: No matching handler found for Mockery_9__config::get('laratrust.middleware_handling'). Either the method was unexpected or its arguments matched no expected argument list for this method

But I did not figure out how to resolve it yet. Any idea?

from laratrust.

@Naoray Indeed i'm working on it. I think the problem is the test.

from laratrust.

Could you please try it out, i got it working in 556515a. remember to add the middleware_handling and middleware_params to your config/laratrus.php file

from laratrust.

you've had a typo in the middlewares... fixed it in #24 . It works perfectly, but the tests are still failing!! :/

from laratrust.

@Naoray yeah i had the typo, maybe i did the typo in the tests too, i'm going to check it out

from laratrust.

@Naoray Yeah the problem was that the typo was on the tests too. i'm solving it now.

Thank you! 👍

from laratrust.

@santigarcor got them! fixed in #25

from laratrust.