[Symfony][sfDoctrineGuard]Logowanie a nieaktywne konto., Jak dodać error o nieaktywnym koncie. Opcje

[cyklotrial]

Witam!

Przy użyciu pluginu sfDoctrineGuard utworzyłem sobie system użytkowników (rejestracja, logowanie itp).
Niby wszystko działa jak należy ale jedna rzecz cały czas nie daje mi spokoju... Dokładniej rzecz ujmując "nie podobają" mi się komunikaty o błędach podczas logowania.
Dodałem sobie do /lib/form/doctrine plik sfGuardFormSignin.class.php, tam ustawiłem sobie polskie komunikaty o błędach i wszystko byłoby OK gdyby nie brak komunikatu o tym że konto jest nieaktywne.

Podejrzałem sobie co zawiera metoda sfGuardValidatorUser znajdująca się w pliku sfGuardValidatorUser.class.php i ku mojemu zdziwieniu, sprawdzanie czy konto jest aktywne czy też nie, wykonywane jest wraz ze sprawdzaniem poprawności hasła. Skutkuje to tym, że pomimo iż dane podane przez użytkownika podczas logowania są poprawne a jego konto jest nieaktywne, zwracany jest ogólny błąd o niepoprawności loginu/hasła.

W jaki sposób należałoby poprawnie zwrócić błąd o nieaktywnym koncie?
Domyślam się że należałoby nadpisać metodę doClean z pliku sfGuardValidatorUser.class.php, oraz że ingerencja w pliki pluginu to niezbyt dobry pomysł... Gdzie więc należałoby umieścić plik ze zmodyfikowaną metodą doClean

[destroyerr]

Zazwyczaj staram się odtworzyć strukturę frameworka czy pluginu w swoim projekcie. W związku z czym umieściłbym to w lib/validator/GuardValidatorUser.class.php.

[cyklotrial]

destroyerr - Dziękuje - ta ścieżka faktycznie jest logiczna.

Ok no to jeszcze jedno pytanko mam do tego tematu
W jaki sposób poprawnie zwrócić taki error/wyjątek?

zrobiłem coś takiego:
nadpisałem metode configure:

[PHP] pobierz, plaintext 
  public function configure($options = array(), $messages = array())
  {
    parent::configure();
    $this->setMessage('inactive', 'To konto jest nieaktywne.');
  }
[PHP] pobierz, plaintext 

no i w odpowiednim miejscu dodałem sprawdzanie czy konto jest aktywne:

[PHP] pobierz, plaintext 
if($user->getIsActive())
{
   array_merge($values, array('user' => $user));
}
else
{
    throw new sfValidatorError($this, 'inactive');
}
[PHP] pobierz, plaintext 

Ale otrzymałem taki oto błąd:

Kod

sfGuardValidatorUser does not support the following error code: 'inactive'.

Co oznacza ten błąd - WIEM, ale niestety moja wiedza na temat SYMFONY jest niestety nadal zbyt mała, abym wiedział jak sprawić, by moj "kod błędu" został przyjęty.

Wszelkie pomysły/sugestie tu bardzo mile widziane.

[destroyerr]

http://www.symfony-project.org/api/1_4/sfV...thod_addmessage

[cyklotrial]

destroyerr raz jeszcze dziękuje za pomoc.
Nie wiem jakim cudem ta metoda umknęła mojej uwadze - dwa razy czytałem opis sfValidatorBase


Niestety, nadal coś nie działa tak jak powinno
Ciągle otrzymuje tylko komunikat o nieprawidłowym loginie lub haśle.
Kod metody doClean wygląda tak:

[PHP] pobierz, plaintext 
protected function doClean($values)
  {
    $username = isset($values[$this->getOption('username_field')]) ? $values[$this->getOption('username_field')] : '';
    $password = isset($values[$this->getOption('password_field')]) ? $values[$this->getOption('password_field')] : '';
 
    $allowEmail = sfConfig::get('app_sf_guard_plugin_allow_login_with_email', true);
    $method = $allowEmail ? 'retrieveByUsernameOrEmailAddress' : 'retrieveByUsername';
 
    // don't allow to sign in with an empty username
    if ($username)
    {
       if ($callable = sfConfig::get('app_sf_guard_plugin_retrieve_by_username_callable'))
       {
           $user = call_user_func_array($callable, array($username));
       } else {
           $user = $this->getTable()->retrieveByUsername($username);
       }
        // user exists?
       if($user)
       {
          // password is ok?
          if ($user->checkPassword($password))
          {
              //account is active?
              if($user->getIsActive())
              {
                  return array_merge($values, array('user' => $user));
              }
              else
              {
                  throw new sfValidatorError($this, 'inactive');
              }
          }
       }
    }
 
    if ($this->getOption('throw_global_error'))
    {
      throw new sfValidatorError($this, 'invalid');
    }
 
    throw new sfValidatorErrorSchema($this, array($this->getOption('username_field') => new sfValidatorError($this, 'invalid')));
  }
[PHP] pobierz, plaintext 

Domyślam się jedynie, że linia 42 wysyła ogólny error z kodem "invalid"... Jak sprawić żeby błąd o nieaktywnym koncie został wyświetlony?

[destroyerr]

Najprawdopodobniej wyjątek z Twoim błędem nie jest w ogóle rzucany. Po kodzie wszystko wygląda ok.

[cyklotrial]

Chyba sobie jakoś poradziłem z tym problemem. Zastanawiam się tylko czy zrobiłem to we właściwy sposób.

W /lib/form/doctrine utworzyłem plik: sfGuardFormSignin.class.php

[PHP] pobierz, plaintext 
class sfGuardFormSignin extends BasesfGuardFormSignin
{
  public function configure()
  {
      parent::configure();
      $this->validatorSchema->setPostValidator(new sfGuardValidatorUserCustom(
            array('throw_global_error' => true),
            array(
                'invalid' => "The email doesn't exist or the password is invalid",
                'inactive' => "This account is inactive")
            ));
  }
}
[PHP] pobierz, plaintext 

W /lib/validator utworzyłem plik: sfGuardValidatorUserCustom.class.php

[PHP] pobierz, plaintext 
class sfGuardValidatorUserCustom extends sfValidatorSchema
{
  public function __construct($options = array(), $messages = array())
  {
    $this->addOption('username_field', 'username');
    $this->addOption('password_field', 'password');
    $this->addOption('throw_global_error', false);
 
    $this->addMessage('inactive', 'This account is inactive.');
 
    parent::__construct(null, $options, $messages);
  }
 
 
 
  protected function doClean($values)
  {
    $username = isset($values[$this->getOption('username_field')]) ? $values[$this->getOption('username_field')] : '';
    $password = isset($values[$this->getOption('password_field')]) ? $values[$this->getOption('password_field')] : '';
 
 
    // don't allow to sign in with an empty username
    if ($username)
    {
 
        $query = Doctrine_Query::create()
        ->from('sfGuardUser')
        ->where('username = ?', $username);
 
        // Get the user
        $user = $query->fetchOne();
 
        // user exists?
        if($user)
        {
            // password is ok?
            if ($user->checkPassword($password))
            {
                //account is active?
                if($user->getIsActive())
                {
                    return array_merge($values, array('user' => $user));
                }
                else
                {
                    throw new sfValidatorError($this, 'inactive');
                    throw new sfValidatorErrorSchema($this, array($this->getOption('username_field') => new sfValidatorError($this, 'inactive')));
                }
            }
            else
            {
                throw new sfValidatorError($this, 'invalid');
                throw new sfValidatorErrorSchema($this, array($this->getOption('username_field') => new sfValidatorError($this, 'invalid')));
            }
        }
        else
        {
            throw new sfValidatorError($this, 'invalid');
            throw new sfValidatorErrorSchema($this, array($this->getOption('username_field') => new sfValidatorError($this, 'invalid')));
        }
    }
    return $values;
  }
}
[PHP] pobierz, plaintext 

Nie licząc faktu ze takie rozwiązanie działa - Czy jest ono poprawne i ma sens?