Je vois peut-être des choses, mais ce composant attend un modèle contenant un objet topic :
Cependant, lorsque je déconnecte ce modèle, il n’y a plus d’attribut topic ? :
Y a-t-il une raison pour laquelle l’attribut topic a été supprimé ? Je l’utilisais aussi pour une extension de l’adplugin
2 « J'aime »
david
(David Taylor)
Février 14, 2024, 11:45
3
Salut Robert, je ne pense pas que ce soit intentionnel. As-tu pu savoir à quoi @model fait référence ? Ce serait bien de déterminer s’il s’agissait d’un changement de cœur ou de plugin.
1 « J'aime »
Est-ce que le modèle post passé par le connecteur post-bottom n’est pas pris en compte ?
{{post-bottom-ad model=this}}
Cela dit, j’ai effectué une recherche pour post-bottom sur le dépôt discourse et elle ne donne rien… Je vais creuser un peu plus.
1 « J'aime »
Ce code semble très ancien :
transformed.cloaked = _cloaked[post.id];
postTransformCallbacks(transformed);
if (transformed.isSmallAction) {
result.push(
this.attach("post-small-action", transformed, { model: post })
);
} else {
transformed.showReadIndicator = attrs.showReadIndicator;
result.push(this.attach("post", transformed, { model: post }));
}
// Post gap - after
const afterGap = after[post.id];
if (afterGap) {
result.push(
this.attach(
"post-gap",
{ pos: "after", postId: post.id, gap: afterGap },
{ model: post }
Qui contient l’objet topic, donc les choses changent en dessous de ceci
2 « J'aime »
david
(David Taylor)
Février 14, 2024, 1:43
6
Ok, je vois ce qui s’est passé ici - merci pour l’info @merefield
main ← widget-model
opened 01:41PM - 14 Feb 24 UTC
Changes in 0948d6b19e34320f95226cc43fbce976b71ceb31 caused the post-bottom-ad co… mponent to start receiving the widget-ised 'transformedPost' instead of the original `post` model. In most cases this difference didn't matter, but it did cause noticable issues with `@model.category` and `@model.topic`.
This commit also deletes the unused `post-bottom/discourse-adplugin.hbs` connector. In the past, the adplugin itself was defining the post-bottom outlet. But now, we use RenderGlimmer to load the `post-bottom-ad` directly.
Deux choses :
Lors d’un refactoring il y a quelques mois, nous avons involontairement commencé à passer le ‘transformedPost’ au lieu de ‘post’ dans post-bottom-ad
Lors du même refactoring, nous avons laissé traîner un ancien fichier post-bottom/discourse-adplugin.hbs. Il ne faisait rien en réalité, mais cela a rendu ce problème beaucoup plus difficile à résoudre
4 « J'aime »
David Taylor:
difficile à comprendre
C’est la description parfaite
L’architecture de l’interface utilisateur du flux de publication est assez redoutable !
Merci beaucoup d’avoir examiné cela !!
6 « J'aime »
david
(David Taylor)
A fermé ce sujet ()
Février 19, 2024, 8:00
8
Ce sujet a été automatiquement fermé après 4 jours. Les nouvelles réponses ne sont plus autorisées.