[spip-dev] rc1: barre_raccourci ne marche plus si il y a <docXX>

Bonsoir

La chose a déjà été signalée (http://article.gmane.org/gmane.comp.web.spip.devel/51034/match=barre+des+raccourcis)
FF 3.0.3, debian/unstable (vu aussi sur du FF ?? avec xp, je ne peux tester qu'au boulot, c'est compliqué)

Avec une mise à jour svn d'il y à 2 minutes firebug dit (avec des plugins eux aussi à jour):
barre_raccourci is not defined
MajStats is not defined
swap_couche is not defined

pareil sur une install toute vierge, pour la première réponse.

J'ai raté quel rtfm ?

* Frédéric Moinard tapuscrivait, le 29/11/2008 17:45:

Bonsoir

La chose a déjà été signalée (http://article.gmane.org/gmane.comp.web.spip.devel/51034/match=barre+des+raccourcis)

FF 3.0.3, debian/unstable (vu aussi sur du FF ?? avec xp, je ne peux tester qu'au boulot, c'est compliqué)

Avec une mise à jour svn d'il y à 2 minutes firebug dit (avec des plugins eux aussi à jour):
barre_raccourci is not defined
MajStats is not defined
swap_couche is not defined

ça, c'est pas la Barre typo de SPIP, c'est la Barre Typo V2 ==> c'est une question à poser sur la zone.

Bonjour,

RealET a écrit :

* Frédéric Moinard tapuscrivait, le 29/11/2008 17:45:

Bonsoir

La chose a déjà été signalée (http://article.gmane.org/gmane.comp.web.spip.devel/51034/match=barre+des+raccourcis)

FF 3.0.3, debian/unstable (vu aussi sur du FF ?? avec xp, je ne peux tester qu'au boulot, c'est compliqué)

Avec une mise à jour svn d'il y à 2 minutes firebug dit (avec des plugins eux aussi à jour):
barre_raccourci is not defined
MajStats is not defined
swap_couche is not defined

ça, c'est pas la Barre typo de SPIP, c'est la Barre Typo V2 ==> c'est une question à poser sur la zone.

Je reformule:
- installation neuve (branches/spip-2.0 [13300]) sur base vide
- une modif dans la config "autoriser ajout document articles"
- création d'une rubrique et d'un article (à la main): la barre de raccourcis marche
- ajout d'une image, appel dans l'article avec <img1>: la barre ne marche plus, et FB dit 'barre_raccourci is not defined'
- j'enlève <img1>, sauve, modifie: la barre remarche
En revanche:
- je joins une image en document
- appel avec <doc2>: la barre marche
- appel avec <img2>: la barre marche
- tentative de tricherie: appel avec <doc1> (affichage parfait): la barre ne marche pas.
J'en conclue péniblement que la barre_raccourci ne marche pas si je mets <imgXX> dans l'article et si l'image à été ajoutée dans le formulaire d'ajout d'image/document qui ne propose que <img> comme inclusion.

Donc, avec FF sur une debian ; à l'air pareil avec un IE6 sur du xp (avec virtualbox).

Les erreurs sur MajStats et swap_couche n'apparaîtront qu'avec la barre typo, soit, mais j'avais posté ici vu que l'erreur survient sur un spip tout neuf aussi.

Frédéric Moinard a écrit :

J'en conclue péniblement que la barre_raccourci ne marche pas si je mets <imgXX> dans l'article et si l'image à été ajoutée dans le formulaire d'ajout d'image/document qui ne propose que <img> comme inclusion.

reproduit ici.

apparemment,
   <script type='text/javascript'
   src='../prive/javascript/spip_barre.js'></script>
n'est pas appelé dans ce cas

je subodorre un retour à vide de l'appel de ecrire/inc/documents.php (ligne 335)

denisb a écrit :

je subodorre un retour à vide de l'appel de ecrire/inc/documents.php (ligne 335)

voilà (ce me semble)
au moment de l'affichage du cartouche du doc (colonne de gauche),
si on sélectionne la première position (left)
ou
si on insère directement dans le texte <imgxx>
alors
   (++$num > 1) ?
est faux car :
   ++$num = 1
donc pas d'affichage de l'inclusion de spip_barre.js

denisb a écrit :

alors
  (++$num > 1) ?
est faux car :
  ++$num = 1
donc pas d'affichage de l'inclusion de spip_barre.js

hum non.
c'est pas tout à fait ça...

en fait (!)
si le doc/image à insérer est affiché dans la colonne de gauche,
alors
   (++$num > 1) ?
est toujours vrai.
donc pas d'affichage d'appel du script spip_barre.js

Bonjour,

denisb a écrit :

denisb a écrit :

alors
  (++$num > 1) ?
est faux car :
  ++$num = 1
donc pas d'affichage de l'inclusion de spip_barre.js

hum non.
c'est pas tout à fait ça...

en fait (!)
si le doc/image à insérer est affiché dans la colonne de gauche,
alors
  (++$num > 1) ?
est toujours vrai.
donc pas d'affichage d'appel du script spip_barre.js

ben là ch'suis impressionné :wink:

Frédéric Moinard a écrit :

ben là ch'suis impressionné :wink:

bof

plus précisemment :
si le *premier* document de la liste de gauche
est inséré dans l'article, alors ça foire.

si on insére le, les, suivants (<imgxx> ou <docxx>)
ça fonctionne (et pour cause...)

Frédéric Moinard a écrit :

J'en conclue péniblement que la barre_raccourci ne marche pas si je mets <imgXX> dans l'article et si l'image à été ajoutée dans le formulaire d'ajout d'image/document qui ne propose que <img> comme inclusion.

je reviens là-dessus :

concerne le fichier ecrire/inc/documents.php,
ses fonctions affiche_raccourci_doc() et afficher_case_document().

problème :
si un document ou une image présent dans la colonne de gauche
est déjà placé dans le texte de l'article (par <docxx>, <imgxx|left>...)
alors l'appel du script :
<script type="text/javascript" src="../prive/javascript/spip_barre.js"></script>
n'est pas fait et la barre typo devient inutilisable (raccourcis inaccessibles par double clic).

ce problème est dû au critère $doublon défini dans la fonction afficher_case_document() :

  // le doc est-il appele dans le texte ?
  $doublon = est_inclus($id_document);

qui considère actuellement que si le doc/img est déjà placé dans le texte, alors il n'est pas nécessaire de proposer, dans la colonne de gauche, les 3 possibilités de placement (left, center, right).

or l'affichage de l'appel du script "spip_barre.js" est lié à l'affichage de ces raccourcis.

donc :
si
   le doc/img est déjà placé dans le texte
alors
   pas d'appel du script.

solutions possibles :
1- toujours proposer les 3 raccourcis en supprimant le critère $doublon
     de la fonction afficher_case_document()
2- ne jamais faire afficher l'appel du script depuis la fonction
     affiche_raccourci_doc(), puisque de toute façon, ce script sera
     appelé par ecrire/inc/barre.php, en supprimant le test :
     ((++$num > 1) ? '' : http_script("", "spip_barre.js"))

* denisb tapuscrivait, le 30/11/2008 12:19:

Frédéric Moinard a écrit :

ben là ch'suis impressionné :wink:

bof

plus précisemment :
si le *premier* document de la liste de gauche
est inséré dans l'article, alors ça foire.

si on insére le, les, suivants (<imgxx> ou <docxx>)
ça fonctionne (et pour cause...)

Tu as fait un ticket que ça ne se perde pas ?

RealET a écrit :

Tu as fait un ticket que ça ne se perde pas ?

euh...
les tickets et moi, ça fait trois.

mais c'est là :
   http://article.gmane.org/gmane.comp.web.spip.devel/51282/

c'est en principe corrigé par 13323, si tu peux vérifier !

Cédric

Bonjour,

cedric.morin@yterium.com a écrit :

http://article.gmane.org/gmane.comp.web.spip.devel/51282/

c'est en principe corrigé par 13323, si tu peux vérifier !

Cela semble marcher, merci ! (Juste testé sur un article, une image, un document: toutes les icones marchent, y compris celles des plugins.)

cedric.morin@yterium.com a écrit :

http://article.gmane.org/gmane.comp.web.spip.devel/51282/

c'est en principe corrigé par 13323, si tu peux vérifier !

malgré de sévères malaxages (!),
ça roule nickel.

c'est parfait ! merci.