Skip to content
This repository has been archived by the owner on Mar 18, 2022. It is now read-only.

merge & fix syncRoles(), add syncPermissions() update documentation #140

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

webkonstantin
Copy link

No description provided.

@webkonstantin webkonstantin changed the title merge & fix syncRoles, update documentation merge & fix syncRoles(), add syncPermissions() update documentation Dec 17, 2015
@mackhankins
Copy link

Any chance of getting some of this merged any time soon?

@mackhankins
Copy link

@sw-double I'm not going through the pain of creating another pull request for this, but in the HasRoleAndPermission Trait you need to change SyncPermissions to

    /**
     * Sync permissions for a user.
     *
     * @param array|\Bican\Roles\Models\Permission[]|\Illuminate\Database\Eloquent\Collection $permissions
     * @return array
     */
    public function syncPermissions($permissions)
    {
        $this->userPermissions = null;

        return $this->userPermissions()->sync($permissions);
    }

@webkonstantin
Copy link
Author

@mackhankins thanks! I believe it is fixed now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants