🐛 Bulk adding tags created "duplicate" posts_tags rows from editor po…

…sts page

#21414
The commit includes a migration which adds a unique index to posts_tags table and handles the related error when bulk adding posts.

The errors are ignored because the tags user wants to add are added/exist. This respects the editor's expectation as posts have the tags as the end result.

ghost/core/core/server/data/migrations/versions/5.100/2024-12-01-18-54-00-add-unique-tag-post-relationship.js

@@ -0,0 +1,9 @@
+const {createIrreversibleMigration} = require('../../utils');
+const {addUnique} = require('../../../schema/commands');
+const logging = require('@tryghost/logging');
+
+module.exports = createIrreversibleMigration(async (knex) => {
+    logging.info('Adding unique constraint to tag-post relationship');
+
+    await addUnique('posts_tags', ['tag_id', 'post_id'], knex);
+});

ghost/core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ghost",
-  "version": "5.101.6",
+  "version": "5.102.0",
   "description": "The professional publishing platform",
   "author": "Ghost Foundation",
   "homepage": "https://ghost.org",

ghost/posts-service/lib/PostsService.js

@@ -362,7 +362,7 @@ class PostsService {
             }));
         }, []);
 
-        await options.transacting('posts_tags').insert(postTags);
+        await options.transacting('posts_tags').insert(postTags).onConflict().ignore();
         await this.models.Post.addActions('edited', postRows.map(p => p.id), options);
 
         return {