Ověření, jestli uživatel již neexistuje, jak na to?

Zuben45

Dobrý den,
potřeboval bych poradit, ať to řeším jak chci, stále se mi ukazuje pouze stránka s chybou 500.

můj kód:

    protected function createComponentRegisterForm() {
        $form = new Form;

        $form->addText('username', 'Jméno:', 35)
        ->addRule(callback($this, 'isNickAvailable'), 'Zadané uživatelské jméno již existuje');
        $form->addText('email', 'E-mail: *', 35)
                ->setEmptyValue('@')
                ->addRule(Form::FILLED, 'Vyplňte Váš email')
                ->addCondition(Form::FILLED)
                ->addRule(Form::EMAIL, 'Neplatná emailová adresa');
        $form->addPassword('password', 'Heslo: *', 35)
                ->addRule(Form::FILLED, 'Vyplňte Vaše heslo');
        $form->addPassword('password2', 'Heslo znovu: *', 35)
                ->addConditionOn($form['password'], Form::VALID)
                ->addRule(Form::FILLED, 'Heslo znovu')
                ->addRule(Form::EQUAL, 'Hesla se neshodují.', $form['password']);
        $form->addCheckbox('mcregister', 'Zaregistrovat na MC')
             ->setDefaultValue(FALSE);
        $form->addCheckbox('agree', 'Souhlasím s podmínkami')
             ->addRule(Form::EQUAL, 'Je potřeba souhlasit s podmínkami', TRUE);
        $form->addSubmit('send', 'Registrovat');
        $form->onSuccess[] = callback($this, 'registerFormSubmitted');
        return $form;
    }

    public function registerFormSubmitted(UI\Form $form) {

      $values = $form->getValues();
      $exist = count($this->users->findShow()->where('username = ?',$this->username)->limit(1)->fetch());

      if($exist > 0)
      {
         $this->redirect('Homepage:default');
      }
      else
      {
      $new_user_id = $this->users->register($values);
      if($new_user_id){
        $this->redirect('Homepage:default');
        $mail = new Message;
        $mail->setFrom('Informace <info@gcore.cz')
          ->addTo($values->email)
          ->setSubject('Registrace')
          ->setHTMLBody('<h1>Registrace úspěšná</h1> <p>Dobrý den,<br>Vaše registrace na portále <a href="http://gcore.cz">GCore.cz</a> byla úspěšná.</p><p>Již se můžete přihlásit a využívat našich služeb.</p><p>S pozdravem<br>redakce GCore.cz</p>');
        $mailer = new SendmailMailer;
        $mailer->send($mail);
      }
     }

    }
    public function isNickAvailable( $username)
    {
      echo $username->getValue();

      $username = $this->users()->where(array(
     'username' => $username->getValue()))->limit(1)->fetch();

      if ($result !== FALSE && !$result->username)
      {
        return true;
      }
      else
      {
        return false;
      }
    }

Já už opravdu nevím :(, potřeboval bych aby mi to neodeslalo, když je uživatel již existuje.

David Matějka

stránka s chybou 500

lepsi popis chyby nemas, jo?

@dobby: pro zjisteni hodnoty controlu se pouziva getValue, takze to ma spravne..

Editoval matej21 (24. 12. 2013 22:49)

dobby

Ano, uvedomil som si to az teraz.

Chybu vidim este v tomto:

<?php
$username = $this->users()->where(array(
 'username' => $username->getValue()))->limit(1)->fetch();
?>

preco sa v if-e pytas na $result?

<?php
if ($result !== FALSE && !$result->username)
?>

Zuben45

díky za odpovědi, ale už jsem to vyřešil následujícím způsobem :)

   public function registerFormSubmitted(UI\Form $form) {

      $values = $form->getValues();
      $exists = $this->users->findShow();
      foreach($exists as $exist)
      {
        if($exist->username == $values->username)
        {
          $ok = FALSE;
        }
        else
        {
          $ok = TRUE;
          if($exist->email == $values->email)
          {
            $mailok = FALSE;
          }
          else
          {
            $mailok = TRUE;
          }
        }
      }

      //$mail_exist = count($this->users->findShow()->where('email = ?',$values->email)->limit(1)->fetch());
      if($ok == FALSE)
      {
          $form->addError('Uživatelské jméno se již používá.');
      }
      else
      {
        if($mailok == FALSE)
        {
          $form->addError('Email se již používá.');
        }
        else
        {
          $new_user_id = $this->users->register($values);
          if($new_user_id)
          {
            $this->redirect('Homepage:default');
            $mail = new Message;
            $mail->setFrom('Informace <info@gcore.cz')
              ->addTo($values->email)
              ->setSubject('Registrace')
              ->setHTMLBody('<h1>Registrace úspěšná</h1> <p>Dobrý den,<br>Vaše registrace na portále <a href="http://gcore.cz">GCore.cz</a> byla úspěšná.</p><p>Již se můžete přihlásit a využívat našich služeb.</p><p>S pozdravem<br>redakce GCore.cz</p>');
            $mailer = new SendmailMailer;
            $mailer->send($mail);
          }
        }
      }

    }

David Matějka

chtel jsem k tomu napsat neco vic, ale tohle bude stacit: ouuuuuuch!!

Zuben45

matej21 napsal(a):

chtel jsem k tomu napsat neco vic, ale tohle bude stacit: ouuuuuuch!!

co ještě by tam mělo být ? :)

David Matějka

spis co by tam nemelo byt – to prolezani vsech uzivatelu, abys zjistil, jestli tam nekdo neni s tim jmenem a emailem..

dej zpet tu kontrolu pres rule a rekni, co ti to pise za chybu

thunderbuff

O vlastním ověření uniátnosi doporučuji přečíst toto

mdjimy

Hmm moc hezká funkce todle to

$values = $form->getValues();
$exists = $this->users->findShow();
foreach($exists as $exist) {
  if($exist->username == $values->username) {
    $ok = FALSE;
  } else {
    $ok = TRUE;
    if($exist->email == $values->email) {
      $mailok = FALSE;
    } else {
      $mailok = TRUE;
    }
  }
}

Nejen ze uděláš práci za DB, ale ještě k tomu dojdeš ke špatnému výsledku…

Pokud budeš hledat FrantaNovak s emailem franta.novak@seznam.cz

A budeš iterovat přes tyto uživatele v tomto pořadí

FrantaNovak – franta.novak@seznam.cz
PepaZDepa – pepa.z.depa@centrum.cz
:
:

Tak budeš mít v proměnné $ok = TRUE, $mailok = TRUE …

+

Určitě by neuškodilo brát login „Honza“ a „honza“ za stejné… stejně pak u mailu…

Editoval mdjimy (23. 4. 2014 23:03)

greeny

Chyba bude možná i tady:

<?php
$exist = count($this->users->findShow()->where('username = ?',$this->username)->limit(1)->fetch());
?>

fetch() vrací NULL nebo ActiveRow, takže count() na fetch() je blbost.

Šaman

Co se ti nelíbí na kódu ze Sandboxu?