Evolution #582: Implémenter le mode nopass

app/Resources/translations/flash.en.yml

@@ -31,6 +31,9 @@ user:
     success:       Your address has been updated.
     wait:          You must wait to change your email again.
   session_expired: You have been disconnecter, please, log in again.
+  change_username:
+    success:       Your username has been updated.
+    failure:       An error occured.
 
 presubscription:
   success:         An email has been sent for confirmation.

app/Resources/translations/flash.fr.yml

@@ -31,6 +31,9 @@ user:
     success:       Votre email a correctement été mis à jour.
     wait:          Vous devez attendre avant de pouvoir faire une nouvelle demande de changement d'email.
   session_expired: Vous avez été déconnecté, veuillez vous ré-identifier
+  change_username:
+    success:       Votre nom d'utilisateur a bien été modifié
+    failure:       Une erreur s'est produite
 
 presubscription:
   success:         Un email a été envoyé a votre adresse email pour confirmation (vérifiez aussi dans vos SPAM)

app/Resources/translations/userform.en.yml

@@ -2,5 +2,6 @@
 password.actual: Actual
 password.new.first: New
 password.new.second: New (check)
-
+username: Username
+new_username:  New username
 accept_cgu_registration: By processing registration you accept <a href="%link%">terms of service</a> of muzi.ch

app/Resources/translations/userform.fr.yml

@@ -3,7 +3,8 @@ password.actual: Actuel
 password.new.first: Nouveau
 password.new.second: Nouveau (vérification)
 
-username:  Nom d'utilisateur
+username:      Nom d'utilisateur
+new_username:  Nouveau nom d'utilisateur
 email:     Email
 password:
   first:   Mot de passe

app/Resources/translations/userui.fr.yml

@@ -40,10 +40,12 @@ title:
   about:         À propos
   search:        Recherche
   cgu:           Informations légales
-    
+  change_username: Change son nom d'utilisateur
+  
 my_account: 
   title:          Mon compte
   username:         Nom d'utilisateur
+  change_username:  modifier
   email:            Adresse email
   favorite_tags:    Vos tags préférés
   change_password:  Changer mon mot de passe
@@ -73,7 +75,11 @@ my_account:
     submit:         Enregistrer mes préférences
     success:        Vos préférences ont bien été mises à jour
     error:          Une erreur est survenue lors de la mise à jour de vos préférences
-  
+
+change_username:
+  submit:           Changer
+  warning:          Attention: une fois votre nom d'utilisateur modifié vous ne pourrez plus en changer !
+
 connexion:
   password_lost:    Mot de passe perdu ?
   

app/Resources/translations/validators.en.yml

@@ -38,8 +38,9 @@ error:
   registration:
     email:
       duplicate:         Email address is already in use on muzi.ch
+  change_username:
+    duplicate:           Username is already in use on muzi.ch
   
-      
 registration:
   token:
     error:               The code entered for validation is not valid anymore.

app/Resources/translations/validators.fr.yml

@@ -7,7 +7,6 @@
 'The username is already used'   :   Le nom d'utilisateur est déjà utilisé.
 'The email is already used'      :   Cette adresse email est déjà utilisée.
 
-
 error:
   group:
     name:
@@ -38,8 +37,11 @@ error:
   registration:
     email:
       duplicate:         L'adresse email est déjà utilisé sur muzi.ch
+  change_username:
+    duplicate:           Le nom d'utilisateur est déjà utilisé sur muzi.ch
+    min:               Votre nom d'utilisateur doit contenir au moins %limit% caractères.
+    max:               Votre nom d'utilisateur doit contenir au maximum %limit% caractères.
   
-      
 registration:
   token:
     error:               Le code saisi pour l'inscription est incorrect ou n'est plus utilisable

src/Muzich/CoreBundle/Entity/User.php

@@ -1112,7 +1112,7 @@ class User extends BaseUser
       $this->username_updatable = false;
   }
   
-  public function isUsernameUpdateable()
+  public function isUsernameUpdatable()
   {
     if ($this->username_updatable)
       return true;

src/Muzich/CoreBundle/Resources/public/css/main.css

@@ -2055,4 +2055,13 @@ div.side_margin_top
 p.accept_cgu
 {
   font-size: 11px;
-}
+}
+
+li.user_name a
+{
+  color: inherit;
+}
+
+li.user_name a:hover
+{
+text-decoration: underline;

src/Muzich/UserBundle/Controller/UserController.php

@@ -685,4 +685,74 @@ class UserController extends Controller
     ));
   }
   
+  public function changeUsernameAction(Request $request)
+  {
+    if (!$this->getUser()->isUsernameUpdatable())
+    {
+      return new RedirectResponse($this->generateUrl('my_account'));
+    }
+    $errors = array();
+    $form = $this->getChangeUsernameForm($this->getUser());
+    if ($request->getMethod() == 'POST')
+    {
+      $form->bind($request);
+      $errors = $this->checkChangeUsernameValues($form);
+      if ($form->isValid() && !count($errors))
+      {
+        $form->getData()->setUsernameUpdatable(false);
+        $this->persist($form->getData());
+        $this->flush();
+        $this->setFlash('success', 'user.change_username.success');
+        return new RedirectResponse($this->generateUrl('my_account'));
+      }
+      else
+      {
+        $this->setFlash('error', 'user.change_username.failure');
+      }
+    }
+    
+    return $this->render('MuzichUserBundle:User:change_username.html.twig', array(
+      'form'   => $form->createView(),
+      'errors' => $errors
+    ));
+  }
+  
+  protected function checkChangeUsernameValues($form)
+  {
+    $errors = array();
+    $userManager = $this->container->get('fos_user.user_manager');
+    if ($userManager->findUserByUsername($form->getData()->getUsername()))
+    {
+      $errors[] = $this->trans('error.change_username.duplicate', array(), 'validators');
+    }
+    
+    if (strlen($form->getData()->getUsername()) < 3)
+    {
+      $errors[] = $this->trans(
+        'error.change_username.min', 
+        array('%limit%' => 3),
+        'validators'
+      );
+    }
+  
+    if (strlen($form->getData()->getUsername()) > 32)
+    {
+      $errors[] = $this->trans(
+        'error.change_username.max', 
+        array('%limit%' => 32),
+        'validators'
+      );
+    }
+    
+    return $errors;
+  }
+  
+  protected function getChangeUsernameForm(User $user)
+  {
+    return $this->createFormBuilder($user)
+      ->add('username', 'text')
+      ->getForm()
+    ;
+  }
+  
 }

src/Muzich/UserBundle/Resources/config/routing.yml

@@ -64,3 +64,7 @@ user_hide_help:
 subscribe_or_login:
   pattern: /ajax/subscribe-or-login
   defaults: { _controller: MuzichUserBundle:User:subscribeOrLogin }
+
+change_username:
+  pattern: /account/username/update
+  defaults: { _controller: MuzichUserBundle:User:changeUsername }

src/Muzich/UserBundle/Resources/views/User/account.html.twig

@@ -12,6 +12,9 @@
     <ul>
       <li>
         <b>{{ 'my_account.username'|trans({}, 'userui') }}</b>: {{ user.name }}
+        {% if app.user.isUsernameUpdatable %}
+          (<a href="{{ path('change_username') }}">{{ 'my_account.change_username'|trans({}, 'userui') }}</a>)
+        {% endif %}
       </li>
       <li>
         <b>{{ 'my_account.email'|trans({}, 'userui') }}</b>: {{ user.email }}

src/Muzich/UserBundle/Resources/views/User/change_username.html.twig

@@ -0,0 +1,40 @@
+{% extends "MuzichUserBundle::layout.html.twig" %}
+
+{% block title %}{{ 'title.change_username'|trans({}, 'userui') }}{% endblock %}
+{% block mainbox_classes %}mainbox_padding{% endblock %}
+
+{% block content %}
+
+<h1>{{ 'title.change_username'|trans({}, 'userui') }}</h1>
+
+  <form id="change_username_update" action="{{ path('change_username') }}" method="post">
+    
+    <p class="warning">
+      {{ 'change_username.warning'|trans({}, 'userui') }}
+    </p>
+    
+    {{ form_errors(form) }}
+    
+    {% if errors|length > 0 %}
+    <ul class="error_list">
+      {% for error in errors %}
+        <li>{{ error }}</li>
+      {% endfor %}
+    </ul>
+    {% endif %}
+    
+    <div class="field">
+      {{ form_errors(form.username) }}
+      {{ form_label(form.username, 'new_username'|trans({}, 'userform')) }}:
+      {{ form_widget(form.username, {'attr':{'class':'niceinput'}}) }}
+    </div>
+    
+    {{ form_widget(form._token) }}
+    
+    <div class="alignright">
+      <input type="submit" class="button" value="{{ 'change_username.submit'|trans({}, 'userui') }}" />
+    </div>
+    
+  </form>
+
+{% endblock %}