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

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

Open
webkonstantin wants to merge 6 commits into
romanbican:masterfrom
webkonstantin:master
Open

merge & fix syncRoles(), add syncPermissions() update documentation#140
webkonstantin wants to merge 6 commits into
romanbican:masterfrom
webkonstantin:master

Conversation

@webkonstantin
Copy link
Copy Markdown

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
Copy Markdown

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

@mackhankins
Copy link
Copy Markdown

@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
Copy Markdown
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