issue in deleting role in Zizaco/entrust

Question

Iam trying to delete a role

$role = Role::findOrFail(1);
$role->delete();

I got the following error

FatalErrorException in Model.php line 945:
Class name must be a valid object or a string

in vendor/zizaco/entrust/src/commands/MigrationCommand.php on line 86

$usersTable  = Config::get('auth.providers.users.table');
$userModel   = Config::get('auth.providers.users.model');

Role model class

namespace App\Models;

use Zizaco\Entrust\EntrustRole;

class Role extends EntrustRole
{
       protected $fillable =  ['name', 'display_name', 'isActive','description', 'created_at', 'updated_at'];

}

ok, but it's better to edit the original posting/the question and add it there - so that other people can see it too - maybe they have an idea. — herrjeh42, Jun 20 '16 at 20:45
And: check out this issue if this is your problem eventually. https://github.com/Zizaco/entrust/issues/500 — herrjeh42, Jun 20 '16 at 20:46

score 4 · Accepted Answer · answered Jun 21 '16 at 18:09

4

I think this is the problem:

Find the file: vendor/zizaco/entrust/src/Entrust/Traits/EntrustRoleTrait.php

Replace

Line 51: ... Config::get('auth.model') ...

with

Line 51: ... Config::get('auth.providers.users.model') ...

answered Jun 21 '16 at 18:09

dimitrisd

566
1
9
30

core packages must not be changes. Your changes might replace on your next update. – Tarunn Aug 31 '16 at 07:34

score 0 · Answer 2 · answered Aug 31 '16 at 07:33

Its not good practice to update anything in core packages because in case you update your package, this change will be replace, instead override that function by adding this in your App\Role.php

/**
 * Many-to-Many relations with the user model.
 *
 * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
 */
public function users()
{
    return $this->belongsToMany(Config::get('auth.providers.users.model'), Config::get('entrust.role_user_table'),Config::get('entrust.role_foreign_key'),Config::get('entrust.user_foreign_key'));
   // return $this->belongsToMany(Config::get('auth.model'), Config::get('entrust.role_user_table'));
}

score 0 · Answer 3 · answered Mar 31 '18 at 06:58

I agree with @Tarunn. If you want code simplicity....

use App\User;

/**
* Many-to-Many relations with the user model.
*
* @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
*/

public function users()
{
   return $this->belongsToMany(User::class);
}

issue in deleting role in Zizaco/entrust

3 Answers3