phpBB-fr.com

[Hibou57]

• Titre: Smiley ALT
• Description: Assigne l'attribut ALT des smileys au texte de l'émotion
• Version: 1.0.1
• Auteur: Hibou57
• Historique des versions:
  
  • 1.0.0: Version initiale
  • 1.0.1: Application des suggestions de ABDev (dans le second message de ce fil)
• Captures d’écran: n/a
• Téléchargement: smiley_alt_1_0_1.zip

Petit MOD pour remplacer l’attribut ALT des smileys, qui est par défaut assigné au BBCode du smiley, par le texte associé à l’émotion (le même que pour l’attribut TITLE). Ça peut toujours se discuter, mais disons que normalement un attribut ALT doit contenir un texte lisible en toutes lettres…

Ça ne s’applique qu’aux smileys dans les messages édités après l’ajout du MOD. Il était possible de faire autrement, mais l’autre manière le faisait entrer en conflit avec un MOD déjà existant, Smile Middle.

[ABDev]

Hello,
Ton fichier n'est pas conforme à la norme MODX.
Il doit normalement s'appeler install.xml

De plus, faire un remplacement d'une ligne complète n'est pas recommandé.
Il vaut mieux faire une recherche partielle, comme ceci :

Code: Tout sélectionner

<open src="includes/message_parser.php">
<edit>
   <find><![CDATA[$replace[] = '<!-- s' . $row['code'] . ' --><img src="{SMILIES_PATH}/' . $row['smiley_url'] . '" alt="' . $row['code'] . '" title="' . $row['emotion'] . '" /><!-- s' . $row['code'] . ' -->';]]></find>
   <inline-edit>
      <inline-find>$row['code']</inline-find>
      <inline-action type="replace-with">$row['emotion']</inline-action>
   </inline-edit>
</edit>
</open>

[Hibou57]

Ton fichier n'est pas conforme à la norme MODX.
Il doit normalement s'appeler install.xml

J’aurais dut faire confiance au doute que j’avais

De plus, faire un remplacement d'une ligne complète n'est pas recommandé.

Même une recherche sur le contenu d’une ligne complète aussi en fait. J’ai corrigé les deux (premier message mis à jour).

J’ai bien fait de poster ce MOD alors, j'ai appris une chose et pensé à faire plus attention à une autre.

Merci pour tes deux commentaires

[ABDev]

Je ne comprends pas pourquoi tu t'embêtes à rechercher '" alt="' . $row['code'] alors que $row['code'] est amplement suffisant ...
Idem pour le remplacement qui suit !

[Hibou57]

Je pensais que c’était plus parlant.

J'ai remplacé par

Code: Tout sélectionner

<inline-find><![CDATA[code]]></inline-find>
<inline-action type="replace-with"><![CDATA[emotion]]></inline-action>


Cherche code et remplace par emotion. Après tout la concision, c’est bien aussi…

Merci encore pour tout

[ABDev]

Quand on modifie une donnée d'un tableau, on conserve sa structure. Ça fait partie des prérogatives !
Donc ce que tu as fait là n'est toujours pas bon, d'où mes deux précédents écrits.