vespakoen / authority-laravel Goto Github PK

On Mac you can have problem with / when including files. This can be solved to use \ or DIRECTORY_SEPARATOR constant.

The problem is caused on bundles/authority/start.php

<?php

Laravel\Autoloader::map(array(
    'Authority' => __DIR__.'/authority'.EXT,
));

Authority::initialize(Auth::user());

Suggestion: Remove / and use DIRECTORY_SEPARATOR

The whole section "Configure Auth config" in the read me file is a bit out-of-date, I think. Those methods in config/auth.php are already there by default, but using the non-Eloquent methods.

Not sure if you're updating this anymore - but the bundle API has been down for some time, so the instructions for installation need to be updated, simply by changing the current details to:

git submodule add ... etc. (if it's an existing repository) or
git clone ... etc. to the bundles/authority directory for projects without version control.etc.

or git archive...etc. whatever.

Currently it's impossible to install with the current directions, which could be an issue for users still on L3.

Hi,

I don't know if this is normal but it doesn't sound logic from my POV...

created_at automatically update every time I edit my table but not updated_at. Shouldn't be the opposite? According to their names, I'm expecting the reverse behavior.

I used the code in migrations and both are similar so I can imagine this is a Laravel issue?

$table->timestamp('created_at');
$table->timestamp('updated_at');

Sorry to disturb you :)

Should line 18 be:

if( count($user->roles) == 0) return false;

instead of

if( ! count($user->roles) == 0) return false;

Is it possible to use Authority in a filter? That way, I could filter an entire controller (and/or just one method) to restrict access somehow. Something like (semi-pseudo code):

<?php

class Admin_Controller extends Controller {

    public function __construct() {

        // require csrf on posts
        $this->filter('before', 'csrf')->on('post');

        // on admins can view this stuff
        $this->filter('authority', 'has_role:admin');

        parent::__construct();

    }
}

The install instructions need updating for the latest version of the develop branch. The 'bundle' setting no longer exists in application/config/application.php, it was removed in this commit.

I really enjoyed using this with Codeigniter, so it'll be great to see how it works with Laravel!

:)

Please update:
http://bundles.laravel.com/bundle/authority

Or redirect to this page:
http://vespakoen.github.com/authority-laravel/

Is possible in future update, you add groups?

just a quick heads up that if you follow the exact steps of the installation procedure the migration fails because the bundle hasn't been registered yet. You have to add authority to your bundles file first before you can run

php artisan migrate authority