tab dans critères {0,n} #4945

Open
opened 2 weeks ago by JLuc · 3 comments
JLuc commented 2 weeks ago

Une tabulation après le 100 dans le critère {0,100 } provoque (avec SPIP 4.0.0)

Erreur de compilation | Line 46 : syntax error, unexpected '0' (T_LNUMBER)f=
...
$command['limit'] = ''0'.','.';
...

Ce serait bien d'éviter cette erreur.

Une tabulation après le 100 dans le critère `{0,100 }` provoque (avec SPIP 4.0.0) ``` Erreur de compilation | Line 46 : syntax error, unexpected '0' (T_LNUMBER)f= ... $command['limit'] = ''0'.','.'; ... ``` Ce serait bien d'éviter cette erreur.
b_b added this to the 4.0 milestone 2 weeks ago
b_b added the
amélioration
label 7 days ago
Owner

Ta proposition semble correcte.

Ceci dit comme dit dans la PR, il parait plus logique de mettre tout le critère à la ligne dans ce cas pour une présentation claire… mais soit.

Ta proposition semble correcte. Ceci dit comme dit dans la PR, il parait plus logique de mettre tout le critère à la ligne dans ce cas pour une présentation claire… mais soit.
Owner

Cela dit, ça ne fonctionne pas dès qu’il y a un saut de ligne avant ou après le / si c’est pas une balise

<BOUCLE_a(ARTICLES){id_rubrique}{
    1 /
     2
}>#TITRE</BOUCLE_a>
2021-11-22 14:07:17 ::1 (pid 9542) ecrire/public/debusquer.php:L180:debusquer_compose_message()::Pub:info: Debug: Critère inconnu &lt;span class='spip-debug-arg'>1 /
     2
&lt;/span>
Cela dit, ça ne fonctionne pas dès qu’il y a un saut de ligne avant ou après le `/` si c’est pas une balise ```html <BOUCLE_a(ARTICLES){id_rubrique}{ 1 / 2 }>#TITRE</BOUCLE_a> ``` ```php 2021-11-22 14:07:17 ::1 (pid 9542) ecrire/public/debusquer.php:L180:debusquer_compose_message()::Pub:info: Debug: Critère inconnu &lt;span class='spip-debug-arg'>1 / 2 &lt;/span> ```
JLuc commented 6 days ago
Poster

Le problème signalé par ce ticket c'est l'erreur PHP. Le patch a amélioré la situation puisqu'il n'y a plus d'erreur PHP. La syntaxe SPIP a aussi été assouplie (...par effet de bord). Mais le traitement de son parsing est dispatché dans plusieurs endroits du code et visiblement il reste des syntaxes non acceptées par SPIP pour ce critère.
Mais au moins l'erreur que tu signales @marcimat n'est pas une erreur php sauvage, mais le log qui accompagne une erreur joliement signalée par SPIP avec civilité.

Le problème signalé par ce ticket c'est l'erreur PHP. Le patch a amélioré la situation puisqu'il n'y a plus d'erreur PHP. La syntaxe SPIP a aussi été assouplie (...par effet de bord). Mais le traitement de son parsing est dispatché dans plusieurs endroits du code et visiblement il reste des syntaxes non acceptées par SPIP pour ce critère. Mais au moins l'erreur que tu signales @marcimat n'est pas une erreur php sauvage, mais le log qui accompagne une erreur joliement signalée par SPIP avec civilité.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.