removed encoder, set hasher

This commit is contained in:
tracer 2021-06-13 15:16:03 +02:00
parent f7109bbd02
commit a5e665cbe4
1 changed files with 17 additions and 22 deletions

View File

@ -29,13 +29,13 @@ class UserCrudController extends AbstractCrudController
} }
/** /**
* @var UserPasswordEncoderInterface * @var UserPasswordHasherInterface
*/ */
private $passwordHasher; private UserPasswordHasherInterface $passwordHasher;
/** /**
* @var Security * @var Security
*/ */
private $security; private Security $security;
/** /**
* UserCrudController constructor. * UserCrudController constructor.
@ -61,7 +61,6 @@ class UserCrudController extends AbstractCrudController
} }
/** /**
* @param string $pageName * @param string $pageName
* *
@ -69,7 +68,6 @@ class UserCrudController extends AbstractCrudController
*/ */
public function configureFields(string $pageName): iterable public function configureFields(string $pageName): iterable
{ {
//$id = TextField::new('id')->
$username = TextField::new('username'); $username = TextField::new('username');
$email = TextField::new('email'); $email = TextField::new('email');
$firstName = TextField::new('firstName'); $firstName = TextField::new('firstName');
@ -80,7 +78,8 @@ class UserCrudController extends AbstractCrudController
$roles = ChoiceField::new('roles', 'Roles') $roles = ChoiceField::new('roles', 'Roles')
->allowMultipleChoices() ->allowMultipleChoices()
->autocomplete() ->autocomplete()
->setChoices([ 'User' => 'ROLE_USER', ->setChoices([
'User' => 'ROLE_USER',
'Admin' => 'ROLE_ADMIN', 'Admin' => 'ROLE_ADMIN',
'SuperAdmin' => 'ROLE_SUPER_ADMIN'] 'SuperAdmin' => 'ROLE_SUPER_ADMIN']
); );
@ -92,22 +91,18 @@ class UserCrudController extends AbstractCrudController
switch ($pageName) { switch ($pageName) {
case Crud::PAGE_INDEX: case Crud::PAGE_INDEX:
return [ $username, $firstName, $lastName, $email, $lastLoginAt ]; return [$username, $firstName, $lastName, $email, $lastLoginAt];
break;
case Crud::PAGE_DETAIL: case Crud::PAGE_DETAIL:
return [ $username, $firstName, $lastName, $email, $lastLoginAt, $createdAt, $roles, $isVerified ]; return [$username, $firstName, $lastName, $email, $lastLoginAt, $createdAt, $roles, $isVerified];
break;
case Crud::PAGE_NEW: case Crud::PAGE_NEW:
return [ return [
$password, $password,
]; ];
break;
case Crud::PAGE_EDIT: case Crud::PAGE_EDIT:
return [ $username, $password, $firstName, $lastName, $email, $lastLoginAt, $roles, $isVerified ]; return [$username, $password, $firstName, $lastName, $email, $lastLoginAt, $roles, $isVerified];
break;
} }
return [];
} }
/** /**
@ -117,7 +112,7 @@ class UserCrudController extends AbstractCrudController
*/ */
public function updateEntity(EntityManagerInterface $entityManager, $entityInstance): void public function updateEntity(EntityManagerInterface $entityManager, $entityInstance): void
{ {
// set new password with encoder interface // set new password with hash interface
if (method_exists($entityInstance, 'setPassword')) { if (method_exists($entityInstance, 'setPassword')) {
$plainPassword = trim($this->get('request_stack')->getCurrentRequest()->request->all('User')['password']); $plainPassword = trim($this->get('request_stack')->getCurrentRequest()->request->all('User')['password']);
@ -126,8 +121,8 @@ class UserCrudController extends AbstractCrudController
$entityInstance->setPassword($this->password); $entityInstance->setPassword($this->password);
} else { } else {
$user = new User(); $user = new User();
$encodedPassword = $this->passwordHasher->hashPassword($user, $plainPassword); $hashedPassword = $this->passwordHasher->hashPassword($user, $plainPassword);
$entityInstance->setPassword($encodedPassword); $entityInstance->setPassword($hashedPassword);
} }
} }