Security\DefaultType: fixed admin form

Venne · Oct 25, 2014 · cb3ab40 · cb3ab40
1 parent f412ccb
commit cb3ab40
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 16 deletions.
diff --git a/src/Security/DefaultType/AdminFormFactory.php b/src/Security/DefaultType/AdminFormFactory.php
@@ -45,7 +45,7 @@ public function create()
 		$user->addText('name', 'Name');
 		$user->addTextArea('notation', 'Notation', 40, 4)
 			->getControlPrototype()->attrs['class'] = 'input-block-level';
-		$user->addMultiSelect('entityRoles', 'Roles')
+		$user->addMultiSelect('roleEntities', 'Roles')
 			->setOption(IComponentMapper::ITEMS_TITLE, 'name');
 		$user->addText('key', 'Lock key')
 			->setOption('description', 'If is set user cannot log in.');

diff --git a/src/Security/DefaultType/AdminFormService.php b/src/Security/DefaultType/AdminFormService.php
@@ -60,7 +60,7 @@ protected function save(Form $form, $entity)
 	protected function error(Form $form, \Exception $e)
 	{
 		if ($e instanceof \Kdyby\Doctrine\DuplicateEntryException) {
-			$form['name']->addError($form->getTranslator()->translate('Name must be unique.'));
+			$form['user']['name']->addError($form->getTranslator()->translate('Name must be unique.'));
 
 			return;
 		}

diff --git a/src/Security/User.php b/src/Security/User.php
@@ -289,17 +289,26 @@ public function __toString()
 	/**
 	 * @return ArrayCollection|Role[]
 	 */
-	public function getEntityRoles()
+	public function getRoleEntities()
 	{
 		return $this->roleEntities->toArray();
 	}
 
-	public function addEntityRole(Role $role)
+	public function addRoleEntity(Role $role)
 	{
 		$this->roleEntities[] = $role;
 	}
 
-	public function removeRole(Role $role)
+	/**
+	 * @param \Venne\Security\Role $role
+	 * @return boolean
+	 */
+	public function hasRoleEntity(Role $role)
+	{
+		return $this->roleEntities->contains($role);
+	}
+
+	public function removeRoleEntity(Role $role)
 	{
 		$this->roleEntities->removeElement($role);
 	}
@@ -338,17 +347,6 @@ public function getNotation()
 		return $this->notation;
 	}
 
-	/**
-	 * @param \Venne\Security\Role $roleEntity
-	 * @return $this
-	 */
-	public function addRoleEntity(Role $roleEntity)
-	{
-		$this->roleEntities->add($roleEntity);
-
-		return $this;
-	}
-
 	/**
 	 * Returns a list of roles that the user is a member of.
 	 *