1. Christian Salazar
  2. cruge
  3. Issues

Issues

Issue #47 new

Poder indicar el password desde el formulario al utilizar el componente para crear el usuario

Carlos Belisario
created an issue

estoy usando el api ya que necesitava algunas cosas extra en el formulario de registro, al usar el createNewUser https://bitbucket.org/christiansalazarh/cruge/src/2623a2082a3b5c0bfa73205a9922f617cba66043/components/CrugeUserManager.php?at=master#cl-712 no pude enviar el password que estaba usando desde el formulario, ya que este metodo te genera el password desde el componete CrugeUtil según vi, en lo personal lo veo como una pequeña debilidad (no estoy seguro de la palabra) pero pienso que debería de ser opcional, algo como

#!php

if(!isset($mapped_values['password']) {
    $password = CrugeUtil::passwordGenerator();
} else {
    $password = $mapped_values['password'];
}

de manera que haya la opción de indicar la contraseña desde un form.

y claro agregar en el condicional

#!php

 if(($fieldname == 'username') || ($fieldname == 'email') || ($fieldname == 'password')) 

Comments (2)

  1. Carlos Belisario reporter

    o incluso para no complicar lo un ternario

    #!php
    
    $password = isset($mapped_values['password']) ? $mapped_values['password'] : CrugeUtil::passwordGenerator();
    

    No pude editar el texto anterior, saludos

  2. Carlos Belisario reporter

    Y viendo un poco más el método sería interesante que el envio de mail también sea opcional :) (como un parámetro de la función)

    #!php
    
    public function createNewUser($mapped_values, $sendPasswordByMail = true)
    ...
    if($sendPasswordByMail) {
         Yii::app()->crugemailer->sendPasswordTo($user,$password);
    }
    

    ya que hay apps que no necesitan eso (por ejemplo la que estoy haciendo), mientras más opcional tengas el componente, más simple de usar es ya que se puede hacer lo que necesita, que puede ser el componente entero con envío de mail y generador de password o no si no se desea, saludos no lo hice como un PR ya que lo vi como algo simple que puede hacer el admin, saludos creo que ya no tengo mas detalles que agregarle

  3. Log in to comment