2 améliorations suggérées

[atlan]

Lors de l'inscription d'un client (ou d'une commande par un invité), remplir la case du code postal n'est pas obligatoire alors que le menu déroulant pour le Département l'est. Est-ce qu'il ne serait pas mieux de faire l'inverse. Le code postal est obligatoire pour expédier une commande alors que le Département pourrait être facultatif.
Merci de votre intérêt.

[LeorLindel]

Tu peux en effet rendre le champ du code postal obligatoire, il te faut pour cela modifier plusieurs fichiers.

Pour le fichiers :

catalog/controller/account/create.php
catalog/controller/account/address.php
catalog/controller/checkout/address.php
catalog/controller/checkout/guest_step_1.php

Il faut trouver ces lignes :

Code: Tout sélectionner

         
      if (isset($this->error['city'])) {
         $this->data['error_city'] = $this->error['city'];
      } else {
         $this->data['error_city'] = '';
      }



Et ajouter ces lignes dessous :

Code: Tout sélectionner

      if (isset($this->error['postcode'])) {
         $this->data['error_postcode'] = $this->error['postcode'];
      } else {
         $this->data['error_postcode'] = '';
      }



Puis chercher ces lignes :

Code: Tout sélectionner

       if ((strlen(utf8_decode($this->request->post['city'])) < 3) || (strlen(utf8_decode($this->request->post['city'])) > 128)) {
            $this->error['city'] = $this->language->get('error_city');
       }



Et ajouter ces lignes dessous :

Code: Tout sélectionner

       if ((strlen(utf8_decode($this->request->post['postcode'])) < 5)) {
            $this->error['city'] = $this->language->get('error_postcode');
       }



Pour le fichiers :

catalog/language/french/account/create.php
catalog/language/french/account/address.php
catalog/language/french/checkout/address.php
catalog/language/french/checkout/guest_step_1.php

Chercher cette ligne ;

Code: Tout sélectionner

$_['error_city']        = 'La <b>Ville</b> doit &ecirc;tre compos&eacute;e de 3 &agrave; 128 caract&egrave;res !';


Et ajouter celle-ci dessous :

Code: Tout sélectionner

$_['error_postcode']    = 'Le champ <b>Code Postal</b> doit &acirc;tre compos&eacute; de 5 caract&egrave;res !';


Pour le fichiers :

catalog/view/theme/default/template/account/create.tpl
catalog/lview/theme/default/template/account/address.tpl
catalog/view/theme/default/template/checkout/address.tpl
catalog/view/theme/default/template/checkout/guest_step_1.tpl

Chercher ces lignes :

Code: Tout sélectionner

          <tr>
            <td><?php echo $entry_postcode; ?></td>
            <td><input type="text" name="postcode" value="<?php echo $postcode; ?>" /></td>
          </tr>


Et les remplacer par celles-ci :

Code: Tout sélectionner

          <tr>
            <td><span class="required">*</span><?php echo $entry_postcode; ?></td>
            <td><input type="text" name="postcode" value="<?php echo $postcode; ?>" /></td>
              <?php if ($error_postcode) { ?>
              <span class="error"><?php echo $error_postcode; ?></span>
              <?php } ?></td>
          </tr>


It's all

[atlan]

Merci beaucoup pour ce codage 5 étoiles.

Le code postal a été rendu obligatoire sans difficultés.

[Marc]

Ceci est une amélioration techniquement bénigne mais qui me semble indispensable pour la France...

J'imagine que le fonctionnement d'origine est liés au système postal américain...
Si c'est bien le cas, je pense que ce serait difficile de demander d'ajouter cela dans la version de base...

Pensez-vous que cela pourrait faire l'objet d'un ajout systématique dans la version française (en plus de la traduction) ?

Ma question est peut-être bête... J'avoue ne pas bien me rendre compte du travail que ça représenterait pour l'équipe française d'ajouter ça avant publication de chaque version... Aussi je comprendrais que cela ne puisse être possible aussi facilement que je l'imagine

[LeorLindel]

J'imagine que le fonctionnement d'origine est liés au système postal américain...
Si c'est bien le cas, je pense que ce serait difficile de demander d'ajouter cela dans la version de base...

Tout à fait le système postal américain est basé différemment du système français et cela ne sera jamais modifié dans ce sens dans la version de base d'Opencart.

Pensez-vous que cela pourrait faire l'objet d'un ajout systématique dans la version française (en plus de la traduction) ?

La règle que nous nous sommes fixé est de ne pas différer de la version de base, donc inclure cette modification aux versions postérieures serait y déroger.
Donc si vous souhaitez que cette modification soit effective sur votre boutique, il vous faudra l'inclure "à la mano".

Sorry.

[Marc]

Merci pour cette réponse claire.
je ne connaissait pas cette politique. Je la comprend aussi. Plus on s'éloigne de l'origine, plus il y a de la maintenance à faire lors des mises à jour...

Reste à trouver le moyen de proposer des "mises à jour externes" simples à insérer pour tout le monde sans avoir à ouvrir des fichiers de code et sans que ça annule des modifications précédente d'autres modules...
Je compte me pencher sur ce sujet pour proposer sur le forum français les modifs que je mets en place sur mes projets en développement.

=> Si vous avez déjà une méthode efficace, des outils ou de simples conseils : je suis preneur !!

[LeorLindel]

Dans l'idéal, il faudrait que ces modifications de code soient mis en pack pour le téléchargement incluant tous les fichiers modifiés en signalant bien entendu la version concernée, mais cela deviendrait vite ingérable en cas de fichiers modifiés par plusieurs modifications ainsi que par l'évolution des versions.
Par contre si quelqu'un désire s'occuper de cette maintenance et mettre en téléchargement les fichiers concernés en indiquant la version d'opencart s'y référant, il peut poster dans chaque sujet impliqué.

[Marc]

La modification de LeorLindel est intéressante...

En accord avec ce que suggérait atlan, je m'apprête à retirer l'obligation de fournir le département.

Je n'ai pas encore regardé dans le détail la gestion des modes de livraison... Cela risque-t-il de créer des bugs dans ces modules ? Et dans d'autres modules ?

[LeorLindel]

Comme l'a souligné Delete l'obligation de rendre obligatoire le code postal est désormais mis en application sur la version 1.4.9, donc les modifications ci-dessus sont valables pour les versions < 1.4.9.

[Marc]

Excuses moi LeordLindel... Ce n'est pas ma question

Je ne parlais pas des modifications que tu as apportées ici qui ne concernent pas le département.
Or delete comme toi avez bien exprimé le fait que les modifications que tu as données, concernant exclusivement le code postal, ont été inséré dans la 1.4.9.

Ma question porte sur le département : peut-on retirer l'obligation de remplir ce champ sans souci ?

Je n'ai pas encore vu la 1.4.9 : le département peut-il être non obligatoire ? Si tu me dis que oui, alors là ça répond pleinement à ma question

[LeorLindel]

Dans la version 1.4.9, le département est obligatoire.

[Parenthèse]
Pour la petite histoire, je m'intéresse à une adaptation qui remplirait le champs "ville" par menu déroulant en fonction du code postal, mais il faudrait modifier l'ordre de ces informations dans le fichier create.tpl car actuellement le code postal se trouve après la ville, ce qui illogique pour nous français.
Voir les sources.
Mais je m'occuperai de cela un peu plus tard ou si quelqu'un se sent l'envie de le faire.
Mais ce n'est pas le sujet, je déborde.
[/Parenthèse]