Hello,
à propos de bug, je n'ai pas réussi à changer la vignette d'un document
associé à un article, sur la V209.
Charger document ok
Associer vignette ok
Supprimer vignette ne fait rien !
Obligé de supprimer le document, le recharger puis lui associer une nouvelle vignette.
JLuc
C'est
http://trac.rezo.net/trac/spip/changeset/12472/spip/ecrire/action/documenter.php
qui a modifié la regexp qui depuis n'accepte plus le cas prévu ligne 223 de ecrire/inc/legender.php.
Je n'ai pas suivi le changement d'interface introduit par 12472, d'autant que
http://trac.rezo.net/trac/spip/changeset/13385/spip/ecrire/action/documenter.php
aggrave encore plus le bug en changeant le nombre de parenthèses dans la Regexp sans reporter la modif
dans les index du tableau fourni par preg_match.
Bref, je passe la main.
Committo,Ergo:Sum
JLuc a écrit :
à propos de bug, je n'ai pas réussi à changer la vignette d'un document
associé à un article, sur la V209.
il me semble que si on ne peut supprimer
la vignette depuis :
?exec=articles_edit&id_article=xx
on peut le faire depuis :
?exec=articles&id_article=xx
mais oui : il y a bien un bug
http://trac.rezo.net/trac/spip/changeset/13385/spip/ecrire/action/documenter.php
aggrave encore plus le bug en changeant le nombre de parenthèses dans la Regexp sans reporter la modif
dans les index du tableau fourni par preg_match.
lecture un peu rapide puisque
http://trac.rezo.net/trac/spip/changeset/13385/spip/ecrire/action/documenter.php
modifie la regexp ET la ligne 58 qui en utilise le résultat, en décalant la liste d'une valeur à la fin avec une virgule supplémentaire.
Mais, oui, tout ce code est peu lisible et très difficile à maintenir car les formats des appels sont dispersés dans tout ecrire/
Pour info, j'ai recodé tout ce qui concerne les documents dans le plugin gestion_documents en supprimant notamment toutes ces actions qui font plein de choses à la fois au profit d'actions plus simples et clairement nommées.
Cédric
lecture un peu rapide puisque
http://trac.rezo.net/trac/spip/changeset/13385/spip/ecrire/action/documenter.php
modifie la regexp ET la ligne 58 qui en utilise le résultat, en décalant la liste d'une valeur à la fin avec une virgule supplémentaire.
Ah oui, pas les yeux en face des trous.
Mais, oui, tout ce code est peu lisible et très difficile à maintenir car les formats des appels sont dispersés dans tout ecrire/
Pour info, j'ai recodé tout ce qui concerne les documents dans le plugin gestion_documents en supprimant notamment toutes ces actions qui font plein de choses à la fois au profit d'actions plus simples et clairement nommées.
Il faudrait envisager de l'intégrer dans le noyau alors, parce que ça fait deux codes à maintenir et on en voit pas la raison.
Committo,Ergo:Sum
C’est bien le but, pour la 2.1.
J’avais commencé à sortir tout le code des documents dans un plugin, pour ensuite remplacer, mais ce chantier est en suspens faute de temps. D’autre part, l’interface refondue n’est pas encore aussi fluide et ergonomique qu’il faudrait.
Cédric