<INCLURE{fond=mon_fond}{ajax}{id_truc=xx}..>

suppression de var_fragment et de ses implications dans le compilateur, remplaces avantageusement par [11289]
le critere {fragment} peut il etre supprime ?

$boucle->join['nom']=array("'table_arrivee'","'id_depart'","'id_arrivee'");
lorsque non precises, le comportement anterieur est conserve (Lx avex x le rang dans la table join, et cle arrivee=cle depart)
application au critere recherche qui dysfonctionnait en cas de recherche vide et/ou conditionnelle
incrementation de spip_version_code pour forcer recompilation des squelettes

deux nouveaux pipelines, 'pre_boucle' et 'post_boucle', permettent d'ajouter ou de supprimer des traitements au niveau des boucles, soit avant l'appel à la fonction boucle_XX, soit juste après cet appel. Il ne sera plus nécessaire de surcharger ces fonctions, et on pourra ainsi activer plusieurs plugins traitant les mêmes tables sans risques d'incompatibilité (du moins au niveau des déclarations de fonctions).

Amélioration de la détection des boucles dont le corps est constant, afin de transformer le While-Fetch en For sans requête (le Numrows initial suffit).
Le manque se situait dans la non prise en compte des {{{\'}}} dans l'analyse du texte.

Application au plugin Nuage qui veut mettre en exergue les mots clés d'un article:

{{{
<BOUCLE2(MOTS)><br />
<span<BOUCLE3(MOTS){id_article}{id_mot}> class='on'</BOUCLE3>>#TITRE</span>
</BOUCLE2>
}}}
qui fournit:
{{{
 $result = calculer_select(
        array("1"), # SELECT
        array('mots' => 'spip_mots','L1' => 'spip_mots_articles'), # FROM
        array(), # FROM    type jointure    
        
            array(
            array('=', 'L1.id_article', intval(@$Pile[0]['id_article'])), 
            array('=', 'mots.id_mot', intval($Pile[$SP]['id_mot']))), # WHERE
        array(1 => array('mots', 'id_mot')), # WHERE pour jointure
        array(), # GROUP
        array(), # ORDER
        '', # LIMIT
        
            array(), # HAVING
        'mots', # table
        '3', # boucle
        ''); # serveur
    $Numrows['3']['total'] = @sql_count($result,'');
    $t0 = "";
    for($x=$Numrows["3"]["total"];$x>0;$x--)
            $t0 .= 
// 15 signes
' class=\'arial1\'';
    @sql_free($result,'');
    return $t0;
}}}

	* la concaténation de deux constantes est ramenée à une seule;

	* strval n'est plus utilisé au retour d'une boucle, c'est totalement superflu;

	* les boucles vides ne sont plus testées au retour puisqu'on connait le résultat.

Il est vraiment utile que les boucles destinées à ne calculer que des doublons ou des compteurs aient un corps parfaitement vide, ça devient une optimisation vraisemblablement perceptible. Donc écrire  par exemple:

{{{<BOUCLEnom(type) />#TOTAL_BOUCLE<//Bnom>}}}

et pas

{{{
<BOUCLEnom(type)>
</BOUCLEnom>
#TOTAL_BOUCLE
</Bnom>}}}
}}}

Le compilateur accepte  {{{ <BOUCLEnom(type){critere} /> }}} pour écrire une boucle vide (surtout récursive, pour remplir un critère doublons ou calculer un #TOTAL_BOUCLE).

	* {{{ cpt > debut }}} est plus rapide que {{{ cpt-1 >= debut }}};

	* les appels au traducteur _T() sont calculés une seule fois si la boucle ne change pas la langue.

$boucle->from_type est un tableau du type array(alias du table => type_jointure, ...)
C'est pas necessaire mais possible a declarer le type du jointure pour le cas INNER

Accélération de include_spip+find_in_path: ces deux fonctions vont être fusionnées afin d'éviter des redondances de tests et des allocations de chaînes à chaque appel. Première étape: traitement des cas particuliers.

ATTENTION: les .php3 ne sont plus pris en compte à partir de maintenant.

Permettre aux boucles SPIP de ne pas raler si la table demandée est introuvable. Pour cela, il faut ajouter un ? juste avant la fin de parenthèse : <BOUCLE_nom(TABLE?)>, <BOUCLE_nom(TABLE jointure?)> ou encore <BOUCLE_nom(TABLE ?)>

Cela permet aux squelettes qui utilisent un plugin optionnel ayant créé table X de ne pas signaler par une erreur l'absence de cette table dans une boucle si le plugin n'a pas été installé.

<BOUCLE_plugin(X ?)>#BALISE</BOUCLE_plugin>
</B_plugin>
Texte affiché si X n'est pas présent ou si la boucle n'a pas d'élément
<//B_plugin>

Il était déjà possible d'obtenir à peu près le même comportement (sauf le texte sur l'absence d'élément) en utilisant la balise plugin et une noisette :
[(#PLUGIN{prefixe_plugin}|?{' ',''})
  <INCLURE(fond=squelette_contenant_la_boucle)>
]

L'initialisation de {{{$connect}}} dans le code compilé doit se faire aussi dans la fonction principale, car une inclusion peut en avoir besoin. On le déplace aussi en début de chaque fonction car il peut être nécessaire aux critères, mais c'est insatisfaisant: ça corrige seulement le cas du paramètre {{{$connect}}} dans l'URL, pas celui d'un squelette où une boucle sur un serveur A transmet à {{{typo}}} un champ d'une boucle sur un serveur B. Confusion absolue: une boucle récursive avec un critère concernant un serveur A au premier appel, et un serveur B aux appels suivants. Bug plus théorique qu'autre chose a priori, mais il faudra y revenir.

traitement plus subtil du hack xml ; attention je change la signature de recuperer_fond()... bugs a prevoir

Un fichier séparé pour les fonctions gérant les jointures: elles n'ont jamais été limitées à la compilation des critères, il aurait dû exister depuis le début.

utf8 reconnu en synonyme de utf-8 dans inc-charset car il apparait dans certains contextes (les stats par exemple)

Le standard SQL précise qu'une chaîne avec apostrophe se code avec une double apostrophe et non un \ ce que SQLite fait aussi, contrairement à MySQL et PG. 
En prévision des portages utilisant cette réprésentation, l'interface s'enrichit de la fonction {{{sql_quote}}}, qui s'ajoute à celles définies dans [10707] [10667], [10433], [10131], [10146], [10154] et [10113]

{{{
quote => fonction d'abstraction de la citation d'une constante SQL
}}}

Pour MySQL et PG, cette fonction est donc équivalente à {{{_q()}}} qui reste disponible, mais doit être considérée comme obsolète. Le présent dépôt a été obtenu par le script ci-dessous, dont on peut faire usage pour ses extensions personnelles:

{{{
for i in $(grep  -l '_q(' [bigpeau]*/*p)
do
	sed 's/_q(/sql_quote(/g' $i > x
	mv  x $i
done
}}}

Un ajustement manuel a été nécessaire pour le critère "=", le compilateur testant si le code qu'il a produit contient la fonction de citation.

Suppression de 3 paramètres superflus dans sql_select, sql_countsel, sql_fetsel, sql_getfetsel, afin d'avoir une interface de programmation avec des fonctions ayant au plus 8 arguments.

Petites modifications, normalement transparentes, dans le code produit par le compilateur pour maximiser le nombre d'occurrences de {{{$Pile[O]['}}}''nom''{{{']}}}. Cela concerne:

 * la balise ENV
 * les #PARAM non indiqués par les boucles

et déplacements de fonctions et inclusions pour que le validateur puisse exécuter le compilateur hors contexte.

La fonction '''trouver_table''' devient surchargeable sous le nom de '''base_trouver_table''', ce qui devrait permettre de résoudre proprement le problème ddes tables nommées bizarrement évoqué dans http://archives.rezo.net/spip-dev.mbox/200709.mbox/%3C46E5795D.60704@math.univ-montp2.fr%3E.

Maintenant qu'on cherche la définition de la table SQL en début de compilation, on la mémorise dans la structure générale plutot que de rappeler trouver_table pour chaque champ (et presque pour chaque critère). Les champs id_table et primary deviennent redondants mais c'est pas grave.

Rationnalisation des appels à '''trouver_table''': étant à présent parfois appelée hors d'une boucle à compiler, son deuxième argument, toujours optionnel, ne peut être un objet de la classe boucle, mais directement le nom du serveur.
L'erreur de table inconnue est indiquée dans spip_log, et à l'écran une fois pour toutes lors de la compilation de {{{<BOUCLEn(inconnue)...}}}. Dans le cas d'une table de jointure inconnue, seul spip.log l'indiquera: il n'est pas sûr que ce soit une erreur tellement fatale en l'état des jointures automatiques calculées par SPIP.

supprimer spip_var_export, du coup c'est php 4.2.0 minimum (mais cette fonction est dans spip.log et personne n'a rale

Limitation des occurrences de la globale {{{ table_des_tables }}}, qui doit être considérée comme une facilité d'abréviation du nom des tables SQL et rien d'autre.

Multi-base: les boucles référençant des connexions externes {{{<BOUCLE1(A:...}}} ou les pages appelées ainsi (paramètre d'URL {{{&connect=...}}} calculent à présent correctement les modèles contenus dans les champs SQL soumis aux filtre '''typo()''' et '''propre()'''. C'est un pas en avant dans la résolution complète de #716.

Différences avec les versions précédentes:

	* (bug de PCRE) les notes nommées {{{ [[<*> ...}}} ne peuvent plus contenir d'espaces entre les deux crochets et le chevron;

	* les fonctions redéfinissables {{{avant_propre}}} et {{{apres_propre}}} sont évacuées, faisant double emploi avec les pipelines {{{pre_propre}}} et {{{post_propre}}}.

	* surtout: le pipeline {{{pre_propre}}} recevra un texte où les raccourcis de liens ({{{ [->art1] }}} etc) seront déjà expansés.

Test:
{{{
Doc 2239 centré
<doc2239|center>

Je mets 2 fois un raccourci de glossaire
[?SPIP]  [?SPIP] 
puis un [raccourci->art1]
et un autre, automatique, [->art3634].

Suivre la note
[[elle est terminée par un  [raccourci->art1]]].

Un moment de poésie.
<poesie>
un 
   haiku
sur
2 lignes
</poesie>

Elle préfère parler d'un {accroissement de la dispersionsalariale} [[
 [{Perspectives économiques}->http://www.oecd.org/document/4/0,3343,fr_2649_201185_20347588_1_1_1_1,00.html]
- Vol. 2007-1, n¡~81, mai 2007, p. ~184. ]]

Elle aussi préfère parler d'un {accroissement de la dispersion salariale}
mais sur une seule ligne  [[  [{Perspectives économiques}->http://www.oecd.org/document/4/0,3343,fr_2649_201185_20347588_1_1_1_1,00.html]
- Vol. 2007-1, n¡~81, mai 2007, p. ~184. ]].

Une Juliette [[<*> sans numéro.]].
}}}

Multi-base: la fonction '''table_from_primary''' est à présent équivalente à '''trouver_table(table_objet($x))''', ce qui achève de centraliser dans '''trouver_table''' toutes les recherche sur les tables. Réécriture de la balise '''#EXPOSE''', seule utilisatrice de '''table_from_primary''', cette dernière fonction effectuant un calcul supplémentaire, reporté donc dans le calcul de cette balise.

#209: nouvelle rationnalisation de la fonction '''trouver_table''' introduite par [10133], qui affecte tous son résultat à la globale '''$connexions''' plutot que d'en recalculer une partie à chaque appel. Le tableau construit a toujours  les index '''field''', '''index''', '''table''' (nom SQL) et à présent '''id_table''' qui donne le nom du suivant le '''AS''' dans la requête SQL (il était nommé "type" auparavant, ce qui était trompeur maintenant qu'il existe le type du serveur dans cette structure de données).

Par ailleurs, les jointures dans une base externe ne marchaient plus. Quant aux balises #URL_ utilisées dans une base externe non SPIP, elles provoquaient une récursion infinie débouchant sur un {{{Illegal Instruction}}} par PHP.

Multibase et #877: les globales '''$tables_des_serveurs_sql''' et '''$type_des_serveurs''' disparaissent. Les informations quelles contenaient se retrouvent dans la globale '''connexion''', avec les sous-index respectifs '''tables''' et '''spip_connect_version'''. Une valeur nulle pour '''spip_connect_version''' indique que la base externe n'est pas sous SPIP (donc pas de gestion de préfixe). 

Ces disparitions permettent de centraliser dans la fonction '''trouver_table''' toutes les recherches de table afin d'améliorer facilement ce genre de recherche à l'avenir. Cette fonction quitte le fichier "criteres.php" car elle n'est plus spécifique à la compilation des criteres. Elle dispense le compilateur d'initialiser à chaque appel le tableau des tables SQL prédéfinies, ce qui est
aussi un gain.

Multibase: #716 ayant prémonitoirement demandé le multi-squelette suggéré dans [10113], le présent dépot (aidé de [10133] et [10138]) le réalise en deux coups de cuillère à Post. A présent, si on appelle une page avec un variable d'URL nommé '''connect''', SPIP considèrera qu'il doit utiliser non pas la base principale, mais celle indiquée par par l'entrée du tableau '''connexions''' indexée par la valeur du paramètre '''connect'''. Rappelons que tout fichier '''config/connect'''X'''.php''' garni l'entrée X du tableau '''connexions'''. Dans cette situation, les boucles des squelettes utilisés pour produire la page seront implicitement préfixées par la connexion demandée: {{{<BOUCLE(T) ...}}} sera vue comme {{{<BOUCLE(X:T) ...}}} quand l'URL comporte {{{&connect=X}}}.

Aspects techniques

La compilation d'un squelette produira autant de fichier dans tmp/cache/skel que d'application à des bases différentes. Il y aurait moyen de faire un plus compact, mais avec une petite chute de performances. On sacrifie donc l'espace au temps, ça ne semble pas un problème vu la taille usuelle d'un squelette compilé.

Les différences entre les compilations d'un même squelettes proviennent des balises #URL_* et assimilées. Il y aura peut-etre une petite réorganisation des fonctions sur les balises et les critères à opérer pour gérer ça plus astucieusement.

Le multi-base détecte quand un site distant est sous SPIP, et le compilateur de squelette applique alors les mêmes abréviations pour {{{<BOUCLE(A:T ...}}} que pour une boucle T locale, en prenant en compte le préfixe de table spécifique au site distant, ce qui réalise le souhait de [10113].

Quelques remarques:

	* la détection repose sur l'affectation de la variable '''spip_connect_version''' dans le fichier de connexion, et elle doit avoir la valeur 0.6. Détruire ce fichier et réinstaller le site distant si ce n'est pas le cas.

	* une manière d'obtenir un squelette pour site distant nommé SITE_DISTANT à partir d'un squelettes standard est {{{sed 's/(\([ABDFRS]\)/(SITE_DISTANT:\1/'}}}

	* la fonction '''trouver_def_table''' était appelée de manière incohérente (parfois avec le préfixe de table, parfois non) et était en partie redondante avec la fonction '''description_type_requete'''. Pour y remédier et atteindre le but ici décrit, ces deux fonctions ont été réunies en une seule, '''trouver_table''', dont les spécifications sont celles qu'avait '''description_type_requete''' (autrement dit cette fonction a été renommée). Etant très internes au compilateur, ces fonctions ne figurent pas dans vieilles_def (mais on peut les mettre si c'est vraiment nécessaire).

#209: l'élimination d'apostrophes (exigée par PG) autour d'une constante numérque dans un critère ''col=valeur'' était mal calculé par [9859] car cette colonne n'est pas forcément dans la table de la boucle, elle peut-etre dans une table de jointure. Le code est donc reporté dans la fonction détectant le besoin de jointure pour récupérer la description de la bonne table.

Correction de #954: bien gérer le critère '''doublons''' dans une table externe. La fonction '''trouver_def_table''' n'était pas utilisée à l'initialisation du compilateur c'était incohérent avec la suite. Mais il reste une dizaine d'occurrences du préfixe de table "'spip_'" qu'il faudrait revoir, il y a peut-être le même bug derrière.

Renommage de toutes les fonctions '''spip_abstract_''' en '''sql_''', c'est plus court et plus parlant puisqu'il s'agit des fonctions d'interfaces avec un serveur SQL. De nouveau attention il faut vider le cache (en fait seulement celui des squelettes: tmp/cache/skel) car ce sont des fonctions que le compilateur place dans le code compilé.

En conséquence, l'annonce de [9852] est modifiée: '''spip_fetch_array''' est remplacée par '''sql_fetch'''. Rien de modifié en revanche en ce qui concerne le fait que spip_fetch_array avec un deuxième argument égal à SPIP_NUM n'a pas d'équivalent et doit être réécrit si on éviter le recours à vieilles_def.php.

Dépot obtenu par le script suivant:
{{{
grep  -v '// http://doc.spip.org/@spip_abstract_' base/abstract_sql.php > b
mv b base/abstract_sql.php 

for i in $(grep -l "spip_abstract_" */*p|grep -v vieilles_def)
do
sed "s/spip_abstract_/sql_/g" $i > x; mv  x $i
done
}}}

et intervention manuelle sur vieilles_def.php.

#209: comme l'avait prévu [9859], autre cas de nombres à ne pas entourer d'apostrophies, les clés de jointure. Il a fallu réfléchir longtemps, mais le changement est minimal, preuve que le compilateur a les bons utilitaires c'est rassurant.

Aussi dans ce dépot, quelques corrections  dans l'inteface PG:
	° [9861] était vraiment à l'arraché, il ne fallait pas oublier de supprimer les AS ingjctés dans le GROUPBY;
	* traitement du HAVING par la fonction pour PG pas pour MySQL;
	* inclusion inopérante du débusqueur.