Project

General

Profile

Anomalie #3210

flux rss sur les forums

Added by jluc - almost 7 years ago. Updated almost 7 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
espace privé
Target version:
Start date:
05/05/2014
Due date:
% Done:

0%

Resolution:
invalid
Navigateur:

Description

SPIP 2.1.26

Dans les fichiers du répertoire prive/rss il y a dans le HTML des paramètres type= qui ne correspondent à rien
et devraient être remplacés par des statut=

Il y a aussi des valeurs de paramètres #ENV{page} qui ne correspondent à rien et devraient être remplacés par #ENV{statut}

Dans forums_public, qui devrait renseigner sur les forums publics, il y a le critère en dur : {statut IN publie,prop,off,spam}
il faudrait plutôt {statut=publie}
et plus loin il y a #URL_ECRIRE{controle_forum,statut=prop...} il faudrait plutôt statut=publie

J'ai aussi une interrogation pour prive/rss.html : on y trouve #INCLURE*{fond=prive/rss/(#ENV{op}|match{'^\w+$'}
ce qui fait que pour surcharger il faut créer un sous répertoire privé dans le répertoire squelette.
Est ce bien là ce qu'il faut faire habituellement ? Ne devrait ce pas plutôt être #INCLURE*{fond=rss/(#ENV{op}|match{'^\w+$'} ?

History

#1 Updated by Suske Suske almost 7 years ago

Salut,

Tu as localisé et testé les modifs que tu préconises ? Je veux bien tester un patch mais me réinstaller un 2.1 pour chercher le bug :-(

Par ailleurs, je crois qu'il est urgent qu'on redirige les notifs de tickets ici vers SPIP-dev... Mais ça c'est à moitié hs, ok.

:-*

Suske

#2 Updated by Suske Suske almost 7 years ago

  • Category changed from divers to espace privé
  • Target version set to 2.1

#3 Updated by jluc - almost 7 years ago

Non je n'ai pas corrigé.
À moins qu'il y ait des fonctionnalités mystérieuses implémentées par des hacks ninja, tous les fichiers générant les rss sont à la ramasse en ce qui concerne ces critères. Vu le niveau de foirade pressenti, il est probable que personne ne se sert des fonctionnalités proposées, sinon le bug aurait été remonté. En conséquence, le plus simple serait de les virer tous sauf un, le plus basique, et encore, en enlevant les potentielles fonctionnalités avancées foireuses.
Et par contre il faudrait vérifier si ç'a été mystérieusement corrigé dans SPIP3...

#4 Updated by esj - almost 7 years ago

jluc - a écrit :

SPIP 2.1.26

Dans les fichiers du répertoire prive/rss il y a dans le HTML des paramètres type= qui ne correspondent à rien

Pas du tout. Ce paramètre est utilisé pour produire l'URL correcte vers exec/controle_forum. Faut éviter d'accuser un code d'être "à la ramasse" sans pouvoir donner un bug précis sous peine de voire son propos être accusé de la même chose.

#5 Updated by b b almost 7 years ago

Hop, du coup, quel est l'objet précis du ticket ? Quel est le bug observé et le comportement attendu ? S'il n'y a pas de bug, on peut fermer le ticket non ?

#6 Updated by jluc - almost 7 years ago

ESJ : Désolé si mon vocabulaire à l'emporte pièce a heurté ta sensibilité. Puissent tes futures expressions également épargner les internautes !

b_b : Je ne suis plus là dessus, mais apparamment j'avais confondu "public" et "publie" et je n'arrivais pas à paramétrer ces squelettes comme il était censément possible de le faire. Peut être n'y a t il aucun des premiers problèmes évoqués.

Reste le dernier point : pour inclure un squelette de 'prive', il ne faut habituellement pas inclure 'prive/le_squel' (comme il est fait via #INCLURE*{fond=prive/rss/#etc} dans 'prive/rss.html') mais directement 'le_squel', non ? Du coup j'ai été obligé (pour surcharger les squelettes) de créer un sous-répertoire 'prive/rss/' dans le dossier 'squelettes/', au lieu de seulement créer un sous-répertoire 'rss/'.
cf http://core.spip.org/projects/spip/repository/entry/branches/spip-2.1/prive/rss.html

#7 Updated by jluc - almost 7 years ago

Si j'en crois http://zone.spip.org/trac/spip-zone/changeset/83493 le dernier point est normal.
C'était il y a longtemps donc je ne suis plus certain des premiers mais peut être bien qu'il faut fermer ce signalement.

#8 Updated by b b almost 7 years ago

  • Status changed from Nouveau to Fermé
  • Resolution set to invalid

Ok, on ferme pour l'instant, et tu pourras revenir sur le ticket si tu penses qu'il était justifié.

Also available in: Atom PDF