Project

General

Profile

Anomalie #2963

Flux RSS des révisions

Added by Guillaume Fahrner over 6 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
Target version:
Start date:
03/23/2013
Due date:
04/15/2013
% Done:

0%

Estimated time:
2.00 h
Resolution:
fixed
Navigateur:

Description

Les URLS de type http://monsite.org/spip.php?page=rss&op=revisions&id=1&cle=MACLE ne semble pas fonctionner. Le flux XML retourné est correct mais ne contient aucun élément alors que la page http://monsite.org/ecrire/?exec=revisions montre bien des révisions existantes.

Version utilisé :
- SPIP 3.0.5 [19905]
- écran de sécurité 1.1.3

Associated revisions

Revision 71094 (diff)
Added by suske@brubel.net over 6 years ago

le rss des révisions passe en spip3 ^^. Il reste à y remettre la gestion des langues par contre :-/. Ferme #2963 quand même.

History

#1 Updated by b b over 6 years ago

Salut, le pb date depuis ce commit :

http://zone.spip.org/trac/spip-zone/changeset/31459

Depuis cette modif on peut utiliser les révisions sur tous les types d'objets, du coup il faut virer la jointure de la boucle sur spip_articles dans le squelette du flux :

http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/revisions/prive/rss/revisions.html#L15

Par contre, cela implique de virer le critère {lang==^#EVAL{"$_REQUEST['lang']"}|sinon{.*}} sans quoi la boucle génère une erreur, du coup on perd la possibilité de filtrer les révisions du flux par langue. Du coup, on perdrait une fonctionnalité pour en rétablir une autre, à moins que quelqu'un trouve une astuce pour régler le problème.

ps : j'oubliais de préciser que le critère statut pose aussi problème si on vire la jointure...

++

#2 Updated by b b over 6 years ago

Et hop, proposition de patch de Suske : http://spip.pastebin.fr/26838 => la même sans ajout de chaîne de langue : http://spip.pastebin.fr/26839

#3 Updated by Suske Suske over 6 years ago

http://zone.spip.org/trac/spip-zone/changeset/71094 améliore très sensiblement la chose. C'est http://spip.pastebin.fr/26839 avec mention de l'objet de son id et de la version dans le titre de l'item.

Quelque chose vaut mieux qu'un flux vide mais donc, par rapport à avant, pour ce qu'on m'en dit:

- la langue n'est pas prise en compte
- le statut non plus

#4 Updated by Guillaume Fahrner over 6 years ago

Un autre bug : La balise <rss> n'est pas fermée dans le flux XML généré, cela fait planter les parsers XML...

Un comportement inattendu : tout les liens du flux pointent vers /ecrire/ a cause d'une accolade de trop dans #URL_ECRIRE

Le code corrigé ici : http://spip.pastebin.fr/27022

#5 Updated by Guillaume Fahrner over 6 years ago

Un autre bug : le nom/email de l'auteur n'est pas affiché dans la balise dc:creator car pas dans le contexte de la boucle appelante.

La code corrigé ici utilise comme solution http://spip.pastebin.fr/27023 une boucle AUTEUR avec de se servir de #NOM et #EMAIL

#6 Updated by b b over 6 years ago

Un autre bug : La balise <rss> n'est pas fermée dans le flux XML généré, cela fait planter les parsers XML...

Voilà qui corrige la fermeture de balise rss manquante :

http://zone.spip.org/trac/spip-zone/changeset/72030

http://zone.spip.org/trac/spip-zone/changeset/72031

#7 Updated by b b over 6 years ago

Un comportement inattendu : tout les liens du flux pointent vers /ecrire/ a cause d'une accolade de trop dans #URL_ECRIRE

Je ne vois pas d'accolade en trop dans le code, et tous les liens pointent vers le privé car c'est dans le privé qu'on affiche les révisions (c'était déjà le cas en SPIP 2.1).

Donc soit il n'y a pas de problème de ce côté ou alors je n'ai pas compris ta remarque ?

#8 Updated by b b over 6 years ago

Un autre bug : le nom/email de l'auteur n'est pas affiché dans la balise dc:creator car pas dans le contexte de la boucle appelante.

Corrigé (ajout d'une jointure plutôt qu'une boucle supplémentaire) :

http://zone.spip.org/trac/spip-zone/changeset/72032

http://zone.spip.org/trac/spip-zone/changeset/72033

#9 Updated by b b over 6 years ago

Je crois que cette fois on doit pouvoir fermer ce ticket, non ? Reste peut être à éclaircir le point abordé ici : http://core.spip.org/issues/2963#note-7

#10 Updated by Suske Suske over 6 years ago

  • Status changed from Nouveau to Résolu
  • Resolution set to fixed

Pas d'éclaircissement, résolu/fixed.

#11 Updated by Suske Suske over 6 years ago

  • Project changed from SPIP to Révisions
  • Due date set to 04/15/2013
  • Category deleted (espace privé)
  • Assignee set to b b
  • Target version set to 3.0
  • Estimated time set to 2.00 h

plus d'infos pour mieux utiliser redmine ?

#12 Updated by denisb - over 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF