[plugin] MyTeam

DjbWebmaster

mais il fonction sous la version 5.5 de pluxml sans le popup des medias

Il faut juste placer le lien de l'image

jol5926

DjbWebmaster a écrit:

mais il fonction sous la version 5.5 de pluxml sans le popup des medias

Il faut juste placer le lien de l'image

OK,
alors pourquoi j'ai cette erreur :

Dans la partie "admin" du plugin quand je modifie un membre pour le mettre à jour, je clique sur "mettre à jour", la mise à jour se fait mais j'ai ce message en haut de la page.

DjbWebmaster

Tu dois avoir un probleme dans ton fichier MyTeam.xml (/data/configuration/plugins/) lors de la mise à jour.

supprimer ce fichier et le plugin, puis réaliser une nouvelle installation.

DjbWebmaster

Mise à jour

cyrille

je confirme qu'il fonctionne en 5.5.

Par contre la correction des caractères spéciaux, l'édition des profils ?

cyrille

le plugin dans sa dernière version est planté. Pas d'affichage dans le menu des users, les boutons ne donnent rien. Ca vient du fait que je suis en https et qu'il y a un appel à cloudfare bloquant / bloqué en http.

cyrille

en corrigeant le config et en mettant https sur l'url ça passe

DjbWebmaster

ok cyrille, merci pour le retour

DjbWebmaster

Cyrille,

Tu peux tester la dernière version avec la possibilité de modifier l'avatar dans l'onglet gestion des membres,
il suffit de clic sur l'avatar et de sauvegarder.

cyrille

je vais faire.

cyrille

j'ai pris le dernier github et j'ai collé franco

- le bug du https pas corrigé
- encodage des caractères spéciaux toujours pas
- problème dans l'affichage de l'avatar depuis le menu quand on fait naviguer, j'ai l'image cassée avec chrome

DjbWebmaster

cyrille a écrit:

- le bug du https pas corrigé

Oui j'ai pas tester en https (as tu une piste)

cyrille a écrit:

- encodage des caractères spéciaux toujours pas

Je viens de modifier:

[== PHP ==]
'name'      => '".$this->getParam('mnuName')."',

devient

[== PHP ==]
'name'      => '".addslashes($this->getParam('mnuName'))."',

cyrille a écrit:

- problème dans l'affichage de l'avatar depuis le menu quand on fait naviguer, j'ai l'image cassée avec chrome

oui sur la partie admin du plugin, je force l'affichage de l'image en 64px par 64px

cyrille

- pour le https comme je te le mentionnais, il suffit d'ajouter le "s" dans ton lien vers cloudfare
- il faudrait pas que le titre de la page
- pour les gens qui n'ont que le pseudo, je noterai en intitulé nom ou pseudo et laisserait le prénom en facultatif.
- pour l'url de l'image je pense que tu mets deux fois le nom de domaine, j'obtiens si j'ouvre l'image : https://cyrille-borne.com/https://cyrille-borne.com/data/images/avatar/cep.jpg cela vient qu'à la création j'ai passé l'url complète de l'image

DjbWebmaster

Pour les gens qui n'ont que le pseudo, je noterai en intitulé nom ou pseudo et laisserait le prénom en facultatif.

Il suffit de supprimer cette condition (ligne 24, fichier config.php)

[== PHP ==]
if (!empty($_POST['nom-new']) AND !empty($_POST['prenom-new']) AND !empty($_POST['fonction-new']))  {
        # création d'un nouveau membre

cette ligne vérifie si les champs: nom, prénom et fonction sont bien renseignés

- pour l'url de l'image je pense que tu mets deux fois le nom de domaine, j'obtiens si j'ouvre l'image : https://cyrille-borne.com/https://cyril … ar/cep.jpg cela vient qu'à la création j'ai passé l'url complète de l'image

lors de la création il ne faut par rentrer l'url complète, mais juste data/images/avatar/avatar4.png

cyrille

pour la seconde partie j'aurai tendance à dire que c'est à toi de faire le contrôle d'intégrité sur l'url de l'image.

DjbWebmaster

si on veut utiliser la nouvelle fenêtre popup pour sélectionner les images, j'ai placé ce code

[== Indéfini ==]
<a id="toggler_thumbnail" href="javascript:void(0)" onclick="mediasManager.openPopup('avatar<?php echo $i; ?>', true)">
<img src="<?php echo  PLX_ROOT.$plxPlugin->getParam(avatar.$i) ?>" alt="avatar" height="64" width="64">
</a>

Si je supprime PLX_ROOT, on peut mettre l'adresse https://cyrille-borne.com/data/images/avatar/cep.jpg
mais après c'est impossible d'utiliser les images de la fenêtre du popup.

[== Indéfini ==]
<?php echo  PLX_ROOT.$plxPlugin->getParam(avatar.$i) ?>

Je n'ai pas de solution pour le moment, avis à la communauté

cyrille

en fait le plus simple, c'est au moment de l'insertion de l'avatar, masquer l'url et mettre en évidence un gros +, de façon à se sentir obligé d'aller chercher l'avatar dans la data.

Est ce qu'on peut passer dans les champs libres des urls ?

Andrea

Ce plugin semble encore fonctionner même si le fichier de configuration n'est pas créé via l'administration (on peut l'ajouter nous-même en utilisant le FTP)

Suite à un autre message du forum à propos du plugin FAQ j'ai ajouté des ' ainsi dans config.php pour éliminer des notices php

$plxPlugin->getParam(nom.$i);
$plxPlugin->getParam('nom'.$i);

PS: testé avec PluXml 5.8.18 et PHP 8.2

Andrea

Pour ce qui est de la création du fichier de configuration MyTeam.xml

En suivant une récente modification de @gcyrillus-nomade avec le plugin Testimonials (également de @DjbWebmaster)

Remplacer cette portion de code (au début du fichier config.php)

$nbmembres = floor(sizeof($plxPlugin->getParams())/6);

par

$nbmembres='0';
if (is_array($plxPlugin->getParams())) $nbmembres = floor(count($plxPlugin->getParams())/6);

Andrea

Par contre ce qui m'échappe c'est que le 1ier membre qu'on ajoute a l'id n°2 (ce qui n'est pas bien grave mais je serais curieux de savoir pourquoi ?)

J'ai essayé de comparer avec les autres plugins de Djb qui n'ont pas ce soucis mais je n'ai vu aucune différence!

Tous ont +/- cette portion de code alors ça ne doit pas se situer ici

                        <tbody>
                            <?php for($i=1; $i<=$nbmembres; $i++) {?>

                            <?php $nom = $plxPlugin->getParam('nom'.$i);

                            if(!empty($nom)) { ?>

                            <tr class="line-<?php echo $i%2 ?>">
                                <td class="id">
                                    <?php echo $i; ?>
                                </td>

gcyrillus-nomade

Bonsoir @Andrea

Par contre ce qui m'échappe c'est que le 1ier membre qu'on ajoute a l'id n°2 (ce qui n'est pas bien grave mais je serais curieux de savoir pourquoi ?

Peut-être as tu effacé un membre durant tes tests, l'enregistrement est vide mais encore là j'imagine

J'ai fait une mise à jour à l'attention de tous : https://ressources.pluxopolis.net/banque-plugins/plugins/MyTeam.1.2.zip
Correspond- elle à ce que tu as fait ? est-elle fonctionnelle pour toi ?

Andrea

En théorie pour ce qui est de cet ID qui commence à 2 j'avais fait plusieurs tests mais je vais vérifier ça @gcyrillus-nomade avec votre version

gcyrillus-nomade

En théorie pour ce qui est de cet ID qui commence à 2 j'avais fait plusieurs tests mais je vais vérifier ça @gcyrillus-nomade avec votre version

Il faut regarder dans le fichier de configuration du plugin pour voir quels et combien de champs il contient pour vérifier comment le compte est fait .On peut ensuite savoir si il y a un bug ou si c'est du à la conception.

Andrea

Côté HTML des balises ne sont pas fermées (fichier config.php) un paragraphe ici

                <p>
                    <label for="mnuDisplay">Afficher la page dans la navigation</label>
                    <select name="mnuDisplay" id="mnuDisplay">
                        <option value="1"  <?php if ($mnuDisplay == '1') { echo'selected';}?> >Oui</option>
                        <option value="0" <?php if ($mnuDisplay == '0') { echo'selected';}?> >Non</option>
                    </select>

Et aussi au niveau de ce div là

<!-- contenu de la page configuration -->
<div class="tabby-content" id="example-tab-content">

PS: pas remarqué de soucis dans l'affichage du navigateur mais voilà

gcyrillus-nomade

@Andrea archive mise à jour. Merci de tes retours https://ressources.pluxopolis.net/banque-plugins/plugins/MyTeam.1.2.zip

Andrea

Question HTML @gcyrillus-nomade le paragraphe a bien été fermé mais pas le div apparemment...

Andrea

En tout cas merci @gcyrillus-nomade pour l'implantation de la traduction c'était sur ma liste des choses à faire mais vous avez été plus rapide

Andrea

Pour terminer avec le HTML j'avais supprimé le paragraphe autour de ce titre (de mémoire ça ne se fait pas en HTML 5)

<p>
    <h2><?php echo $plxPlugin->getInfo("description") ?></h2>
</p>

PS: comme vous avez indiqué dans le fichier infos.xml de ce plugin la compatibilité avec PHP 8 j'avais en fait utilisé autre chose que la description

Andrea

Pour revenir à la traduction @gcyrillus-nomade une question de syntaxe

Par exemple

<li><a data-target="tab1" class="active" href="#"><?php $plxPlugin->lang('L_NAV_LIEN1'); ?></a></li>

Parfois vous avez utilisé une balise courte echo

<th><?= $plxPlugin->lang('L_ID'); ?></th>

Mais vu dans le fichier core/admin/comments.php de PluXml

<th class="datetime"><?= L_COMMENTS_LIST_DATE ?></th>

alors si on utilise cette balise courte echo on peut se dispenser d'écrire $plxPlugin->lang('...') non ? pas dans le cadre d'un plugin ?

gcyrillus-nomade

Pour la syntaxe des traductions en echo, tu peut effectivement faire directement <?= L_UNE_CONSTANTE_DU_FICHIER_LANG ?> , La difference entre les deux façon de faire:

1. <?php $plxPlugin->lang('L_NAV_LIEN1'); ?> Si cette constante n'existe pas, alors s'affiche L_NAV_LIEN1
2. <?= L_NAV_LIEN1 ?> Si cette constante n'existe pas, alors tu as un beau message d'erreur de PHP qui casse tout . J'utilise géneralement cette syntaxe qu'avec les traductions natives de PluXml , il n'y a à priori pas de raisons quelles soit manquantes et permet de differencier dans le code l'origine des traductions.

cdt