[SPIP Zone] r27637 - in /_plugins_/_test_/inscription2/inscription2_2_0/formulaires: inscription2.php inscription2_modif_infos_connection.html

Archives Zone
1

Hello,

Pourquoi dupliques tu le cod ? Ce qu'il y avait juste au dessus de ton ajout ca n'était pas déjà le changement de mot de passe ?

Il faudrait eviter de faire des redondances si c'est possible.

BoOz

pierre.fiches-GANU6spQydw@public.gmane.org a écrit :

Author: pierre.fiches@free.fr
Date: Fri Apr 3 22:40:54 2009
New Revision: 27637

Log:
Permettre de changer le password.

(merci de vérifier qu'il n'y ai pas de degats collateraux)

Modified:
    _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php
    _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html

Modified: _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php

--- _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php (original)
+++ _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php Fri Apr 3 22:40:54 2009
@@ -142,6 +142,23 @@
           $erreurs['pass'] = _T('inscription2:password_obligatoire');
         }
       }
+ }else if($p = _request('password')) {
+ if(strlen($p)){
+ if (strlen($p) < 6) {
+ $erreurs['pass'] = _T('info_passe_trop_court');
+ $erreurs['message_erreur'] .= _T('info_passe_trop_court')."<br />";
+ } elseif ($p != _request('password1')) {
+ $erreurs['pass'] = _T('info_passes_identiques');
+ $erreurs['message_erreur'] .= _T('info_passes_identiques')."<br />";
+ }
+ }else{
+ if(!is_numeric($id_auteur)){
+ // Si on est dans la modif d'id_auteur on garde l'ancien pass si rien n'est rentré
+ // donc on accepte la valeur vide
+ // dans le cas de la création d'un auteur ... le password sera nécessaire
+ $erreurs['pass'] = _T('inscription2:password_obligatoire');
+ }
+ }
     }
   }
   
@@ -206,10 +223,10 @@
   global $tables_principales;
   
   if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') != 'on'))
- OR (is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) && (strlen(_request('pass')) == 0)){
+ OR (is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) && (strlen(_request('password')) == 0)){
     $mode = 'modification_auteur_simple';
   }
- else if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on'))){
+ else if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) and (strlen(_request('password')) != 0)){
     $mode = 'modification_auteur_pass';
   }
   else if((lire_config('inscription2/pass') == 'on') && (strlen(_request('pass')))){
@@ -270,7 +287,8 @@
   
   //Vérification du password
   if(($mode == 'inscription_pass') || ($mode == 'modification_auteur_pass')){
- $new_pass = _request('pass');
+ if (strlen(_request('password')) != 0){$new_pass = _request('password');}
+ else{$new_pass = _request('pass');}
     if (strlen($new_pass)) {
       include_spip('inc/acces');
       $htpass = generer_htpass($new_pass);
@@ -341,7 +359,7 @@
          if (!$new){
         $message = _T('inscription2:profil_modifie_ok');
- if($mode = 'modification_auteur_simple'){
+ if($mode == 'modification_auteur_simple'){
           $message .= '<br />'._T('inscription2:mot_passe_reste_identique');
         }
         $editable = true;

Modified: _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html

--- _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html (original)
+++ _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html Fri Apr 3 22:40:54 2009
@@ -27,12 +27,12 @@
       </li>]
     
     [(#CONFIG{inscription2/pass_fiche_mod}|=={on}|oui)
- <li class="editer_password[ (#CONFIG{inscription2/password_obligatoire}|=={on}|?{obligatoire})][ (#ENV**{erreurs}|table_valeur{password}|oui)erreur]">
+ <li class="editer_password[ (#CONFIG{inscription2/pass_obligatoire}|=={on}|?{obligatoire})][ (#ENV**{erreurs}|table_valeur{pass}|oui)erreur]">
         <label for="password"><:inscription2:pass:></label>
         <input type="password" class="text" autocomplete="off" name="password" id="password" value="#ENV{password}" />
       <label for="password1"><:inscription2:password_retaper:></label>
         <input type="password" class="text" autocomplete="off" name="password1" id="password1" value="#ENV{password1}" />
- [<p class='erreur'>(#ENV**{erreurs}|table_valeur{password})</p>]
+ [<p class='erreur'>(#ENV**{erreurs}|table_valeur{pass})</p>]
     </li>]
   </ul>
</fieldset>
\ No newline at end of file

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

2

Je l'ai cru mais non c'est uniquement valable pour la création du pass à l'inscription _request('pass').
Pour sa modif, les contraintes ne sont pas les même : un pass existe déjà et si on offre la possibilité de le changer il n'est pas certain qu'on l'utilise...de plus on ne l'affiche pas dans le html (d'où le _request('password')).
J'ai passé pas mal de temps la dessus mais c'est à peu près certain que ce code peut-être optimisé...À vrai dire je suis encore un peu limité en php :frowning:

pierre

Le 5 avr. 09 à 22:53, BoOz a écrit :

Hello,

Pourquoi dupliques tu le cod ? Ce qu'il y avait juste au dessus de ton ajout ca n'était pas déjà le changement de mot de passe ?

Il faudrait eviter de faire des redondances si c'est possible.

BoOz

pierre.fiches@free.fr a écrit :

Author: pierre.fiches@free.fr
Date: Fri Apr 3 22:40:54 2009
New Revision: 27637
Log:
Permettre de changer le password.
(merci de vérifier qu'il n'y ai pas de degats collateraux)
Modified:
   _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php
   _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html
Modified: _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php

--- _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php (original)
+++ _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2.php Fri Apr 3 22:40:54 2009
@@ -142,6 +142,23 @@
          $erreurs['pass'] = _T('inscription2:password_obligatoire');
        }
      }
+ }else if($p = _request('password')) {
+ if(strlen($p)){
+ if (strlen($p) < 6) {
+ $erreurs['pass'] = _T('info_passe_trop_court');
+ $erreurs['message_erreur'] .= _T('info_passe_trop_court')."<br />";
+ } elseif ($p != _request('password1')) {
+ $erreurs['pass'] = _T('info_passes_identiques');
+ $erreurs['message_erreur'] .= _T('info_passes_identiques')."<br />";
+ }
+ }else{
+ if(!is_numeric($id_auteur)){
+ // Si on est dans la modif d'id_auteur on garde l'ancien pass si rien n'est rentré
+ // donc on accepte la valeur vide
+ // dans le cas de la création d'un auteur ... le password sera nécessaire
+ $erreurs['pass'] = _T('inscription2:password_obligatoire');
+ }
+ }
    }
  }
  
@@ -206,10 +223,10 @@
  global $tables_principales;
  
  if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') != 'on'))
- OR (is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) && (strlen(_request('pass')) == 0)){
+ OR (is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) && (strlen(_request('password')) == 0)){
    $mode = 'modification_auteur_simple';
  }
- else if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on'))){
+ else if((is_numeric($id_auteur) && (lire_config('inscription2/pass_fiche_mod') == 'on')) and (strlen(_request('password')) != 0)){
    $mode = 'modification_auteur_pass';
  }
  else if((lire_config('inscription2/pass') == 'on') && (strlen(_request('pass')))){
@@ -270,7 +287,8 @@
  
  //Vérification du password
  if(($mode == 'inscription_pass') || ($mode == 'modification_auteur_pass')){
- $new_pass = _request('pass');
+ if (strlen(_request('password')) != 0){$new_pass = _request('password');}
+ else{$new_pass = _request('pass');}
    if (strlen($new_pass)) {
      include_spip('inc/acces');
      $htpass = generer_htpass($new_pass);
@@ -341,7 +359,7 @@
         if (!$new){
        $message = _T('inscription2:profil_modifie_ok');
- if($mode = 'modification_auteur_simple'){
+ if($mode == 'modification_auteur_simple'){
          $message .= '<br />'._T('inscription2:mot_passe_reste_identique');
        }
        $editable = true;
Modified: _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html

--- _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html (original)
+++ _plugins_/_test_/inscription2/inscription2_2_0/formulaires/inscription2_modif_infos_connection.html Fri Apr 3 22:40:54 2009
@@ -27,12 +27,12 @@
      </li>]
    
    [(#CONFIG{inscription2/pass_fiche_mod}|=={on}|oui)
- <li class="editer_password[ (#CONFIG{inscription2/password_obligatoire}|=={on}|?{obligatoire})][ (#ENV**{erreurs}|table_valeur{password}|oui)erreur]">
+ <li class="editer_password[ (#CONFIG{inscription2/pass_obligatoire}|=={on}|?{obligatoire})][ (#ENV**{erreurs}|table_valeur{pass}|oui)erreur]">
        <label for="password"><:inscription2:pass:></label>
        <input type="password" class="text" autocomplete="off" name="password" id="password" value="#ENV{password}" />
      <label for="password1"><:inscription2:password_retaper:></label>
        <input type="password" class="text" autocomplete="off" name="password1" id="password1" value="#ENV{password1}" />
- [<p class='erreur'>(#ENV**{erreurs}|table_valeur{password})</p>]
+ [<p class='erreur'>(#ENV**{erreurs}|table_valeur{pass})</p>]
    </li>]
  </ul>
</fieldset>
\ No newline at end of file
_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

_______________________________________________
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone