Thanks to visit codestin.com
Credit goes to github.com

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

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