Změna hesla v administraci (založeno na ACL)

@Šaman: Díky za tvůj čas a popis.

Zatím jsem to zbastil takto:

AdminPresenter.php

<?php

namespace AdminModule;

use Nette\Application\UI\Form;
use BootstrapFormRenderer\BootstrapRenderer;

class AdminPresenter extends BasePresenter {

    /** @var AdminModule\UserRepository */
    private $userRepository;

    public function inject(UserRepository $userRepository)
	{
		$this->userRepository = $userRepository;
	}

    protected function createComponentChangePassForm() {
	$form = new Form;
	$form->setRenderer(new BootstrapRenderer);
	$form->addPassword('oldPassword', '')
		->addRule(Form::FILLED, 'Enter your password')
		->setAttribute('class', 'input-block-level')
		->setAttribute('placeholder', 'actual password');
	$form->addPassword('newPassword', '')
		->addRule(Form::FILLED, 'Enter new password')
		->setAttribute('class', 'input-block-level')
		->setAttribute('placeholder', 'new password');
	$form->addPassword('newPassword2', '')
		->addRule(Form::FILLED, 'Enter new password')
		->addRule(Form::EQUAL, "Please fill new password two times correct.", $form["newPassword"])
		->setAttribute('class', 'input-block-level')
		->setAttribute('placeholder', 'new password again');

	$form->addSubmit('send', 'Change')->setAttribute('class', 'btn-large btn-block btn-primary');

	$form->onSuccess[] = $this->processChangePassForm;
	return $form;
    }

    public function processChangePassForm($form) {
		$values = $form->getValues();
		$userId = $this->getUser()->id;
		$oldPassword = $values["oldPassword"];
		$newPassword = $values["newPassword"];

		try {

			if ($this->userRepository->setPassword($userId,$oldPassword,$newPassword)) $this->flashMessage('Great! Password was changed.', 'success');
			else $this->flashMessage('Please fill your correct actual password.', 'warning');


		} catch (Nette\Security\AuthenticationException $e) {
			$form->addError('Zadané heslo není správné.');
		}

    }

}

UserREpository.php

<?php

namespace AdminModule;

use Nette;

class UserRepository extends Repository
{


	/**
	 * @return Nette\Database\Table\ActiveRow
	 */
	public function findByName($username)
	{
		return $this->findBy(array('username' => $username))->fetch();
	}

	/**
	 * @return Boolean
	 */
	public function setPassword($userId,$oldPassword,$newPassword)
	{
	    $actualPassword = $this->findBy(array('id' => $userId))->fetch()->password;
	    $password = sha1($oldPassword);
	    if ($actualPassword == $password){
		$this->updatePassword($userId, $newPassword);
		return TRUE;
	    }
	    else return FALSE;
	}

}

Repository.php

<?php

namespace AdminModule;

use Nette;

/**
 * Provádí operace nad databázovou tabulkou.
 */
abstract class Repository extends Nette\Object
{
	/** @var Nette\Database\Connection */
	protected $connection;



	public function __construct(Nette\Database\Connection $db)
	{
		$this->connection = $db;
	}



	/**
	 * Vrací objekt reprezentující databázovou tabulku.
	 * @return Nette\Database\Table\Selection
	 */
	protected function getTable()
	{
		return $this->connection->table("user");
	}



	/**
	 * Vrací všechny řádky z tabulky.
	 * @return Nette\Database\Table\Selection
	 */
	public function findAll()
	{
		return $this->getTable();
	}



	/**
	 * Vrací řádky podle filtru, např. array('name' => 'John').
	 * @return Nette\Database\Table\Selection
	 */
	public function findBy(array $by)
	{
		return $this->getTable()->where($by);
	}
	public function updatePassword($userId, $newPassword)
	{
		$this->getTable()->where(array('id' => $userId))->update(array(
			'password' => sha1($newPassword),
		));
	}
}

Asi je to trochu prasárna, ale jede to. Teď se to budu snažit postupně vylepšovat. Pokud k tomuto máte kdokoliv jakékoliv komenty, budu za ně rád.

K tomu, co psal Matěj bych přidal ještě

4. Všechny repository jsou součástí modelu a tudíž by neměly být v namespace AdminModulu. Model zkrátka stojí mimo strukturu modulů a presenterů.

5. Ten try-catch blok je teď úplně zbytečný, protože při špatném heslu se žádná výjimka nevyhodí. Ono s těma výjimkama je to trochu sporné téma – ideálně bys je měl používat jen kde hrozí, že se algoritmus zhroutí a aplikace by se dostala do nedefinovaného stavu (např. dělení nulou, čtení z neexistující tabulky). V případě toho autentikátoru reálně hrozí, že se někdo pokusí přihlásit se špatným heslem. To není nic výjimečného a osobně bych raději tento případ ošetřil jinak, než výjimkou. Výjimka (výjimečný, neočekávaný případ) by nastala třeba pokud by se nedalo připojit k db, nebo pokud by neexistoval sloupec username a password.

Editoval Šaman (23. 9. 2013 21:05)