Désactiver les flux rss

[pluxml]

Ajouter la possibilité de désactiver (temporairement) les flux rss à partir d'une case à cocher pour éviter d'avoir à renommer le fichier feed.php

[Philippe-M]

Un petit PR : #335

[haruka-7]

Merci beaucoup pour la PR !

Dans le commit ca35acc , je propose néanmoins d'inverser le paramètre pour le changer de "Désactiver l'affichage des flux RSS" à "Afficher les flux RSS", car pour tous les paramètres disponibles dans l'admin c'est géré dans ce sens.

J'ai retiré le test fait sur la vue (sauf pour footer.php) pour le réaliser dans le modèle (class.plx.show.php). Ça permet de faire fonctionner cette nouvelle option sur tous les thèmes existant sans modification, mais ce n'est pas parfait :

• ça ne prend pas en compte le footer (le test est sur la vue)
• lorsqu'on désactive l'affichage des flux RSS, on aura des balises HTML vides (<span></span>, <p></p>).

Qu'en penses-tu ?

[Philippe-M]

Je n'avais pas vu ton commit, la prochaine fois je regarderais avant de me lancer dans le code :)

Lorsque j'ai codé ça je n'ai pas trop prêter attention à la logique côté admin mais ton choix est plus pertinant et ne perdra pas l'utilisateur. En fait je suis d'accord avec toi pour tout sauf pour le dernier point : les balises html vides.

Pour moi cela créé 3 problèmes :

• alourdi (de façon négligeable) la quantité de code et du coups le poids de la partie html transféré au visiteur;
• complexifie le débogage pour un intégrateur/créateur de thème (trouver le pourquoi il y a ces balises vides);
• sans être expert, cela doit avoir un impact sur l'accessibilité.

Je vois pas trop comment gérer ça avec la séparation de la vue et du code php :(

[jerrywham]

Bonjour,
Je trouve dommage de diminuer encore plus la lisibilité des flux rss en les désactivant par défaut.
Beaucoup ne les ré-activeront pas par méconnaissance.
On se prive d'un bel outil au profit à nouveau des GAFAM.

[sudwebdesign]

Il suffit juste qu'ils soient actifs a l'installation et lors de la future MAJ pour qu'il soit dispo par défaut.
[édit] ça c'est fait ;)

[sudwebdesign]

@P3TER : Petit truc en aparté, règle l'indentation de ton éditeur actuellement il semble qu'il utilise les espaces

J'affiche aussi les caractère non imprimable (et parfois c'est l'horreur), il efface aussi les espaces de fin de ligne.
ps: en général remplacer 4 espaces par une tabulation ds tous le doc, parfois la session avec geany ;) rétabli la situation.

[haruka-7]

Je trouve dommage de diminuer encore plus la lisibilité des flux rss en les désactivant par défaut.

@jerrywham, pas de panique, ils sont affichés par défaut à l'installation ou à la mise à jour ;-)

@Philippe-M, le commit que j'ai mis en lien, je l'ai réalisé à partir de ta branche, j'ai repris ton code et fais mes modifications dessus. Pour les balises HTML vides, je ne vois pas d'autres solutions que d'ajouter aux fonctions artFeed, comFeed et tagFeed, une variable $format en paramètre comme pour les autres fonctions de $plxShow. Voir commit de81e25

@sudwebdesign, j'ai pris note de ta remarque, merci. Effectivement j'ai quelques soucis de formatage, mais ça devrait aller mieux pour les prochaines fois.

[sudwebdesign]

@P3TER +1 pour les $format & reste compatible avec les anciens thèmes :)

[Philippe-M]

@P3TER : nikel la variable $format. Je pensais faire des tests lundi en arrivant au boulot avec cette idée et je vois que tu m'a devancé ;)

[haruka-7]

J'ai mergé tout ça dans la branche develop (PluXml 5.8).
Merci à tous !