[RC] [3.1][3.2]LMDI Glossary 1.4.10

Pour les extensions destinées à phpBB 3.1.x

Modérateur: Equipe

Règles du forum
A lire impérativement : Règlement de phpBB-fr.com

Re: LMDI Glossary 1.3.10

Messagepar cabot » 26 Fév 2016 à 10:41

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%; }
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26741
Enregistré le: 13 Fév 2011 à 20:42
Localisation: Rarement sur la plaque

Re: LMDI Glossary 1.3.10

Messagepar cabot » 26 Fév 2016 à 12:46

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

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

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 :
Image
Image
Avatar de l’utilisateur
cabot
Equipe site
Equipe site
 
Messages: 26741
Enregistré le: 13 Fév 2011 à 20:42
Localisation: Rarement sur la plaque

Re: LMDI Glossary 1.3.10

Messagepar pierredu » 26 Fév 2016 à 13:15

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.10

Messagepar pierredu » 02 Mar 2016 à 12:36

cabot a écrit: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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.12

Messagepar pierredu » 03 Mar 2016 à 18:23

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.13

Messagepar pierredu » 07 Mar 2016 à 11:07

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.14

Messagepar pierredu » 11 Mar 2016 à 12:35

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.15

Messagepar pierredu » 22 Mar 2016 à 15:01

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: LMDI Glossary 1.3.19

Messagepar pierredu » 27 Mai 2016 à 17:02

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).
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: [3.1][3.2]LMDI Glossary 1.3.27

Messagepar pierredu » 12 Jan 2017 à 16:35

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).
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: [3.1][3.2]LMDI Glossary 1.4.1

Messagepar pierredu » 09 Mar 2017 à 17:12

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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: [3.1][3.2]LMDI Glossary 1.4.1

Messagepar Zeuhl » 22 Mar 2017 à 18:57

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.
Zeuhl
Apprenti-posteur
Apprenti-posteur
 
Messages: 121
Enregistré le: 23 Avr 2012 à 16:30

Re: [3.1][3.2]LMDI Glossary 1.4.1

Messagepar ABDev » 22 Mar 2017 à 21:17

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'$eforums86400);    // 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'$eforums86400);    // 24 h

Ça t'évite ainsi une boucle inutile.

Bon j'ai remarqué d'autres coquilles, je te ferai un topo ASAP ;).
Pour se payer le luxe d'être feignant, il faut sans cesse faire travailler ses méninges ! (Korben)

On clique sur le bouton Répondre plutôt que le bouton Citer ...
Un problème avec une extension ? Pas de lien vers celle-ci ? Sujet ignoré !
Avatar de l’utilisateur
ABDev
Programmationateur
Programmationateur
 
Messages: 15695
Enregistré le: 12 Mai 2005 à 23:54

Re: [3.1][3.2]LMDI Glossary 1.4.1

Messagepar pierredu » 22 Mar 2017 à 23:28

ABDev,

Je regarde cela demain.
Merci d'avance pour toutes tes autres remarques.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

Re: [3.1][3.2]LMDI Glossary 1.4.1

Messagepar pierredu » 22 Mar 2017 à 23:30

Zeuhl a écrit: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.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1033
Enregistré le: 29 Mai 2011 à 06:49
Localisation: Paris

PrécédenteSuivante

Retourner vers Extensions en développement

 


  • Articles en relation
    Réponses
    Vues
    Dernier message

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 1 invité