[cabot]

A la rigueur tu peux ajouter une classe css sur les balises img et tu déclares un max-width dessus :

Code: Tout sélectionner

img.laclasse { max-width: 100%; }

[cabot]

Petit problème dans le remplacement des termes avec la balise acronym.

J'ai ajouté le terme lorem dans le glossaire :


J'ai publié un bout de texte :

Lorem ipsum dolor sit amet, consectetur adipiscing elit. Cras augue augue, maximus sit amet egestas vitae, pretium quis nisi. Nulla facilisi. Quisque egestas bibendum mi, quis fringilla augue tincidunt sed.

L'affichage est bogué, deux balises acronym altérées sont renvoyées :

[pierredu]

Cabot,

Merci pour le pointeur. Je vais regarder ce qui se passe.
À noter que l'on n'a pas besoin de spécifier avec ou sans majuscule dans les termes à rechercher. Peut-être est-ce la cause ? Je vais voir.

Pour l'idée de limiter la largeur dans une feuille de style, pourquoi pas ?

Je parlais en fait du filtre au téléchargement. On peut fixer les extensions et le poids en ko du fichier, et c'est correctement intercepté, mais pas la largeur en pixels.

Un peu plus tard : c'est bien la présence de deux termes identiques à rechercher qui provoque la bogue (la majuscule dans les termes à rechercher n'a pas d'importance), que le mot-clef dans le texte soit en majuscules ou en minuscules. Ce doit être la fonction de recherche-remplacement utilisée (preg_replace) qui s'exécute récursivement sur chacun des éléments des tableaux qu'on lui passe en paramètres. À noter que cela ne se produit pas dans le cas des formes masculin, féminin, singulier et pluriel (on travaille sur des mots complets).

Comme c'est le genre d'erreur d'utilisation que l'on ne peut pas exclure, il va falloir que j'imagine une solution.

Un peu plus tard encore :
Nouvelle version 1.3.11 téléchargée, qui corrige cette bogue.

[pierredu]

A la rigueur tu peux ajouter une classe css sur les balises img et tu déclares un max-width dessus :

Code: Tout sélectionner

img.laclasse { max-width: 100%; }

Faute de pouvoir trouver la raison pour laquelle mon code ne parvient pas à limiter la taille en pixels, je viens de le faire et ça marche bien. Merci.

[pierredu]

Je viens de télécharger une nouvelle version 1.3.12 sur le dépôt github.

Changement :
J'avais un module (lexicon.php) dont j'avais hérité dans le MOD que je porte et qui n'était pas intégré dans le mécanisme des services et du controller, et je ne savais pas par quel bout le prendre. Au détour d'une extension, j'ai découvert une classe json_response, qui était précisément ce dont j'avais besoin.

Dakin Quelia avait bien dit que mon mécanisme ressemblait à de l'AJAX. C'est vrai, mais à la sauce Json, donc AJAJ, même si cela ne se dit pas trop. J'ai donc pu faire passer l'appel à la fonction de consultation par l'aiguillage du controller et ça marche bien.

[pierredu]

Nouvelle version 1.3.13 téléchargée sur github.

Changements : gestion correcte des contraintes de taille (en pixels) et de poids (en ko) lors du téléchargement d'illustrations.

[pierredu]

Nouvelle version 1.3.14 téléchargée sur github.

Changements : différents aspects cosmétiques, interception d'un essai de téléchargement sans indiquer de fichier à télécharger.

[pierredu]

Nouvelle version 1.3.15 téléchargée sur github.

Changements :
- Protection contre la présence d'un caractère + en tête d'un terme (contradiction avec la valeur du caractère dans une expression régulière)
- Addition d'une fonction permettant à l'administrateur de définir la longueur du texte affiché dans l'infobulle lorsque cette fonctionnalité est validée. Addition d'une rubrique dans la table config, en passant par un second fichier de migration.

[pierredu]

Nouvelle version 1.3.19 téléchargée sur github.

Changement :
Un utilisateur potentiel me demandait sur phpbb.com s'il serait possible de spécifier les forums dans lesquels la fonction du glossaire serait validée. Ce n'est pas vraiment difficile à faire, je l'ai donc fait.
On a donc un double degré : validation globale par un administrateur, validation sélective des forums et validation par l'utilisateur (qui peut ne pas aimer ces éléments visuellement perturbateurs).

[pierredu]

La version la plus récente sur github est compatible 3.1/3.2 (la seule différence réside dans le mode de téléchargement des fichiers d'image).

[pierredu]

La version était bien compatible avec 3.2.0, mais pas tout à fait.

Nous avons fini par découvrir des problèmes dans les messages qui comportaient une citation. Dans ce cas, le balisage du terme du glossaire ne fonctionnait pas.

En demandant de l'aide sur .com, j'ai eu un message de l'auteur du paquetage de formatage de texte qui a été installé dans le code de 3.2.0. Grosso modo, il me demandait de passer après lui. Sauf que, du coup, j'avais à traiter des tas de chaînes qui ne font pas partie du message, comme la signature, la description du forum, etc. La gestion des forums validés/inhibés fonctionnait mal, également, etc.

Et puis je me suis aperçu que le problème se posait également avec les smileys, bref avec tout ce qui n'était pas du code hypersimple. D'ailleurs, l'analyseur syntaxique qui tourne avant l'enregistrement des messages dans le nouveau format pose une balise <t> autour des messages simples et une balise <r> auteur de ceux qui doivent être "traités".

Je suis allé voir le code du renderer (voir rhea\vendor\s9e\text-formatter\src\Bundles\Forum\Renderer.php, ligne 49). Allez voir, ce n'est pas piqué des hannetons, même si c'est du code minifié. Quand je pense qu'on vous renvoie une extension à cause de la présence d'un seul CR et trop !!

Bref, je me suis abonné aux événements avant et après du formateur et je remplace mes balises par un marque-place avant, pour les remettre après.

[Zeuhl]

Bonjour,

Merci pour cette extension dont l'utilité ne fait aucun doute. Je souhaiterais vraiment en faire profiter mes forums où nous usons d'une terminologie qui demande un certain temps d'adaptation.

Ma question est simple : quel risque fais-je courir à mon forum en adoptant une extension qui n'a pas encore reçu une validation officielle de phpBB.com ? Mon plus vieux forum a déjà 10 ans, un millier de membres et plus de 130000 posts. Je ne peux pas vraiment me permettre de prendre le moindre risque.

[ABDev]

Salut Pierre,
Dis, cette partie me "choque" dans ton fichier acp/gloss_module.php :

Code: Tout sélectionner

// Forum enabling/disabling
$enabled_forums = implode(',', $request->variable('mark_glossary_forum', array(0), true));
$sql = 'UPDATE ' . FORUMS_TABLE . '
    SET lmdi_glossary = 0';
$db->sql_query($sql);
if (!empty ($enabled_forums))
{
    $eforums = explode (',', $enabled_forums);
    $nbf = count ($eforums);
    for ($i=0; $i<$nbf; $i++)
    {
        $numf = $eforums[$i];
        $sql = "UPDATE " . FORUMS_TABLE . "
            SET lmdi_glossary = 1
            WHERE forum_id = $numf";
        $db->sql_query($sql);
    }
    $cache->put('_gloss_forums', $eforums, 86400);    // 24 h
} 

Tu pourrais l'optimiser avec ceci :

Code: Tout sélectionner

// Forum enabling/disabling
$enabled_forums = implode(',', $request->variable('mark_glossary_forum', array(0), true));
$sql = 'UPDATE ' . FORUMS_TABLE . '
    SET lmdi_glossary = 0';
$db->sql_query($sql);
if (!empty($enabled_forums))
{
    $sql = 'UPDATE ' . FORUMS_TABLE . '
        SET lmdi_glossary = 1
        WHERE forum_id IN (' . $enabled_forums . ')';
    $db->sql_query($sql);

    $cache->put('_gloss_forums', $eforums, 86400);    // 24 h
} 

Ça t'évite ainsi une boucle inutile.

Bon j'ai remarqué d'autres coquilles, je te ferai un topo ASAP .

[pierredu]

ABDev,

Je regarde cela demain.
Merci d'avance pour toutes tes autres remarques.

[pierredu]

Ma question est simple : quel risque fais-je courir à mon forum en adoptant une extension qui n'a pas encore reçu une validation officielle de phpBB.com ? Mon plus vieux forum a déjà 10 ans, un millier de membres et plus de 130000 posts. Je ne peux pas vraiment me permettre de prendre le moindre risque.

Cette extension tourne sur mon forum (voir ma signature) depuis toujours.
Même si le code peut être amélioré, je ne pense pas qu'il y ait le moindre risque à l'utiliser.