Modération des commentaires avec v 5.8.10

PatE2lPatE2l Member

Bonjour,
Je suis le responsable de l'association l'Ecole du Logiciel Libre : https://e2li.org/
Nous sommes en train de tester PluXml version 5.8.10 pour voir si cela peut convenir pour notre futur blog public. :-)
Et l'affichage des commentaires non publiés (en attente de modération) ne se fait pas dans la liste.
Il s'agit de l'URL : core/admin/comments.php?sel=offline&page=1
Le format regex est bon avec le caractère '_' devant les noms des fichiers, les fichiers sont bien dans data/commentaires, mais la liste est vide !?
Bon courage ;-)
Patrick

Réponses

  • PatE2lPatE2l Member

    Bonjour,
    Je me suis dis que cette fonctionnalité étant importante, cela pouvait venir de la mise à jour du code pour supporter php 8. Et j'ai donc fait deux tests :

    • le 1er sous Debian 11 avec php 7.4
    • le 2e sous Debian 12 php 8.2
      mais le résultat est identique ... :-)

    Patrick

  • PatE2lPatE2l Member

    Bonjour,
    Je pense avoir trouvé : c'est le format regex de départ qui n'est pas bien formulé.
    Je propose de remplacer dans le fichier core/lib/class.plx.glob.php ligne 14 :
    'commentaires' => '#^\d{4,}.(?:\d{10,})(?:-\d+)?.xml$#'
    par
    'commentaires' => '#^_?\d{4,}.(?:\d{10,})(?:-\d+)?.xml$#'

    Merci pour votre travail :-)
    Salutations.
    Patrick

  • bazooka07bazooka07 PluXml Lead Developer, Moderator

    @PatE2l ,
    Ne pas oublier d'échapper les points, sinon c'est n'importe caractère dans les regex.

    Les corrections seront incluses dans PluXml-5.8.11 prochainement. Cette version comportera également un correctif pour les plugins incompatibles avec PHP-8.x.x.

    Pour info, on peut faire tourner ensemble plusieurs versions de PHP sur un même serveur Nginx couplé à un serveur php-fpm pour chaque version de PHP et en choisissant un nom d'hôte différent pour chaque version de PHP. Pour chaque hôte virtuel, il suffit de changer le socket pour que NGinx communique avec le bon serveur php-fpm.

    Je souhaite faire un paquet Debian pour PluXml. Il existe un paquet sur Ubuntu avec une version obsolète de PluXml.
    Avez-vous un packageur Debian dans votre équipe qui pourrait m'aider ?

  • bazooka07bazooka07 PluXml Lead Developer, Moderator
  • PatE2lPatE2l Member
    1 avril modifié

    Bonjour,
    Merci pour la nouvelle version, nous allons la tester prochainement.
    Mais nous sommes juste une école (voir détails sur e2li.org) et ne diffusant pas de logiciel spécifique nous n'avons pas de packageur Debian (ou pour d'autres distributions).
    Par contre nous utilisons très souvent Debian et faisons nos tests avec Docker. On peut vous fabriquer une image Docker avec Debian :-)
    Bien à vous.
    Patrick

  • bazooka07bazooka07 PluXml Lead Developer, Moderator

    Bonjour,
    Non, je recherche réellement un packageur Debian pour

    • automatiser l'installation en gérant les dépendances
    • déplacer le dossier core/lib dans usr/share/lib/pluxml ou en dehors du DocumentRoot
    • gérer le .htaccess au niveau du serveur et non du site
    • laisser une alternative entre Apache2 et NGinx
    • se conformer à la politique de Debian

    Je sais faire ces actions à la main. Mais l'idée est de faire automatiquement une installation prête à l'emploi

    Pour Docker, j'avais créé une image avec Alpine-Linux, noyau de distribution très léger. Mais c'était il y a 5 ans environ.
    Certainement quelques mises à jour à faire. C'est par là :
    https://github.com/bazooka07/docker-alpine-pluxml

    Je fais également tourner PluXml sur une SBC ( small board computer ) Banana Pi Berry, clone du Raspberry Pi, avec un serveur NGinx + PHP-FPM. Permet de changer de version de PHP simplement en changeant d'hôte virtuel.

    Puisque vous êtes une école d'informatique, jevous encourage fortement à cloner le dépôt Github plutôt que de télécharger une archive Zip
    https://github.com/pluxml/PluXml

    Branche master pour la prochaine version stable
    Branche 5.8.10 pour la version stable actuelle

  • PatE2lPatE2l Member
    1 avril modifié

    Bonsoir,

    Merci pour toutes ces infos.
    J'en profite pour vous faire un retour provisoire des premiers tests avec la version 5.8.11 :

    • le bug signalé ci-dessus a bien été corrigé,
    • il y a un message d'erreur lorsque l'on clique sur un des mots clés que voici
      Fatal error : type : 8192 message : Creation of dynamic property plxMotor::$cibleName is deprecated file : /core/lib/class.plx.motor.php line : 225 See https://www.php.net/manual/en/errorfunc.constants.php about type of error
      ============================================================
      User : 001 Profil : 0 PluXml version : 5.8.11 PHP version : 8.2.7 ============================================================
      About this server : HTTP_USER_AGENT : Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0 HTTP_ACCEPT : text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,/;q=0.8 HTTP_ACCEPT_LANGUAGE : fr,fr-FR;q=0.8,en-US;q=0.5,en;q=0.3 HTTP_ACCEPT_ENCODING : gzip, deflate HTTP_REFERER : http://192.168.56.121/index.php?article1/premier-article SERVER_SIGNATURE :
      Apache/2.4.57 (Debian) Server at 192.168.56.121 Port 80
      SERVER_SOFTWARE : Apache/2.4.57 (Debian) SCRIPT_FILENAME : /index.php SERVER_PROTOCOL : HTTP/1.1 REQUEST_METHOD : GET QUERY_STRING : tag/pluxml REQUEST_URI : /index.php?tag/pluxml SCRIPT_NAME : /index.php PHP_SELF : /index.php ============================================================
      Drop this plugin now for running PluXml and report to its author !!

    Dont acte ! :-)
    Bien à vous.
    Patrick

  • 1 avril modifié

    Bonjour,

    C'est un simple défaut du aux données exemples générées par défaut.
    Le mode d'enregistrement des mots clés à changer et ils sont systématiquement passés en minuscule. En réenregistrant ce "premier article" exemple, le bug disparait. Auparavant , les mot clé: pluxml et PluXml ou PLUXML étaient différents mots clés avec leur URLs respectives, maintenant ce n'est plus qu'un mot clé regroupé sous la seule url : pluxml .

    Cdt


    Cordialement,
    gcyrillus , simple membre du forum et utilisateur de pluxml

    Mon site PluXml: https://re7net.com | Plugins: https://ressources.pluxopolis.net/banque-plugins/index.php?all_versions | demos sur free http://gcyrillus.free.fr/new | Thèmes: tester et télécharger @ https://pluxthemes.com
    Indiquez [RESOLU] dans le titre de votre question une fois le soucis réglè, Merci

  • PatE2lPatE2l Member

    Bonjour,

    Merci pour ces informations, mais je pense que ce n'est pas la raison ici.
    Il s'agît de messages de PHP engendrés par le passage à PHP 8.
    Ces erreurs ne se produisent pas avec PHP 7.
    En voici une autre :
    Fatal error : type : 8192 message : Function utf8_decode() is deprecated file : /core/lib/class.plx.utils.php line : 843 See https://www.php.net/manual/en/errorfunc.constants.php about type of error
    ============================================================
    User : 001 Profil : 0 PluXml version : 5.8.11 PHP version : 8.2.7 ============================================================
    About this server : HTTP_USER_AGENT : Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0 HTTP_ACCEPT : text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,/;q=0.8 HTTP_ACCEPT_LANGUAGE : fr,fr-FR;q=0.8,en-US;q=0.5,en;q=0.3 HTTP_ACCEPT_ENCODING : gzip, deflate HTTP_REFERER : http://192.168.56.121/ SERVER_SIGNATURE :
    Apache/2.4.57 (Debian) Server at 192.168.56.121 Port 80
    SERVER_SOFTWARE : Apache/2.4.57 (Debian) SCRIPT_FILENAME : /index.php SERVER_PROTOCOL : HTTP/1.1 REQUEST_METHOD : GET QUERY_STRING : article2/municipales-en-turquie REQUEST_URI : /index.php?article2/municipales-en-turquie SCRIPT_NAME : /index.php PHP_SELF : /index.php ============================================================
    Drop this plugin now for running PluXml and report to its author !!

    Bien à vous.
    Patrick

  • 2 avril modifié

    C'est effectivement une autre erreur.
    Note que PluXml est annoncé compatible jusqu'à la version 8.1.2 pour le moment
    utf8_decode est obsolète depuis la 8.2.0 :(
    Voir https://www.php.net/manual/fr/function.utf8-decode.php Il y est proposé une alternative dans les notes si tu tiens vraiment à rester en php8. 2.x ou 8.3.4

    edit @PatE2l
    La dernière release est là 5.8.12 depuis hier.

    Je viens de la tester et de télecharger php 8.3.4 ( sous windows :anguished: ) et je n'ai aucune erreurs sur une installe toute neuve !?

    re-edit En fait oui, des qu'un commentaire dépasse 25 caractéres il est coupé pour être affiché dans la sidebar , la fonction strCut() provoque : Deprecated: Function utf8_decode() is deprecated


    Il faudrait aussi que tu indiques quels plugins tu utilises, si tu en utilises, car l'erreur que j'obtiens m'affiche la page et les messages d'erreur uniquement là ou devrait être afficher les commentaires. Je n'arrive pas à reproduire ton erreur.

    En modifiant la ligne 843 de class.plx.utils.php en
    return strlen($str) > $length ? mb_convert_encoding(substr(mb_convert_encoding($str, 'UTF-8', 'ISO-8859-1'), 0, $length), 'ISO-8859-1', 'UTF-8').$add_text : $str; je n'ai plus d'erreurs @bazooka07, @Haruka Pouvez vous confirmer que cela serait les modifs à effectuer ?
    J'ai suivi les recommendation sur php.net ... cela me parait verbeux .

    Cdt


    Cordialement,
    gcyrillus , simple membre du forum et utilisateur de pluxml

    Mon site PluXml: https://re7net.com | Plugins: https://ressources.pluxopolis.net/banque-plugins/index.php?all_versions | demos sur free http://gcyrillus.free.fr/new | Thèmes: tester et télécharger @ https://pluxthemes.com
    Indiquez [RESOLU] dans le titre de votre question une fois le soucis réglè, Merci

  • Bonjour, j'ai eu la même erreur et en demandant a Copilot(GPT) il m'a proposé de modifier la ligne 843 par :
    _ return strlen($str) > $length ? mb_convert_encoding(mb_substr($str, 0, $length), "ISO-8859-1", "UTF-8") . $add_text : $str;_
    C'est bon aussi ?

Connectez-vous ou Inscrivez-vous pour répondre.