Les articles des workspaces ne sont pas triés par date de modification

[thom4parisot]

Contraitement aux articles du workspace personnel.

Je ne comprends pas trop la logique de la méthode workspaceResolver/articles() :

stylo/graphql/resolvers/workspaceResolver.js

Lines 121 to 126 in d699f90

	async articles (workspace, _args, context) {
	const articles = (await Promise.all(workspace.articles.map((articleId) => context.loaders.articles.load(articleId))))
	.filter((a) => a !== undefined) // remove deleted articles
	articles.sort((a, b) => a.createdAt > b.createdAt ? -1 : 1)
	return Article.complete(articles, context.loaders)
	},

Pourquoi ça fonctionne différemment de articleResolver/articles() ?

stylo/graphql/resolvers/articleResolver.js

Lines 295 to 303 in d699f90

	async articles (_root, args, context) {
	const { userId } = isUser(args, context)
	return Article.getArticles({
	filter: { $or: [{ owner: userId }, { contributors: { $elemMatch: { user: userId } } }] },
	loaders: context.loaders
	}
	)
	},
	},

[ggrossetie]

Je pense que c'est parce que les articles n'ont pas de lien avec les espaces de travail. On récupère la liste des identifiants des articles sur l'espace de travail et on charge les articles associés (jointure en mémoire avec les loaders).

Concernant le tri, je ne sais pas trop pourquoi on utilise la date de création plutôt que la date de modification, c'était peut être une demande ou une incohérence...

Enfin, pour le filtre sur les articles supprimés, on devrait surement s'assurer qu'on supprime le lien avec le ou les espaces de travail quand on supprime un article. Et surement faire une migration pour s'assurer qu'on a pas ce cas dans la base de données.

.filter((a) => a !== undefined) // remove deleted articles 

[thom4parisot]

Ah ouais, ok je comprends. Je vais regarder comment faire.

Y'a une raison qu'on gère le tags et les workspaces différemment ? Au sens où les tags sont des entités, et on les référence dans les articles. Alors que les workspaces référencent les articles.

[ggrossetie]

Les workspaces sont des conteneurs alors que les tags sont libellés qu'on appose sur des "objets". Je trouve ça plus logique d'avoir articles -> tags et workspaces -> articles.
Les articles sont aussi ordonnés dans le workspace, si on fait le lien au niveau des articles on ne perd cette notion d'ordre.