Recherche avancée

Je dépose ici la discussion qui a eu lieu à ce sujet sur IRC pour en garder une trace :

< marcimat‎ >  dans id_table_objet() je vois que ça fait un traitement même si on passe ’id_auteur’
< marcimat‎ >  du coup, je me dis que quête logo
< marcimat‎ >  pourrait tester si substr($type, 0, 3) !== ’id_’
< marcimat‎ >  appeler id_table_objet($type)
< marcimat‎ >  non ?
< marcimat‎ >  c’est peut être un compromis
< marcimat‎ >  ou un strcmp
< phenix_spip‎ >  Pourquoi faire un test ?
< marcimat‎ >  parce que je vois des calculs dans id_table_objet() systématiques
< marcimat‎ >  même si on envoie déjà id_truc
< phenix_spip‎ > ?
< phenix_spip‎ >  Ha je vois
< marcimat‎ >  @id_table_objet()
< Edgard‎ >  Hop http://code.spip.net/@id_table_objet - Hop (sinon) http://code.spip.net/spip.php?page=recherche&recherche=id_table_objet
< marcimat‎ >  https://core.spip.net/projects/spip/repository/entry/spip/ecrire/base/objets.php#L1066 donc
< marcimat‎ >  mais effectivement, je pense que ça doit pas gêner si quête_logo est simplifié
< phenix_spip‎ >  marcimat : ce serai plus logique que le test soit dans id_table_objet non ?
< marcimat‎ >  enfin modifié pour accepter un vrai type
< marcimat‎ >  certes, mais c’est déjà plus sensible ^^
< phenix_spip‎ > ?
< marcimat‎ >  ok
< marcimat‎ >  en fait objet_type() teste déjà en enlevant id_
< marcimat‎ >  donc, y pa pas spécialement besoin
< phenix_spip‎ >  Ben voilà :)
< marcimat‎ >  donc là https://core.spip.net/projects/spip/repository/entry/spip/ecrire/base/objets.php#L1070
< marcimat‎ >  objet_type(’id_auteur’) doit ressortir ’auteur’
< marcimat‎ >  oui, voilà :)
< marcimat‎ >  all is fine

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.