Chaque table ayant un champ "lang" n'est pas obligé d'avoir de champ...

Chaque table ayant un champ "lang" n'est pas obligé d'avoir de champ langue_choisie (cas de spip_syndic_articles du plugin sites).

Du coup la fonction instituer_langue_objet provoquait des erreurs Mysql du type :

:Pri:ERREUR: Erreur 1054 de mysql: Unknown column 'langue_choisie' in 'field list'
in /blahblah/ecrire/action/instituer_langue_objet.php L39 [sql_updateq(),action_instituer_langue_objet_dist(),objet_modifier_champs(),inserer_article_syndique(),syndic_a_jour(),queue_start_job(),queue_schedule(),action_syndiquer_site_dist(),traiter_appels_actions()]
UPDATE `blablah`.spip_syndic_articles
SET lang='fr',langue_choisie='oui'
WHERE id_syndic_article=32583

Cette fonction n'est utilisée qu'une seule fois dans le core, au lieu de l'alourdir en passant à nouveau par trouver_table, on change sa signature, c'est objet_modifier_champs (où est son seul appel et qui fait déjà un trouver_table) qui stipule si oui ou non on change le champ langue_choisie.

ecrire/action/instituer_langue_objet.php

@@ -27,16 +27,20 @@ if (!defined('_ECRIRE_INC_VERSION')) {
  * @param int $id
  * @param int $id_rubrique
  * @param string $changer_lang
+ * @param bool $langue_choisie : la table à modifier dispose-t-elle d'un champ 'langue_choisie'
  * @return string
  */
-function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_lang) {
+function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_lang, $langue_choisie = true) {
 	if ($changer_lang) {
 		$table_objet_sql = table_objet_sql($objet);
 		$id_table_objet = id_table_objet($objet);
-
+		
 		if ($changer_lang != "herit") {
-			sql_updateq($table_objet_sql, array('lang' => $changer_lang, 'langue_choisie' => 'oui'),
-				"$id_table_objet=" . intval($id));
+			$champs = array('lang' => $changer_lang);
+			if ($langue_choisie) {
+				$champs['langue_choisie'] = 'oui';
+			}
+			sql_updateq($table_objet_sql, $champs, "$id_table_objet=" . intval($id));
 			include_spip('inc/rubriques');
 			if ($table_objet_sql == 'spip_rubriques') {
 				calculer_langues_rubriques();
@@ -48,8 +52,11 @@ function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_
 			if (!$langue_parent) {
 				$langue_parent = $GLOBALS['meta']['langue_site'];
 			}
-			sql_updateq($table_objet_sql, array('lang' => $langue_parent, 'langue_choisie' => 'non'),
-				"$id_table_objet=" . intval($id));
+			$champs = array('lang' => $langue_parent);
+			if ($langue_choisie) {
+				$champs['langue_choisie'] = 'non';
+			}
+			sql_updateq($table_objet_sql, $champs, "$id_table_objet=" . intval($id));
 			$changer_lang = $langue_parent;
 			if ($table_objet_sql == 'spip_rubriques') {
 				include_spip('inc/rubriques');

ecrire/inc/modifier.php

@@ -202,7 +202,7 @@ function objet_modifier_champs($objet, $id_objet, $options, $c = null, $serveur
 					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
 				}
 				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
-				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang);
+				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, isset($desc['langue_choisie']));
 			}
 			// on laisse 'lang' dans $champs,
 			// ca permet de passer dans le pipeline post_edition et de journaliser