Échec traiter_modeles #5329

Closed
opened 1 week ago by tcharlss · 5 comments
tcharlss commented 1 week ago
Owner

Contexte : je me sers de traiter_modeles() pour extraire les ids des modèles d'un type d'objet insérés dans les textes, idem plugin Médias.


Soit l'article n°1 avec 2 raccourcis d'album dans le texte :

<album1> <album2>

Et le code suivant pour tester si traiter_modeles() remplit bien la variable $GLOBALS['doublons_albums_inclus'] avec les ids détectés :

include_spip('inc/texte');
$texte = "<album1> <album2>";
$modeles = ['album'];
$contexte = [
	'objet'      => 'article',
	'id_objet'   => 1,
	'id_article' => 1,
];
traiter_modeles($texte, ['albums' => $modeles], '', '', null, $contexte);
var_dump($GLOBALS['doublons_albums_inclus']);

En SPIP 4.1 c'est bon :

array(2) {
  [0] =>
  string(1) "1"
  [1] =>
  string(1) "2"
}

Mais en SPIP 4.2-dev à jour, rien :

NULL

Possiblement lié aux récentes évolutions ici : https://git.spip.net/spip/spip/commits/branch/master/ecrire/inc/modeles.php
C'est encore en cours peut-être ?

Contexte : je me sers de traiter_modeles() pour extraire les ids des modèles d'un type d'objet insérés dans les textes, idem plugin Médias. ---- Soit l'article n°1 avec 2 raccourcis d'album dans le texte : ``` <album1> <album2> ``` Et le code suivant pour tester si traiter_modeles() remplit bien la variable `$GLOBALS['doublons_albums_inclus']` avec les ids détectés : ```php include_spip('inc/texte'); $texte = "<album1> <album2>"; $modeles = ['album']; $contexte = [ 'objet' => 'article', 'id_objet' => 1, 'id_article' => 1, ]; traiter_modeles($texte, ['albums' => $modeles], '', '', null, $contexte); var_dump($GLOBALS['doublons_albums_inclus']); ``` En SPIP 4.1 c'est bon : ``` array(2) { [0] => string(1) "1" [1] => string(1) "2" } ``` Mais en SPIP 4.2-dev à jour, rien : ``` NULL ``` Possiblement lié aux récentes évolutions ici : https://git.spip.net/spip/spip/commits/branch/master/ecrire/inc/modeles.php C'est encore en cours peut-être ?
tcharlss added this to the 4.2 milestone 1 week ago
tcharlss added the
bug
label 1 week ago
tcharlss changed title from Échec traiter_modeles avec les modèles autres que ceux des documents to Échec traiter_modeles 1 week ago
Poster
Owner

Je me rends compte que les documents insérés dans le texte ne sont plus marqués comme vu non plus, donc ça concerne tous les modèles à priori.

Je me rends compte que les documents insérés dans le texte ne sont plus marqués comme `vu` non plus, donc ça concerne tous les modèles à priori.
Owner

c'est fini, mais ça veut donc dire qu'on a pas de test unitaire sur le repérage des doublons de modèle, ce qu'il faut corriger avant de corriger le bug

c'est fini, mais ça veut donc dire qu'on a pas de test unitaire sur le repérage des doublons de modèle, ce qu'il faut corriger avant de corriger le bug
Owner

Le test est dans la boite et la PR qui va bien donc #5333

Le test est dans la boite et la PR qui va bien donc https://git.spip.net/spip/spip/pulls/5333
Poster
Owner

Merci, c'est tout bon

Merci, c'est tout bon
Owner

c'est intégré

c'est intégré
cerdic closed this issue 5 days ago
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.