Форум ReadyScript — Форма входа

Re: Форма входа

null@example.com (Андрей) — Mon, 11 Jul 2016 05:31:31 +0000

Закусило Александр пишет:

Ну всё дело в том я так думаю, про то что я говорил, что нужно проверять на флажок приходящий из поста о том, что нужно генерировать пароль.
Вы можете свой код закомментировать и попробовать туже схему. Я думаю всё будет как надо. Т.е. Вам не хватает проверок на то, что именно сейчас делается с пользователем. Попробуйте завардампить объекты в Вашем коде и приходящий POST или GET запрос, чтобы проанализировать.

Думаю я с этим уже не справлюсь. Давайте тогда по другому решим этот вопрос

Re: Форма входа

null@example.com (Закусило Александр) — Fri, 08 Jul 2016 13:53:41 +0000

Ну всё дело в том я так думаю, про то что я говорил, что нужно проверять на флажок приходящий из поста о том, что нужно генерировать пароль.
Вы можете свой код закомментировать и попробовать туже схему. Я думаю всё будет как надо. Т.е. Вам не хватает проверок на то, что именно сейчас делается с пользователем. Попробуйте завардампить объекты в Вашем коде и приходящий POST или GET запрос, чтобы проанализировать.

Re: Форма входа

null@example.com (Андрей) — Fri, 08 Jul 2016 13:07:03 +0000

Заметил следующий баг. Если выйти из профиля и попытатся зайти, пишет неверный e-mail или пароль. А если не вводить пароль то авторизовывает. И это только с теми аккаунтами, которые зарегистрированы данным методом(автоматическим паролем)

Re: Форма входа

null@example.com (Закусило Александр) — Fri, 08 Jul 2016 12:14:37 +0000

Он вызывает из этого же класса функцию checkOpenPassword, посмотрите что в ней.

Re: Форма входа

null@example.com (Андрей) — Fri, 08 Jul 2016 10:16:31 +0000

Да, так работает И сразу же исчесла проблема с переадрисацией на страницу регистрации. Теперь все как надо, сразу авторизовывает.
Только почему то регистрация работает, если только заккоментирована вот эта строчка из user.inc.php

Checker' => array(array(__CLASS__, 'checkOpenPassword'), '')

Не хватает какой то проверки?

Re: Форма входа

null@example.com (Закусило Александр) — Fri, 08 Jul 2016 10:01:41 +0000

Да точно! здесь же у нас идёт на прямую запись. Я думаю Вот так поможет.
изменить

$user['pass'] = $user::cryptPass(\RS\Helper\Tools::generatePassword(6));

на

$user['openpass'] = \RS\Helper\Tools::generatePassword(6);
$user['pass'] = $user::cryptPass();

Ато у нас получается [b]openpass[/b] поэтому и пустой, что нигде не присваивается.

Re: Форма входа

null@example.com (Андрей) — Fri, 08 Jul 2016 09:49:22 +0000

Не помогло, все равно не показывает пароль. Единственное что стало, так это два письма приходить
Можно конечно сделать так, но им тогда хеш будет зашифрованный пароль присылаться.
Какой еще есть вариант?

$user['openpass'] = $user['pass'] = $user::cryptPass(\RS\Helper\Tools::generatePassword(6));

Re: Форма входа

null@example.com (Закусило Александр) — Fri, 08 Jul 2016 08:55:35 +0000

Чтобы видеть ошибки, а не 500-ю включите отображение ошибок:
http://readyscript.ru/faq/#faq-errors

 $user = $data['orm'];
 $flag  = $data['flag']; //Флаг insert или update
 if ($flag == self::INSERT_FLAG && \Setup::$INSTALLED && !$this['no_send_notice']) {
            
 // Уведомление пользователю
 $user = new \Users\Model\Notice\UserRegisterUser;
 $user->init($this, $this['openpass']);
 \Alerts\Model\Manager::send($user);

Что касается уведомления, то Вы переприсвоили переменной $user, объект самого уведомления и передали объект самого себя, а не пользователя. Зачем?
Вам в Уведомдение нужно передать объект пользователя который вы получаете из объекта $data выше.

Нужно же так:)

 $user = $data['orm'];
 $flag  = $data['flag']; //Флаг insert или update
 if ($flag == $user::INSERT_FLAG && \Setup::$INSTALLED && !$user['no_send_notice']) {
            
 // Уведомление пользователю
 $user_notice = new \Users\Model\Notice\UserRegisterUser;
 $user_notice->init($user, $user['openpass']);
 \Alerts\Model\Manager::send($user_notice);

Re: Форма входа

null@example.com (Андрей) — Fri, 08 Jul 2016 06:22:55 +0000

Закусило Александр пишет:

Попробуйте в Вашем хуке beforeWrite выполнить сразу после установки openpass

/**
* Действия перед записью объекта пользователя
* 
* @param array $data - Массив  с данными
*/
public static function ormBeforeWriteUsersUser($data){
     /**
     * @var \User\Model\Orm\User
     */
     $user = $data['orm'];
     $flag  = $data['flag']; //Флаг insert или update
     if ($flag==$user::INSERT_FLAG){ //Делаем только на создание (insert)
          $my_autologin = \RS\Http\Request::commonInstance()->request('my_autologin', TYPE_INTEGER, false); //Получаем флаг из запроса
          if ($my_autologin){ //Если флаг стоит

              $user['pass'] = $user::cryptPass(\RS\Helper\Tools::generatePassword(6));
          }

     }
}

Я думаю, тогда 100% всё будет как надо.

Вот сейчас пароль записывается в БД, но на почту поле пароль приходит пустое.

Что же получается нужен хук после записи обьекта?Вот я его создал, но допустил ошибку и выдает ошибку 500 при регистрации

    /**
    * Действия после записи объекта
    * 
    * @param string $flag - insert или update
    */
public static function ormAfterWriteUsersUser($data){
    /**
     * @var \User\Model\Orm\User
     */
     $user = $data['orm'];
     $flag  = $data['flag']; //Флаг insert или update
             if ($flag == self::INSERT_FLAG && \Setup::$INSTALLED && !$this['no_send_notice']) {
            
            // Уведомление пользователю
            $user = new \Users\Model\Notice\UserRegisterUser;
            $user->init($this, $this['openpass']);
            \Alerts\Model\Manager::send($user); 
            
        }

}

Re: Форма входа

null@example.com (Закусило Александр) — Thu, 07 Jul 2016 15:29:15 +0000

Попробуйте в Вашем хуке beforeWrite выполнить сразу после установки openpass

/**
* Действия перед записью объекта пользователя
* 
* @param array $data - Массив  с данными
*/
public static function ormBeforeWriteUsersUser($data){
     /**
     * @var \User\Model\Orm\User
     */
     $user = $data['orm'];
     $flag  = $data['flag']; //Флаг insert или update
     if ($flag==$user::INSERT_FLAG){ //Делаем только на создание (insert)
          $my_autologin = \RS\Http\Request::commonInstance()->request('my_autologin', TYPE_INTEGER, false); //Получаем флаг из запроса
          if ($my_autologin){ //Если флаг стоит

              $user['pass'] = $user::cryptPass(\RS\Helper\Tools::generatePassword(6));
          }

     }
}

Я думаю, тогда 100% всё будет как надо.

Re: Форма входа

null@example.com (Андрей) — Thu, 07 Jul 2016 12:50:43 +0000

Вставляю следующий код в свою форму но пароль не записывается в бд. где искать ошибку

             
                    Пароль:
                    
                        
                        Получить автоматически на e-mail
                        Нужен для проверки статуса заказа, обращения в поддержку, входа в кабинет
                        
                            
                                
                                Пароль
                            
                            
                                
                                Повтор пароля

Re: Форма входа

null@example.com (Закусило Александр) — Thu, 07 Jul 2016 12:23:57 +0000

значит мне в форму регистрации нужно вставить данный код?
а что тогда использовать вместо?

Да, только проверка там не нужна на if ведь объекта $order просто в том шаблоне у Вас не существует. И сделайте name другой например просто my_autologin, чтобы не пересеклось ни где. А потом в хуке на него и проверите.

Re: Форма входа

null@example.com (Закусило Александр) — Thu, 07 Jul 2016 12:21:22 +0000

Потому что в объектах используются функции beforeWrite(перед записью в БД) и afterWrite(После записи в БД) в которых и происходят все дейтсвия. Если вы откроете класс \Users\Model\Orm\User (/modules/users/model/orm/user.inc.php) и посмотрите в метод
beforeWrite.
То там задаётся сам пароль вот так:

if (!empty($this['openpass'])) {
      $this['pass'] = self::cryptPass($this['openpass']);
}

Т.е. если задан openpass, что Вы и делаете в хуке, то pass будет сформирован из него.
Мы не храним в открытом виде пароли в БД.

Re: Форма входа

null@example.com (Андрей) — Thu, 07 Jul 2016 12:15:28 +0000

Да именно так, но если смотреть в var_dump то
["pass"]=>
NULL

в базу ничего не передается, почему?

значит мне в форму регистрации нужно вставить данный код?
а что тогда использовать вместо? name="reg_autologin" {if $order.reg_autologin}checked{/if} value="1" id="reg-autologin">

                
                    Пароль:
                    
                        
                        Получить автоматически на e-mail
                        Нужен для проверки статуса заказа, обращения в поддержку, входа в кабинет
                        
                            
                                {$order.__reg_openpass->formView(['form'])}
                                Пароль
                            
                            
                                {$order.__reg_pass2->formView()}
                                Повтор пароля
                            
                            
                                {$order->getErrorsByForm('reg_openpass', ', ')}

Re: Форма входа

null@example.com (Закусило Александр) — Thu, 07 Jul 2016 12:12:58 +0000

А для чего проверять на это галочку?

Потому, что запись в пользователя в БД может происходить и через админку. Вот представьте вы создаёте в админ панели пользователя указываете у него пароль. А он потом не подходит. Для этого и нужна проверка на галочку. Или программно создаёте пользователя, хотите присвоить ему пароль и никак не получается. Проверьте на параметр в посте.

А на каком же тогда этапе пароль должен записываться в поле pass?
Не понимаю что то я

На этом этапе Вы и передаёте пароль для записи. Просто генерируете его самостоятельно. Ведь Вы же так хотели.