[pierredu]

Pour les extensions qui offrent cette série de 4 boutons (montée, descente, édition, suppression), on pourrait imaginer une demande de confirmation, mais c'est un peu lourd pour quelque chose qui ne le mérite pas vraiment. Dans les essais que j'ai fait, j'ai bien aimé la rapidité. Mais on peut préférer la sécurité.
Il faut que je réfléchisse (et que je regarde comment d'autres composants du panneau d'administration sont codés).

[Manard]

Bonjour Pierre,

Je viens de faire un essai de suppression de BBcode que j'ai créé, il y a bien cette fenêtre



Tu indiques que c'est rapide, effectivement j'ai fait le même constat, mais l'action de suppression n'est pas réalisée dans mes essais

Bon développement

Bernard

[pierredu]

Je viens de télécharger une nouvelle version (1.0.6) qui ajoute une demande de confirmation dans le cas d'une suppression. Pour la mise à jour, désactiver l'extension, installer les nouveaux fichiers et revalider.

[Manard]

Bonjour Pierre,

Tu approches du but, je peux maintenant supprimer avec la fenêtre de confirmation et éditer les liens déclarés. Seul petit hic, s'affichent deux fenêtres identiques en cliquant sur "Configuration de l'extension"



Bonne fin d'après-midi

Bernard

[pierredu]

Ce n'est pas clair qu'il y a la série avant et la série après le lien vers la FAQ ?
Il faudrait avoir une seule liste avec un paramètre avant/après ?
Mais dans ce cas, le reclassement serait difficile à concevoir, non ?

[Manard]

Bonsoir Pierre,

La différenciation "avant" et "après" m'a échappée , j'avais fait la même lecture mea culpa

Je viens de tester l'installation "après" c'est bon

Merci et mes excuses pour cette mauvaise lecture

Bernard

[Skouat]

Hello,

@pierredu: N'utilise pas ton 1er message comme journal de bord.
Merci de le présenter comme indiqué dans le ce message

Concernant la suppression, l'ajout et le déplacement d'éléments, il faut que toutes les actions effectuées par les boutons se prémunissent de la vulnérabilité CSRF.
A lire : https://blog.phpbb.com/2009/01/14/fighting-csrf/

Ce qui signifie que les actions "delete" doivent être traitées via confirm_box() (c'est le cas).
Ex: https://github.com/Skouat/ext_paypal_do ... #L188-L203

Que les actions move_up et move_down doivent être traitées via generate_link_hash().
Ex:
https://github.com/Skouat/ext_paypal_do ... #L263-L266
+
https://github.com/Skouat/ext_paypal_do ... hp#L89-L90

Et les ajouts/modification, via le check_form_key() et add_form_key()

_______________

T'as trop de duplication de code.

Par exemple :
-> https://github.com/pierrdu/lmdi_multili ... p#L83-L104
et
-> https://github.com/pierrdu/lmdi_multili ... #L202-L223

la seule différence c'est lmdi_multilinks_pp et lmdi_multilinks_ap.
Trouve une astuce pour que ap et pp soient variabilisés afin de n'avoir plus qu'un seul case: "delete".

Idem pour les autres case:
là où tu vas créer un case différent pour move_up, move_down (4 fois le même code), je n'ai qu'une seule méthode qui gère le up et le down
Il faut vraiment optimiser à ce niveau, car tu as beaucoup de code en doublon.

Dans les notions d'algorithmique que l'on m'a enseigné il y avait celle de :
Si deux portions de code sont identiques, alors il faut les extraires pour n'en avoir qu'une seule. (c'est la version courte )

Tu devrais réellement songer à tester Scrutinizer, même s'il ne faut pas l'écouter tout le temps, à mon avis il y a quand même pas mal de conseils qui peuvent servir. C'est simple a mettre en place.

A+

[pierredu]

Je suis assez d'accord, dans le principe, avec ce que tu dis.
C'est ce que je ferais dans du code dont je voudrais être fier.
Mais là, on est juste dans un truc jeté rapidement sur le papier pour répondre aux besoins d'utilisateurs (qui ne savent pas coder) qui veulent rajouter leurs liens à la barre de navigation.

[pierredu]

Bonsoir Pierre,
La différenciation "avant" et "après" m'a échappée , j'avais fait la même lecture mea culpa
Je viens de tester l'installation "après" c'est bon
Merci et mes excuses pour cette mauvaise lecture
Bernard

Même pas grave.
Je me suis demandé s'il ne faudrait pas mettre une explication générale au-dessus et au-dessous le code de création des lignes. J'ai peur que cela perde en clarté. Les onglets du panneau d'administration sont très répétitifs et je ne pense pas que l'on devrait rajouter un nouveau mode d'interaction.

[pierredu]

Je ne m'étais pas remis au travail ces derniers temps.

J'ai mis en facteur les deux grands paquets de code, j'ai rajouté la protection par clef de hachage pour les liens de montée et de descente et j'ai ajouté une option de sélection d'icônes (fontawesome sous 3.2.x, icônes anciennes sous 3.1.x).

Une nouvelle version a été installée sur github.

[Skouat]

Hello,

Petite relecture rapide…
Que se passe-t'il dans ton code si $ppap n'est jamais initialisé (donc définit à '') ?
cf : https://github.com/pierrdu/lmdi_multili ... le.php#L29

Concernant la ligne 28, c'est $submit = $request->is_set_post('submit'); qu'il faut utiliser et non ce que tu as mis.

Ligne 118, la variable $bbcodeid n'est pas définie.

Tu peux encore plus factoriser ton code, d'autant plus que tu peux t'appuyer sur la variable $ppap qui te permettrais de faire sauter un des for
https://github.com/pierrdu/lmdi_multili ... #L228-L241
https://github.com/pierrdu/lmdi_multili ... #L246-L259

ce break peut être viré si tu penses mieux ton code (ce qui pemet là aussi d'optimiser le code.

Ligne 270, ligne vide => rien à faire ici.

_______________

Dans ton fichier html lié à l'ACP.
tes input et textarea n'ont aucun ID de label de déclaré, alors qu'au niveau des label tu les initialise
Ex :

Code: Tout sélectionner

			<dt><label for="ml_anchor">{L_ACP_MULTILINK_ANCHOR}{L_COLON}</label><br />{L_ACP_ML_ANCHOR_EXPLAIN}</dt>
			<dd><input type="text" name="ml_anchor" value="{ANCHOR}" /></dd>

A remplacer par

Code: Tout sélectionner

			<dt><label for="ml_anchor">{L_ACP_MULTILINK_ANCHOR}{L_COLON}</label><br />{L_ACP_ML_ANCHOR_EXPLAIN}</dt>
			<dd><input type="text" id="ml_anchor" name="ml_anchor" value="{ANCHOR}" /></dd>

[pierredu]

Merci pour toutes ces remarques.

[pierredu]

À la suite de la demande de testeurs sur phpbb.com, j'ai largement remanié l'extension pour permettre de gérer des fichiers graphiques en plus des icônes. Ça marche bien.
Le dépôt sur github a été mis à jour.

[pierredu]

Une nouvelle version a été téléchargée sur github.

Principaux changements :
Gestion correcte de la taille et de l'alignement des icônes et des fichiers (merci à Cabot)
Élargissement des zones de saisie des textes d'ancrage et des URL
Fonction de transfert d'une rubrique d'une table à une autre (avant ou après le lien vers les FAQ). La plus grosse difficulté pour moi a été de pouvoir accéder à une icône placée dans le dossier adm/style à partir du module de gestion de l'onglet PCA.

Voilà.
Merci d'avance pour vos commentaires.

[Mazeltof]

'soir ici,

pas moyen d'enregistrer un lien, quel qu'il soit, sur cette dernière version