1. PHP / Говнокод #2524

    +148.7

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    public function executeLogin(sfWebRequest $request)
  {
    if ($request->isMethod('post'))
    {
      if (SessionPeer::auth($request->getPostParameter('email'), $request->getPostParameter('password')))
      {
        return $this->redirect($this->getRequest()->getUri());
      }
      $this->getUser()->setFlash('error.login', true);
      return;
    }
  }

    Полезно иногда открывать недавно написанное и так бегло просматривать на предмет нелепостей. В данном случае улыбнул последний return;

    Запостил: miami, 02 Февраля 2010

    Комментарии (11) RSS

    • ava striker 02.02.2010 07:33 # 0

      а что за метод setFlash() ?
      Вспышка слева?
      Ответить

      • ava guest 02.02.2010 08:10 # +1

        Adobe Flash справа
        Ответить

      • ava miami 02.02.2010 10:02 # 0

        http://www.symfony-project.org/book/1_2/06-Inside-the-Controller-Layer#chapter_06_sub_flash_attributes да, вспышка
        Ответить

    • ava HyperGeek 02.02.2010 12:35 # +1

      О, симфони ) Я обычно для аутентификации делаю AuthForm, но и предложенный способ имеет право на существование. Это дело вкуса.

      Еще $this->getRequest() лишнее, ведь уже есть $request.
      Ответить

    • ava Dreyk 02.02.2010 16:27 # 0

      Та ну, а может там вначале было return sfView::ERROR ?
      А ваще поставить sfGuard и не париться =)
      Ответить

      • ava miami 02.02.2010 23:09 # −1

        и loginError.php будет полной копией loginSuccess.php? Только зачем это ) Не залогинился - попробуй снова, в той же форме, а ошибку отобразит Flash-сообщение.

        По уму конечно правильно сделать форму через механизм форм и привязать к Request со всякими валидаторами, как говорит HyperGeek. Эту тему я ещё не до конца осилил, и немного опасаюсь неосиленного.

        sfGuard хорош, но отпадает.Это фича для стандартного зашёл-вышел, увы, здесь не покатит.
        Ответить

    • ava guest 02.02.2010 16:35 # 0

      ниасилил - а в чем прикол то?
      Ответить

      • ava miami 02.02.2010 22:56 # 0

        В последнем return.
        Без него метод также возвращает null
        Ответить

    • ava guest8 11.07.2020 03:41 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 11.07.2020 03:43 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий