Project

General

Profile

Anomalie #3765

Bug spip 3.1 urls arborescentes | Jointures

Added by Karen Bouscarle over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
-
Target version:
-
Start date:
04/02/2016
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Bonjour,

J'ai constaté que lorsqu'on sélectionne la réécriture des urls avec "urls arborescentes" (et seulement cette méthode-ci), on ne peut plus afficher ni les documents joints (quand ils ne sont pas intégrés dans le texte via un raccourci spip) ni les miniatures des images passées en portfolio.

Après l'avoir constaté sur un site en production, je l'ai testé en local et en distant sur un SPIP natif sans squelette, ni plugin, ni réglages particuliers autres que les nécessaires.

Je n'ai malheureusement pas les connaissances nécessaires pour contribuer, et c'est b_b qui a compris l'origine du problème, à savoir un problème avec la balise base href dans le head, puis m'a invitée à ouvrir un ticket ici. Ici notre échange : http://forum.spip.net/fr_263992.html#forum264143

Voilà, novice ici, j'espère que j'ai fait comme il fallait, pas osé monter la priorité au dessus de "haut", c'est déjà peut-être exagéré :-) ?
Merci à tous pour votre super et généreux boulot.

History

#1 Updated by b b over 3 years ago

  • Project changed from SPIP to Urls étendues

Je confirme sur le trunk à jour en r22976, j'attache le ticket au projet urls_etendues.

Comme je le disais sur le forum, le problème semble provenir du base href. En effet, le bug ne se présente pas en appelant l'inclure du portfolio dans un squelette de test vide avec [(#INCLURE{fond=inclure/documents,id_article=X,env})]. Et la désactivation du base href à l'aide du couteau kiss ne règle pas le problème.

< b_b‎ >  ha la loose
< b_b‎ >  on passe un id_rubrique dans l'env avec les arbos
< b_b‎ >  du coup l'inclure documents fait des jointures de fou
< b_b‎ >  pour chercher les docs liés à l'article et la rubrique :p

#2 Updated by b b over 3 years ago

  • Project changed from Urls étendues to SPIP
  • Subject changed from Bug spip 3.1 urls arborescentes to Bug spip 3.1 urls arborescentes | Jointures
  • Category set to code généré

Du coup, le problème est dans les jointures de la requête générée, exemple pour la boucle_documents_portfolio en 3.0 :

SELECT 0+documents.titre AS num, documents.date, documents.id_document, documents.titre, documents.fichier
FROM spip_documents AS `documents`  
INNER JOIN spip_documents_liens AS L1 ON ( L1.id_document = documents.id_document )
WHERE (documents.statut = 'publie')
    AND (documents.taille > 0 OR documents.distant='oui')
    AND (L1.id_objet = 7)
    AND (L1.objet = 'article')
    AND (documents.mode = 'document')
    AND ((documents.extension  IN ('png','jpg','gif')))
    AND (L1.vu = 'non')
ORDER BY num, documents.date

Et en 3.1 :

SELECT 0+documents.titre AS num, documents.date, documents.id_document, L1.id_objet AS id_article, documents.titre, documents.fichier
FROM spip_documents AS `documents`  
INNER JOIN spip_documents_liens AS L2 ON ( L2.id_document = documents.id_document ) 
INNER JOIN spip_documents_liens AS L1 ON ( L1.id_document = documents.id_document )
WHERE (documents.statut = 'publie')
    AND (documents.taille > 0 OR documents.distant='oui')
    AND (L1.id_objet = 7)
    AND (L1.objet = 'article')
    AND (L2.id_objet = 1)
    AND (L2.objet = 'rubrique')
    AND (documents.mode = 'document')
    AND ((documents.extension  IN ('png','jpg','gif')))
    AND (L1.vu = 'non')
GROUP BY documents.id_document
ORDER BY num, documents.date

Je repasse le ticket sur le core.

#3 Updated by b b over 3 years ago

Si on considère que le comportement de la boucle est le bon, le bug est dans la dist, et la solution est d'ajouter un simple id_rubrique='' à l'appel de inclure/documents.

#4 Updated by Karen Bouscarle over 3 years ago

Merci pour le boulot ! Je viens de tester et ça marche très bien pour l'appel des images en portfolio et des documents aussi :-)

#5 Updated by b b over 3 years ago

  • Project changed from SPIP to Squelettes Dist
  • Category deleted (code généré)

Je bascule le ticket sur la dist, avant d'envoyer les correctifs.

#6 Updated by b b over 3 years ago

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

Corrigé par http://zone.spip.org/trac/spip-zone/changeset/96474 dans le trunk et http://zone.spip.org/trac/spip-zone/changeset/96475 en 3.1. Merci pour le signalement, on peut fermer :)

Also available in: Atom PDF