[SPIP Zone] retours skeleditor2

JLuc · Mars 28, 2010, 2:08

Testé sur une 2.1 j'ai trouvé 2 petits points d'améliorations possible :

- le message
  "Pour utiliser l'editeur de squelettes,
  creez un dossier squelettes/ à la racine de votre site."
pourrait utilement préciser :
  ", avec les droits d'écritures pour SPIP".

- les liens de "retours", en haut dans la colonne de gauche
d'une page d'édition, comme
"Retour spip.php?article1"
ou "Retour var_mode=inclure"
ne prennent pas en compte la configuration de l'adresse du site
et pointent donc vers la racine du site,
ce qui est inadéquat par exemple si le site est dans un sousrépertoire.

JLuc

JLuc · Mars 28, 2010, 8:38

Le 28/03/2010 16:08, JLuc a écrit :

Testé sur une 2.1
J'ai trouvé 2 petits points d'améliorations possible :

Et un 3ème à propos de la largeur de la zone d'édition :
par défaut la largeur de la colonne de contenu dans la partie privée
est trop petite pour être pratique pour l'édition de code.

Je dois dire aussi
que ç'a a fait une super avancée depuis la précédente version.

JL

- les liens de "retours", en haut dans la colonne de gauche
d'une page d'édition, comme
"Retour spip.php?article1"
ou "Retour var_mode=inclure"
ne prennent pas en compte la configuration de l'adresse du site
et pointent donc vers la racine du site,
ce qui est inadéquat par exemple si le site est dans un sousrépertoire.

> - le message
> "Pour utiliser l'editeur de squelettes,
> creez un dossier squelettes/ à la racine de votre site."
> pourrait utilement préciser :
> ", avec les droits d'écritures pour SPIP".

JLuc · Avril 2, 2010, 8:28

Le 28/03/2010 16:08, JLuc a écrit :

- les liens de "retours", en haut dans la colonne de gauche
d'une page d'édition, comme
"Retour spip.php?article1"
ou "Retour var_mode=inclure"
ne prennent pas en compte la configuration de l'adresse du site
et pointent donc vers la racine du site,
ce qui est inadéquat par exemple si le site est dans un sousrépertoire.

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu'il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement ../spip.php?page=recherche&var_mode=inclure

JLuc

JLuc · Avril 3, 2010, 9:43

Le 02/04/2010 22:28, JLuc a écrit :

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu'il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement ../spip.php?page=recherche&var_mode=inclure

mais pour ça, la ligne 29
de plugin/skeleditor/prive/exec/skeleditor.html
doit devenir :
[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

ça doit aussi être possible avec #URL_SITE_SPIP
JLuc

J-C · Avril 3, 2010, 10:07

JLuc a écrit :

Le 02/04/2010 22:28, JLuc a écrit :

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu'il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement ../spip.php?page=recherche&var_mode=inclure

mais pour ça, la ligne 29
de plugin/skeleditor/prive/exec/skeleditor.html
doit devenir :
[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

ça doit aussi être possible avec #URL_SITE_SPIP
JLuc

Impressionnant cet empilement ! }})]})]})]

denisb1 · Avril 3, 2010, 10:34

Le 03/04/10 12:07, Jean-Christophe Villeneuve a écrit :

JLuc a écrit :

[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

Impressionnant cet empilement ! }})]})]})]

oui.
on peut simplifier :
   [(#ENV{retour}|=={./}|?{
     #SET{retour, ../}
     ,
     #SET{retour, ../#ENV{retour, ''}}
   })]

JLuc · Avril 3, 2010, 12:09

Le 03/04/2010 12:34, denisb a écrit :

Le 03/04/10 12:07, Jean-Christophe Villeneuve a écrit :

JLuc a écrit :

[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

Impressionnant cet empilement ! }})]})]})]

oui.
on peut simplifier :
[(#ENV{retour}|=={./}|?{
#SET{retour, ../}
,
#SET{retour, ../#ENV{retour, ''}}
})]

Sur un autre point de ce même fichier, peut être saurez vous mieux que moi écrire :
#SET{file,[(#VAL{}|str_replace{”,[(#GET{file})]})]}
qui doit être glissé aprés la ligne 10

Cela nettoierait le surlignage qui casse le nom du fichier fourni par var_mode=inclure
(cf signalement sur spip-dev)

Mais je n'arrive pas à faire accepter ce projet de SET par le compilateur spip...
(il faut le faire une nouvelle fois ensuite avec )

En attendant, ceci marche :
#CACHE{0}
<?php
 $_GET['f']=str_replace('','',$_GET['f']);
 $_GET['f']=str_replace('','',$_GET['f']);
?>
 #SET{file,#EVAL{$_GET['f']}}

Mais je détecte des suites de caractères prohibées...

JLuc

JLuc · Avril 3, 2010, 1:20

Le 03/04/2010 14:09, JLuc a écrit :

Le 03/04/2010 12:34, denisb a écrit :

Le 03/04/10 12:07, Jean-Christophe Villeneuve a écrit :

JLuc a écrit :

[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

Impressionnant cet empilement ! }})]})]})]

oui.
on peut simplifier :
[(#ENV{retour}|=={./}|?{
#SET{retour, ../}
,
#SET{retour, ../#ENV{retour, ''}}
})]

Sur un autre point de ce même fichier, peut être saurez vous mieux que
moi écrire :
#SET{file,[(#VAL{}|str_replace{”,[(#GET{file})]})]}
qui doit être glissé aprés la ligne 10
Connexion · GitLab

Cela nettoierait le surlignage qui casse le nom du fichier fourni par
var_mode=inclure
(cf signalement sur spip-dev)

Mais je n'arrive pas à faire accepter ce projet de SET par le
compilateur spip...
(il faut le faire une nouvelle fois ensuite avec )

En attendant, ceci marche :
#CACHE{0}
<?php
$_GET['f']=str_replace('','',$_GET['f']);
$_GET['f']=str_replace('','',$_GET['f']);
?>
#SET{file,#EVAL{$_GET['f']}}

Mais je détecte des suites de caractères prohibées...

et de plus ça ne marche pas toujours.

Reste le premier patch !
Celui qui corrige le bug des liens de retour.
JLuc

denisb1 · Avril 3, 2010, 1:55

Le 03/04/10 15:20, JLuc a écrit :

et de plus ça ne marche pas toujours.

je ne comprends pas ton problème (celui dont tu parles sur .devel)
qu'y a t'il précisemment dans l'url ? dans le *champ adresse* de ton navigateur ?

il *ne peut pas* y avoir :
f=le_fichier

ça serait traduit par :
$_GET["f"]=>""
$_GET["lt;span_class"]=>""
$_GET["quot;spip_surligne"]=>""
$_GET["quot;"]=>""
$_GET["gt;le_fichier"]=>""
$_GET["lt;/span"]=>""
$_GET["gt;"]=>""

de plus, il vaut mieux corriger cette histoire de var_mode=inclure plus surlignage (que je n'ai toujours pas comprise...) qui te donne des liens cliquables (???)

bref : nettoyer en amont.

JLuc · Avril 3, 2010, 3:49

Le 03/04/2010 15:55, denisb a écrit :

Le 03/04/10 15:20, JLuc a écrit :

et de plus ça ne marche pas toujours.

je ne comprends pas ton problème (celui dont tu parles sur .devel)
qu'y a t'il précisemment dans l'url ? dans le *champ adresse* de ton
navigateur ?

Sur un zpip par exemple, avec skeleditor installé
si je cherche 'rech' au moyen du formulaire de recherche,
puis que je fais var_mode=inclure avec le bouton adhoc proposé en haut à droite,
et que je demande à éditer le squelette
squelettes-dist/formulaires/recherche.html
(carj'aimerais mettre une valeur par défaut pour le champ de recherche par ex)
l'édition n'a pas lieu car spip a surligné le terme recherché à savoir 'rech'
y compris dans le lien qui appelle skeleditor

il *ne peut pas* y avoir :
f=le_fichier

Le lien vaut texto :
ecrire/?exec=skeleditor&retour=spip.php?page=recherche&lang=fr&recherche=rech&f=squelettes-dist/formulaires/%3Cspan%20class=%22spip_surligne%22%3Erecherche%3C/span%3E.html

ça serait traduit par :
$_GET["f"]=>""
$_GET["lt;span_class"]=>""
$_GET["quot;spip_surligne"]=>""
$_GET["quot;"]=>""
$_GET["gt;le_fichier"]=>""
$_GET["lt;/span"]=>""
$_GET["gt;"]=>""

...

de plus, il vaut mieux corriger cette histoire de var_mode=inclure plus
surlignage (que je n'ai toujours pas comprise...) qui te donne des liens
cliquables (???)
bref : nettoyer en amont.

Toutafé.

JLuc

JLuc · Avril 13, 2010, 8:19

Le 03/04/2010 11:43, JLuc a écrit :

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu'il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement ../spip.php?page=recherche&var_mode=inclure

mais pour ça, la ligne 29
de plugin/skeleditor/prive/exec/skeleditor.html
doit devenir :
[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

Quelqu'un avec les droits pourrait-il corriger le plugin
avec ce patch ou sa version simplifiée ?

JLuc

cerdic · Avril 13, 2010, 8:40

Le 13 avr. 2010 à 10:19, JLuc a écrit :

Le 03/04/2010 11:43, JLuc a écrit :

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu'il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement ../spip.php?page=recherche&var_mode=inclure

mais pour ça, la ligne 29
de plugin/skeleditor/prive/exec/skeleditor.html
doit devenir :
[(#ENV{retour}|=={'./'}|?{[(#SET{retour,../})],[(#SET{retour,[(#VAL{../}|concat{#ENV{retour,''}})]})]})]

Quelqu'un avec les droits

les droits sont à tout le monde, suffit de les demander

Cédric

eric_tonton · Avril 13, 2010, 9:28

J’essayerais de le faire ce soir si je rentre tot sinon.

++
Eric

Le 13 avril 2010 10:40, cedric.morin@yterium.com <cedric.morin@yterium.com> a écrit :

Le 13 avr. 2010 à 10:19, JLuc a écrit :

Le 03/04/2010 11:43, JLuc a écrit :

Le lien de .retour vaut par exemple
/spip.php?page=recherche&var_mode=inclure
alors qu’il devrait être
#URL_SITE_SPIP/spip.php?page=recherche&var_mode=inclure
ou éventuellement …/spip.php?page=recherche&var_mode=inclure

mais pour ça, la ligne 29
de plugin/skeleditor/prive/exec/skeleditor.html
doit devenir :
[(#ENV{retour}|=={‹ ./ ›}|?{[(#SET{retour,…/})],[(#SET{retour,[(#VAL{…/}|concat{#ENV{retour,‹ ›}})]})]})]

Quelqu’un avec les droits

les droits sont à tout le monde, suffit de les demander

Cédric

spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone