Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-13 Par sujet Charles Razack

Le 12/09/2019 à 19:11, Maïeul a écrit :
Note que cela ne concerne que les saisies *AUTONOME* (donc celle qui 
ne s'appuient pas sur saisies/_base.html


Je viens de le rendre plus explicite sur l'article 
https://contrib.spip.net/Fonctionnement-technique-de-la-verification-des#nb1


Ah ok, je vois, my bad.
C'est pour cette raison que j'évite absolument de surcharger _base.html 
même quand il y a besoin de présentations assez spéciales.


Top pour l'article !


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet Maïeul

Le 12/09/2019 à 16:44, Charles Razack a écrit :

Le 12/09/2019 à 15:02, Maïeul Rouquette a écrit :

en fait non, c'était pas là. Je croyais l'avoir fait. Je vais faire un
message general.

Merci pour le boulot de réécriture.
Il faut surtout que ça apparaisse en bien visible dans un des articles 
sur contrib.
J'ai plusieurs saisies persos dans la nature, j'étais pas loin d'être 
passé à côté.


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

Note que cela ne concerne que les saisies *AUTONOME* (donc celle qui ne 
s'appuient pas sur saisies/_base.html


Je viens de le rendre plus explicite sur l'article 
https://contrib.spip.net/Fonctionnement-technique-de-la-verification-des#nb1



je viens également de faire que la fonctions soit systématiquement 
chargé, sans avoir besoin de fichier  _fonctions.php



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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet Charles Razack

Le 12/09/2019 à 15:02, Maïeul Rouquette a écrit :

en fait non, c'était pas là. Je croyais l'avoir fait. Je vais faire un
message general.

Merci pour le boulot de réécriture.
Il faut surtout que ça apparaisse en bien visible dans un des articles 
sur contrib.
J'ai plusieurs saisies persos dans la nature, j'étais pas loin d'être 
passé à côté.


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet Maïeul Rouquette
Le jeudi 12 septembre 2019 à 14:15 +0200, nicod_ a écrit :
> Le 12/09/2019 à 14:06, Maïeul Rouquette a écrit :
> > c'était annoncé dans le premier message au tout début de cette
> > discussion, quand j'annoncais l'entame du projet.
> 
> Ça m'avait échappé.
> 
> > pas moyen de faire autrement non, puisque le principe, par
> > définition,
> > est de mettre les tests comme attribut du contenur.
> 
> Oui, je vois pas trop comment faire autrement.
> 
> J'ai fait un grep sur la zone, je vois ces plugins là qui utilisent
> le 
> pipeline saisies_autonomes :
> 
> ckeditor-spip-plugin/branches
> ckeditor-spip-plugin/trunk
> gis/branches/v2
> gis/branches/v3
> gis/trunk
> 
> 

en fait non, c'était pas là. Je croyais l'avoir fait. Je vais faire un
message general.



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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet nicod_

Le 12/09/2019 à 14:06, Maïeul Rouquette a écrit :

c'était annoncé dans le premier message au tout début de cette
discussion, quand j'annoncais l'entame du projet.


Ça m'avait échappé.


pas moyen de faire autrement non, puisque le principe, par définition,
est de mettre les tests comme attribut du contenur.


Oui, je vois pas trop comment faire autrement.

J'ai fait un grep sur la zone, je vois ces plugins là qui utilisent le 
pipeline saisies_autonomes :


ckeditor-spip-plugin/branches
ckeditor-spip-plugin/trunk
gis/branches/v2
gis/branches/v3
gis/trunk


--
nicod_

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet Maïeul Rouquette
Le jeudi 12 septembre 2019 à 14:04 +0200, nicod_ a écrit :
> Le 12/09/2019 à 12:45, Maïeul a écrit :
> > pour info, le fin mot de l'histoire :
> > ainsi qu'annonce, toute saisie perso (soit avec un _base.html
> > perso, 
> > soit qui ne passe pas par _base) doit désormais avoir
> > 
> > [ 
> > data-
> > afficher_si="(#ENV*{afficher_si}|saisies_afficher_si_js{#ENV{_saisie
> > s}})"] 
> > 
> > 
> > sur le contenur.
> 
> Ouhlà, grosse régression là pour le coup...
> 
> On peut vraiment pas faire autrement ?
> 
> C'est annoncé où ? en tout cas ça mérite une annonce bien visible
> plutôt 
> qu'au fond de cette discussion.
> 
c'était annoncé dans le premier message au tout début de cette
discussion, quand j'annoncais l'entame du projet.

pas moyen de faire autrement non, puisque le principe, par définition,
est de mettre les tests comme attribut du contenur. 


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet nicod_

Le 12/09/2019 à 12:45, Maïeul a écrit :

pour info, le fin mot de l'histoire :
ainsi qu'annonce, toute saisie perso (soit avec un _base.html perso, 
soit qui ne passe pas par _base) doit désormais avoir


[ 
data-afficher_si="(#ENV*{afficher_si}|saisies_afficher_si_js{#ENV{_saisies}})"] 



sur le contenur.


Ouhlà, grosse régression là pour le coup...

On peut vraiment pas faire autrement ?

C'est annoncé où ? en tout cas ça mérite une annonce bien visible plutôt 
qu'au fond de cette discussion.


--
nicod_

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-12 Par sujet Maïeul

Le 11/09/2019 à 11:45, Pierre KUHN a écrit :
Oui tout mes formulaires et ce formulaire en a qu'une seul, si je la 
simplifie ça bloque aussi




pour info, le fin mot de l'histoire :
ainsi qu'annonce, toute saisie perso (soit avec un _base.html perso, 
soit qui ne passe pas par _base) doit désormais avoir


[ 
data-afficher_si="(#ENV*{afficher_si}|saisies_afficher_si_js{#ENV{_saisies}})"]


sur le contenur.

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-11 Par sujet Maïeul

Le 11/09/2019 à 11:45, Pierre KUHN a écrit :
Oui tout mes formulaires et ce formulaire en a qu'une seul, si je la 
simplifie ça bloque aussi




le plus simple serait que tu m'envoie ton formulaire...


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-11 Par sujet Pierre KUHN
Oui tout mes formulaires et ce formulaire en a qu'une seul, si je la
simplifie ça bloque aussi

Le mer. 11 sept. 2019 à 11:02, Maïeul  a écrit :

> Le 11/09/2019 à 09:13, Pierre KUHN a écrit :
> > Bonjour Maïeul,
> >
> > Je bloque sur le test : 'afficher_si' => '(@id_lieu@ == "0") ||
> > (@perm_conci_autre@ != "")'
> > J'ai essayer de le simplifier mais je trouve pas la solution.
> >
> > J'ai dans les log
> >
> plugins/saisies/inc/saisies_afficher_si_js.php:L56:saisies_afficher_si_js()::Pub:CRITIQUE:
>
> > Afficher_si incorrect.  syntaxe incorrecte
> >
> > Une idée de correction ?
> >
> > Merci.
> >
>
> je viens de tester directement ce test, et je n'ai pas d'erreur de
> syntaxe. Tu dois en avoir un autre qui pose souci. Ce qui m'étonne c'est
> que le log t'indique pas la condition qui pose problème, comme si la
> fonction la recevait pas.
>
>
> Ton formulaire marchait avant la modification de saisies?
>


-- 

Pierre KUHN
http://www.voisins-spipeurs.net

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-11 Par sujet Maïeul

Le 11/09/2019 à 09:13, Pierre KUHN a écrit :

Bonjour Maïeul,

Je bloque sur le test : 'afficher_si' => '(@id_lieu@ == "0") || 
(@perm_conci_autre@ != "")'

J'ai essayer de le simplifier mais je trouve pas la solution.

J'ai dans les log 
plugins/saisies/inc/saisies_afficher_si_js.php:L56:saisies_afficher_si_js()::Pub:CRITIQUE: 
Afficher_si incorrect.  syntaxe incorrecte


Une idée de correction ?

Merci.



je viens de tester directement ce test, et je n'ai pas d'erreur de 
syntaxe. Tu dois en avoir un autre qui pose souci. Ce qui m'étonne c'est 
que le log t'indique pas la condition qui pose problème, comme si la 
fonction la recevait pas.



Ton formulaire marchait avant la modification de saisies?

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-11 Par sujet Pierre KUHN
Bonjour Maïeul,

Je bloque sur le test : 'afficher_si' => '(@id_lieu@ == "0") ||
(@perm_conci_autre@ != "")'
J'ai essayer de le simplifier mais je trouve pas la solution.

J'ai dans les log
plugins/saisies/inc/saisies_afficher_si_js.php:L56:saisies_afficher_si_js()::Pub:CRITIQUE:
Afficher_si incorrect.  syntaxe incorrecte

Une idée de correction ?

Merci.

Le mar. 10 sept. 2019 à 18:49, Maïeul  a écrit :

> Le 10/09/2019 à 16:46, RastaPopoulos a écrit :
> > Le 10/09/2019 à 15:53, RastaPopoulos a écrit :
> >> Le 06/09/2019 à 17:39, Maïeul Rouquette a écrit :
> >>> Super! Merci. Bon, vu le peu d'enthousiasme a tester, je crois que je
> >>> vais merger aujourd'hui, ca me permettra de voir demain s'il ya des
> >>> merdes : on lance un formulaire :)
> >>
> >> Hello Maieul, et bien c'est parfait, ça m'a permis de voir des merdes
> >> effectivement :p
> >>
> >> Là j'ai mis à jour sur un site avec Profils, et la config de ce plugin
> >> ne marche plus. Ça utilise des afficher_si appliqué à des cases à cocher
> >> qui sont des tableaux : name=config[activer_contact], et quand on clique
> >> dessus, ça doit ouvrir des fieldsets en plus.
> >>
> >> Exemple là :
> >>
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L176
> >>
> >> Ou là :
> >>
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L211
> >>
> >> Tout cela marchant parfaitement avant le merge :)
> >
> > Ah oui, et il n'y a aucune erreur JS ou autre, ça masque bien ce qui
> > doit être masqué au démarrage, mais ça ne s'applique pas du tout sur les
> > événements où on modifie les champs (quand on switch la case ça fait
> > rien, aucun comportement).
> >
> petit bug idiot, qui n'avait rien à voir avec les trucs imbriqués, mais
> avec le nom de la saisie conditionnante
>
> https://zone.spip.net/trac/spip-zone/changeset/117805
>
> en gros historiquement les tests permette d'utilise @config:xxx@ pour
> tester une config. Et en pratique, dans la modif du js, j'avais fait
> test 'config' et pas 'config:'.
>
> + j'avais oublié qu'on pouvait tester @case@ == 'on'
>
>
> https://zone.spip.net/trac/spip-zone/changeset/117806
> 
> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone
>


-- 

Pierre KUHN
http://www.voisins-spipeurs.net

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-10 Par sujet Maïeul

Le 10/09/2019 à 16:46, RastaPopoulos a écrit :

Le 10/09/2019 à 15:53, RastaPopoulos a écrit :

Le 06/09/2019 à 17:39, Maïeul Rouquette a écrit :

Super! Merci. Bon, vu le peu d'enthousiasme a tester, je crois que je
vais merger aujourd'hui, ca me permettra de voir demain s'il ya des
merdes : on lance un formulaire :)


Hello Maieul, et bien c'est parfait, ça m'a permis de voir des merdes
effectivement :p

Là j'ai mis à jour sur un site avec Profils, et la config de ce plugin
ne marche plus. Ça utilise des afficher_si appliqué à des cases à cocher
qui sont des tableaux : name=config[activer_contact], et quand on clique
dessus, ça doit ouvrir des fieldsets en plus.

Exemple là :
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L176

Ou là :
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L211

Tout cela marchant parfaitement avant le merge :)


Ah oui, et il n'y a aucune erreur JS ou autre, ça masque bien ce qui
doit être masqué au démarrage, mais ça ne s'applique pas du tout sur les
événements où on modifie les champs (quand on switch la case ça fait
rien, aucun comportement).

petit bug idiot, qui n'avait rien à voir avec les trucs imbriqués, mais 
avec le nom de la saisie conditionnante


https://zone.spip.net/trac/spip-zone/changeset/117805

en gros historiquement les tests permette d'utilise @config:xxx@ pour 
tester une config. Et en pratique, dans la modif du js, j'avais fait 
test 'config' et pas 'config:'.


+ j'avais oublié qu'on pouvait tester @case@ == 'on'


https://zone.spip.net/trac/spip-zone/changeset/117806

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-10 Par sujet RastaPopoulos
Le 10/09/2019 à 15:53, RastaPopoulos a écrit :
> Le 06/09/2019 à 17:39, Maïeul Rouquette a écrit :
>> Super! Merci. Bon, vu le peu d'enthousiasme a tester, je crois que je
>> vais merger aujourd'hui, ca me permettra de voir demain s'il ya des
>> merdes : on lance un formulaire :)
> 
> Hello Maieul, et bien c'est parfait, ça m'a permis de voir des merdes
> effectivement :p
> 
> Là j'ai mis à jour sur un site avec Profils, et la config de ce plugin
> ne marche plus. Ça utilise des afficher_si appliqué à des cases à cocher
> qui sont des tableaux : name=config[activer_contact], et quand on clique
> dessus, ça doit ouvrir des fieldsets en plus.
> 
> Exemple là :
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L176
> 
> Ou là :
> https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L211
> 
> Tout cela marchant parfaitement avant le merge :)

Ah oui, et il n'y a aucune erreur JS ou autre, ça masque bien ce qui
doit être masqué au démarrage, mais ça ne s'applique pas du tout sur les
événements où on modifie les champs (quand on switch la case ça fait
rien, aucun comportement).

-- 
RastaPopoulos


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-10 Par sujet RastaPopoulos
Le 06/09/2019 à 17:39, Maïeul Rouquette a écrit :
> Super! Merci. Bon, vu le peu d'enthousiasme a tester, je crois que je
> vais merger aujourd'hui, ca me permettra de voir demain s'il ya des
> merdes : on lance un formulaire :)

Hello Maieul, et bien c'est parfait, ça m'a permis de voir des merdes
effectivement :p

Là j'ai mis à jour sur un site avec Profils, et la config de ce plugin
ne marche plus. Ça utilise des afficher_si appliqué à des cases à cocher
qui sont des tableaux : name=config[activer_contact], et quand on clique
dessus, ça doit ouvrir des fieldsets en plus.

Exemple là :
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L176

Ou là :
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/profils/trunk/formulaires/editer_profil.php#L211

Tout cela marchant parfaitement avant le merge :)

-- 
RastaPopoulos


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-07 Par sujet Maïeul Rouquette
t'a essayé un var_mode=recalcul ?
c'est vraiment bizarre. Tu pourrais m'envoyer le code du formulaire qui
pose problème?


Le samedi 07 septembre 2019 à 11:37 +0200, Pierre KUHN a écrit :
> Bonjour
> 
> Je me trouve avec un filtre afficher_si_js inconnu.
> 
> Une idée de correction ?
> 
> Merci.
> 
> 


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-07 Par sujet Pierre KUHN
Bonjour

Je me trouve avec un filtre afficher_si_js inconnu.

Une idée de correction ?

Merci.

Le ven. 6 sept. 2019 à 18:49, Maïeul  a écrit :

> Le 17/08/2019 à 16:24, Maïeul a écrit :
> > Le 21/07/2019 à 21:46, nicod_ a écrit :
>
> >
> Suite aux différents test, notamment ceux de Nicod et de JLuc, et vu
> qu'il n'y a pas eu d'echo negatifs, je viens de reporter (git rebase!)
> les commits sur le trunk.
>
> Si vous voyez des problèmes avec les afficher_si dans les prochains
> jours, me le signaler.
>
> Maïeul
> 
> spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone
>

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-06 Par sujet Maïeul

Le 17/08/2019 à 16:24, Maïeul a écrit :

Le 21/07/2019 à 21:46, nicod_ a écrit :




Suite aux différents test, notamment ceux de Nicod et de JLuc, et vu 
qu'il n'y a pas eu d'echo negatifs, je viens de reporter (git rebase!) 
les commits sur le trunk.


Si vous voyez des problèmes avec les afficher_si dans les prochains 
jours, me le signaler.


Maïeul


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-06 Par sujet Maïeul Rouquette
Le vendredi 06 septembre 2019 à 17:18 +0200, nicod_ a écrit :
> Le 17/08/2019 à 16:24, Maïeul a écrit :
> 
> > des nouvelles des tests ? pour info je viens de poster une modif
> > qui, 
> > lorsqu'on rempli le champ 'afficher_si' d'une config de saisie,
> > vérifie 
> > la syntaxe. Cela laisse peut être passer des faux positifs, mais
> > c'est 
> > déjà bien mieux.
> 
Super! Merci. Bon, vu le peu d'enthousiasme a tester, je crois que je
vais merger aujourd'hui, ca me permettra de voir demain s'il ya des
merdes : on lance un formulaire :)


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-06 Par sujet nicod_

Le 17/08/2019 à 16:24, Maïeul a écrit :

des nouvelles des tests ? pour info je viens de poster une modif qui, 
lorsqu'on rempli le champ 'afficher_si' d'une config de saisie, vérifie 
la syntaxe. Cela laisse peut être passer des faux positifs, mais c'est 
déjà bien mieux.


Salut,
je viens de tester plusieurs formulaires un peu complexes avec beaucoup 
de afficher_si, et tout est ok.
Le code généré est bien plus compact et générique, et les tests de 
syntaxe, génial !


Un grand bravo et merci pour tout ce travail !

Par contre, pas moyen de vérifier qu'on appelle les bon champs (enfin, 
ce serait possible, mais grosse, grosse réécritures).


C'est déjà une sacrée grosse évolution en l'état, ça ira comme ça je pense.

--
nicod_

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Maïeul

Le 02/09/2019 à 17:58, Matthieu Marcillaud a écrit :

Le 02/09/2019 à 17:52, Matthieu Marcillaud a écrit :

Le 02/09/2019 à 17:34, Maïeul a écrit :?
les tests unittaires que j'ai ajouté dans le plugins passent tous, et 
jes les ai adaptés / complétés au fur et à mesure


Une minuscule erreur je pense : Fatal error: Uncaught Error: Call to 
undefined function saisies_evaluer_afficher_si() in 
tests/saisies_afficher_si/saisies_evaluer_afficher_si_config.php on 
line 19


Je n’avais pas regardé le détail.

Et donc je n’ai pas saisies directement dans plugins/
Donc à la place de
find_in_path("../plugins/saisies/inc/saisies_afficher_si.php",'',true);
Je présume que tu veux soit qqc comme :
find_in_path("inc/saisies_afficher_si.php",'',true);
soit qqc comme :
require_once __DIR__ . '../../inc/saisies_afficher_si.php';

MM.

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


Je met en copie Cedric, qui pourra peut être éclairer ma lanterne.
J'avais fait avec test builder puis recopié pour avoir des nouveaux 
tests (mais je comprend pas trop l'interface de testbuilder).


En fait je lance pas les tests unitaires depuis testbuilder (parce que 
je retrouve jamais l'endroit où les lancer, ou bien cela me les lance 
pour tous les plugins + ceux de spip (qui soit en dit en passant, 
échouent pour la moitié) mais en allant directement sur


http://localhost/plugins/saisies/tests/

la question que je me pose c'est : quel est donc la bonne pratique?


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Matthieu Marcillaud

Le 02/09/2019 à 17:52, Matthieu Marcillaud a écrit :

Le 02/09/2019 à 17:34, Maïeul a écrit :?
les tests unittaires que j'ai ajouté dans le plugins passent tous, et 
jes les ai adaptés / complétés au fur et à mesure


Une minuscule erreur je pense : Fatal error: Uncaught Error: Call to 
undefined function saisies_evaluer_afficher_si() in 
tests/saisies_afficher_si/saisies_evaluer_afficher_si_config.php on line 19


Je n’avais pas regardé le détail.

Et donc je n’ai pas saisies directement dans plugins/
Donc à la place de
find_in_path("../plugins/saisies/inc/saisies_afficher_si.php",'',true);
Je présume que tu veux soit qqc comme :
find_in_path("inc/saisies_afficher_si.php",'',true);
soit qqc comme :
require_once __DIR__ . '../../inc/saisies_afficher_si.php';

MM.

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Matthieu Marcillaud

Le 02/09/2019 à 17:34, Maïeul a écrit :?
les tests unittaires que j'ai ajouté dans le plugins passent tous, et 
jes les ai adaptés / complétés au fur et à mesure


Une minuscule erreur je pense : Fatal error: Uncaught Error: Call to 
undefined function saisies_evaluer_afficher_si() in 
tests/saisies_afficher_si/saisies_evaluer_afficher_si_config.php on line 19


Je n’avais pas regardé le détail.

MM.


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Maïeul

Le 02/09/2019 à 15:31, Matthieu Marcillaud a écrit :

Le 02/09/2019 à 12:44, Maïeul a écrit :

Le 17/08/2019 à 16:24, Maïeul a écrit :


plop,
des nouvelles des tests ? 


En parlant de tests/ (unitaires) je crois que faut les adapter justement 
à tes changements.


MM.


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


?
les tests unittaires que j'ai ajouté dans le plugins passent tous, et 
jes les ai adaptés / complétés au fur et à mesure


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Matthieu Marcillaud

Le 02/09/2019 à 12:44, Maïeul a écrit :

Le 17/08/2019 à 16:24, Maïeul a écrit :


plop,
des nouvelles des tests ? 


En parlant de tests/ (unitaires) je crois que faut les adapter justement 
à tes changements.


MM.


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-09-02 Par sujet Maïeul

Le 17/08/2019 à 16:24, Maïeul a écrit :


plop,
des nouvelles des tests ? pour info je viens de poster une modif qui, 
lorsqu'on rempli le champ 'afficher_si' d'une config de saisie, vérifie 
la syntaxe. Cela laisse peut être passer des faux positifs, mais c'est 
déjà bien mieux.


Par contre, pas moyen de vérifier qu'on appelle les bon champs (enfin, 
ce serait possible, mais grosse, grosse réécritures).




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


hop, un petit up.


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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-08-17 Par sujet Maïeul

Le 21/07/2019 à 21:46, nicod_ a écrit :
PS : avec l'expérience, elle maitrise plutôt bien maintenant les tests 
et les conditions et leur logique (et/non/ou). Mais le principal souci 
qu'elle a par moment, c'est que n'étant pas développeuse, il lui arrive 
faire une erreur de syntaxe dans une conditions et dans ces cas là, elle 
est perdue.


Je n'ai pas encore regardé ton code, mais tu penses possible 
d'intercepter les erreurs de syntaxe dans les conditions, pour lever une 
alerte ou un message par exemple ?


Ça me parait un peu complexe, vu que la syntaxe n'est pas du JS pur...


plop,
des nouvelles des tests ? pour info je viens de poster une modif qui, 
lorsqu'on rempli le champ 'afficher_si' d'une config de saisie, vérifie 
la syntaxe. Cela laisse peut être passer des faux positifs, mais c'est 
déjà bien mieux.


Par contre, pas moyen de vérifier qu'on appelle les bon champs (enfin, 
ce serait possible, mais grosse, grosse réécritures).




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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-07-21 Par sujet Maïeul

Le 21/07/2019 à 21:46, nicod_ a écrit :
PS : avec l'expérience, elle maitrise plutôt bien maintenant les tests 
et les conditions et leur logique (et/non/ou). Mais le principal souci 
qu'elle a par moment, c'est que n'étant pas développeuse, il lui arrive 
faire une erreur de syntaxe dans une conditions et dans ces cas là, elle 
est perdue.


Je n'ai pas encore regardé ton code, mais tu penses possible 
d'intercepter les erreurs de syntaxe dans les conditions, pour lever une 
alerte ou un message par exemple ?


Ça me parait un peu complexe, vu que la syntaxe n'est pas du JS pur...


Merci

je dirais que même des dev font des erreurs de syntaxe. J'ai apporté une 
petite modif à formidable (indépendamment du problème ici) : le memento 
affiche désormais aussi les clé possibles pour un champ.


Cela étant, pour revenir à ta question des erreurs de syntaxe : le fait 
de passer par un parseur commun PHP/JS (dont les règles sont décrites 
dans l'article de doc) permet justement de détecter les erreurs de syntaxe.


Typiquement, dans la nouvelle branche :
- une condition qui ne respecterait pas cette syntaxe sera mis à vide
- un message de log sera écrit (dans le fichier saisies.log)

On pourrait imaginer d'afficher un message d'erreur le cas échéant, 
c'est juste un problème d'intégration/d'ergonomie (et un peu de travail 
pour mieux séparer encore les choses).


La nouvelle version est-même capable de détecter si tu fais un test sur 
un champ inexistant (et dans ce cas il supprime le tests correspondant 
au champ).


L'analyse syntaxique est quasi complète. Pourquoi quasi ? parceque cela 
fonctionne sous tests par sous tests.


C'est à dire si j'ai
 && 

il va me renvoyer &&, car il a detecté les deux tests faux, ce qui est 
est aussi faux.


Mais bon, ca peut se corriger ca. Faux juste reflechir un peu aux 
différents cas possible




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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-07-21 Par sujet nicod_
PS : avec l'expérience, elle maitrise plutôt bien maintenant les tests 
et les conditions et leur logique (et/non/ou). Mais le principal souci 
qu'elle a par moment, c'est que n'étant pas développeuse, il lui arrive 
faire une erreur de syntaxe dans une conditions et dans ces cas là, elle 
est perdue.


Je n'ai pas encore regardé ton code, mais tu penses possible 
d'intercepter les erreurs de syntaxe dans les conditions, pour lever une 
alerte ou un message par exemple ?


Ça me parait un peu complexe, vu que la syntaxe n'est pas du JS pur...

--
nicod_

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


Re: [SPIP Zone] Saisies et afficher si : la grande réécriture

2019-07-21 Par sujet nicod_

Le 21/07/2019 à 17:45, Maïeul a écrit :

Chers tous, chères toutes,

J'ai créé une branche pour réécrire la manière dont les afficher_si 
fonctionnaient niveau javascript. Est-ce que des bonnes âmes voudraient 
bien tester d'ici le 15 septembre ?


Cher Maïeul,

merci pour tout ce boulot (énorme).

J'ai une utilisatrice qui génère régulièrement de (très) gros 
formulaires administratifs pleins d'afficher_si et de tonnes d'autres 
options dans tous les sens.

Je vais prévoir une séance de tests de mon côté, et avec elle.
On devrait pouvoir caser ça avant le 15 septembre.

La bise,

--
nicod_

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