Projet

Général

Profil

Actions

Anomalie #1111

fermé

plugin-auto : Bug sur menu liste des finitions

Ajouté par Jérémy HIEULLE il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
08/11/2017
Echéance:
% réalisé:

100%

Temps estimé:

Description

Petite typo dans le dernier commit https://git.tuxfamily.org/galette/plugin-auto.git/commit/?h=develop&id=3af1f1d48036bf533c535d768c553ac7104c0c36
Ligne 7 du fichier menu.tpl
- <li{if $cur_route eq "finitionsList || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})"} class="selected"{/if}><a href="{path_for name="finitionsList"}">{_T string="Finitions list" domain="auto"}</a></li>
+ <li{if $cur_route eq "finitionsList" || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})"} class="selected"{/if}><a href="{path_for name="finitionsList"}">{_T string="Finitions list" domain="auto"}</a></li>

Mis à jour par Jérémy HIEULLE il y a plus de 6 ans

Jérémy HIEULLE a écrit :

Petite typo dans le dernier commit https://git.tuxfamily.org/galette/plugin-auto.git/commit/?h=develop&id=3af1f1d48036bf533c535d768c553ac7104c0c36
Ligne 7 du fichier menu.tpl
- <li{if $cur_route eq "finitionsList || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})"} class="selected"{/if}><a href="{path_for name="finitionsList"}">{_T string="Finitions list" domain="auto"}</a></li>
+ <li{if $cur_route eq "finitionsList" || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})"} class="selected"{/if}><a href="{path_for name="finitionsList"}">{_T string="Finitions list" domain="auto"}</a></li>

Ben mois aussi je me trompe, la bonne ligne :
+ <li{if $cur_route eq "finitionsList" || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})"} class="selected"{/if}><a href="{path_for name="finitionsList"}>{_T string="Finitions list" domain="auto"}</a></li>

Cordialement,

Mis à jour par Jérémy HIEULLE il y a plus de 6 ans

Désolé pour le spam... la prochaine fois je mettrai le patch sinon c'est illisible..

<li{if $cur_route eq "finitionsList" || ($cur_route eq 'propertyEdit' && $cur_subroute eq {_T string='finition' domain='auto_routes'})} class="selected"{/if}><a href="{path_for name="finitionsList"}">{_T string="Finitions list" domain="auto"}</a></li>

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Statut changé de Résolu à Fermé
  • Assigné à mis à Johan Cwiklinski

Merci pour le correctif ; c'est appliqué.

Pour proposer un correctif, le mieux reste le patch GIT (au moins, le commit porte le nom de celui qui l'a réellement effectué... et pas le mien qui ait bêtement copié/collé).

Tu peux tout à fait fournir ce type de patch ici ou sur la liste de développement ; ou si tu préfères, ouvrir une PR sur github. Pour moi, ça ne change pas grand chose, et c'est bien plus simple et lisible ;)

Actions

Formats disponibles : Atom PDF