Brave Linux, thème Meta. Je ne sais pas si c’est intentionnel, mais cela n’a pas l’air correct
2 « J'aime »
bryce
(Bryce Huhtala)
Août 13, 2024, 12:53
2
Hmm, je vois la même chose sur le thème par défaut.
Il y a un élément pour le nom, mais il est vide !
Cela ne me semble pas intentionnel, je vais donc le déplacer dans Bug pour le moment.
5 « J'aime »
Voici un aperçu rapide pour aider à résoudre le problème avant de se coucher.
https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/helpers/category-link.js#L159
displayName n’existe pas. Il y a un champ name à la place.
Les changements ont été effectués dans ce commit :
committed 03:33PM - 08 Aug 24 UTC
This commit adds two new getters to the category model:
- `displayName`
- `des… criptionText`
These getters are used instead of `name` and `description_text` where appropriate.
On top of this two transformers have been added to allow plugins to alter these getters:
```javascript
api.registerValueTransformer(
"category-display-name",
({ value, context }) =>
value + "-" + context.category.id + "-transformed"
);
```
```javascript
api.registerValueTransformer(
"category-description-text",
({ value, context }) =>
value + "-" + context.category.id + "-transformed"
);
```
Il semble que les getters soient manquants dans le contexte du résumé utilisateur, car les données sont récupérées à partir de /summary.json.
https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/models/user.js#L1068-L1073
Peut-être que mapper top_categories avec Category.create() ou une classe spécifique de base serait acceptable.
4 « J'aime »
Moin
Août 13, 2024, 5:20
4
Cela me fait penser à Category names missing on group permissions page . Les deux ont perdu le texte, donc c’est peut-être le même problème.
2 « J'aime »
Il semble que cela pourrait être le résultat du même problème, mais dans un domaine différent. Je ne vais pas prétendre savoir comment fonctionne Discourse, donc je ne peux pas vraiment donner mon avis là-dessus.
1 « J'aime »
@j.jaffeux ceci et Category names missing on group permissions page ressemblent à des régressions de DEV: allows to alter category name/description (#28263) · discourse/discourse@c197daa · GitHub
categoryName renvoie une chaîne vide, on dirait que ces quelques endroits ne fonctionnent pas avec le nouveau getter displayName ?
4 « J'aime »
j.jaffeux
(Joffrey Jaffeux)
Août 13, 2024, 10:05
7
Cela devrait résoudre le problème :
main ← jjaffeux:name-description
opened 09:37PM - 13 Aug 24 UTC
`defaultCategoryLinkRenderer` is using a fake category object which doesn’t have… access to the functions and getters of category model.
This had been incorrectly set in https://github.com/discourse/discourse/commit/c197daa04c3114184cbd3ec2f1cfade1b5701793
As we don't get a real category object, we have to call the transformers manually and also pass the fake category object as context, this is not ideal as people might try to access properties in the transformer which are not available on the category object given they will be different based on the context. Hopefully one day this helper and all the chain can be refactored to use a real category model.
This commit also adds tests for these two properties in the category-link helper.
6 « J'aime »
j.jaffeux
(Joffrey Jaffeux)
A fermé ce sujet ()
Août 16, 2024, 10:05
8
Ce sujet a été automatiquement fermé 3 jours après la dernière réponse. Les nouvelles réponses ne sont plus autorisées.