From 8bdd23e1ccef6d19e18619b9b47f12e0a622954d Mon Sep 17 00:00:00 2001
From: Simon Backx <simon@ghost.org>
Date: Wed, 12 Apr 2023 15:53:17 +0200
Subject: [PATCH] Improved bulk delete and unpublish modals copy

refs https://github.com/TryGhost/Team/issues/2677
---
 ghost/admin/app/components/posts-list/context-menu.js    | 9 +++------
 .../app/components/posts-list/modals/delete-posts.hbs    | 6 +++---
 .../components/posts-list/modals/edit-posts-access.hbs   | 2 +-
 .../app/components/posts-list/modals/unpublish-posts.hbs | 4 ++--
 ghost/admin/app/utils/selection-list.js                  | 4 ++++
 5 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/ghost/admin/app/components/posts-list/context-menu.js b/ghost/admin/app/components/posts-list/context-menu.js
index 34c4e54f79..158caa3e58 100644
--- a/ghost/admin/app/components/posts-list/context-menu.js
+++ b/ghost/admin/app/components/posts-list/context-menu.js
@@ -27,8 +27,7 @@ export default class PostsContextMenu extends Component {
     async deletePosts() {
         this.menu.close();
         await this.modals.open(DeletePostsModal, {
-            isSingle: this.selectionList.isSingle,
-            count: this.selectionList.count,
+            selectionList: this.selectionList,
             confirm: this.deletePostsTask
         });
     }
@@ -37,8 +36,7 @@ export default class PostsContextMenu extends Component {
     async unpublishPosts() {
         this.menu.close();
         await this.modals.open(UnpublishPostsModal, {
-            isSingle: this.selectionList.isSingle,
-            count: this.selectionList.count,
+            selectionList: this.selectionList,
             confirm: this.unpublishPostsTask
         });
     }
@@ -47,8 +45,7 @@ export default class PostsContextMenu extends Component {
     async editPostsAccess() {
         this.menu.close();
         await this.modals.open(EditPostsAccessModal, {
-            isSingle: this.selectionList.isSingle,
-            count: this.selectionList.count,
+            selectionList: this.selectionList,
             confirm: this.editPostsAccessTask
         });
     }
diff --git a/ghost/admin/app/components/posts-list/modals/delete-posts.hbs b/ghost/admin/app/components/posts-list/modals/delete-posts.hbs
index 8a38224ae3..089ed0154a 100644
--- a/ghost/admin/app/components/posts-list/modals/delete-posts.hbs
+++ b/ghost/admin/app/components/posts-list/modals/delete-posts.hbs
@@ -1,13 +1,13 @@
 <div class="modal-content" data-test-modal="delete-posts">
     <header class="modal-header">
-        <h1>Are you sure you want to delete {{if @data.isSingle 'this post' (concat @data.count ' posts')}}?</h1>
+        <h1>Are you sure you want to delete {{if @data.selectionList.isSingle 'this post' (concat @data.selectionList.count ' posts')}}?</h1>
     </header>
     <button type="button" class="close" title="Close" {{on "click" (fn @close false)}} data-test-button="close">{{svg-jar "close"}}<span class="hidden">Close</span></button>
 
     <div class="modal-body">
         <p>
-            You're about to delete {{if @data.isSingle 'this post' (concat @data.count ' posts')}}.
-            This is irreversible.
+            You're about to delete <strong>{{if @data.selectionList.isSingle (concat '"' @data.selectionList.first.title '"') (concat @data.selectionList.count ' posts')}}</strong>.
+            This is permanent! We warned you, k?
         </p>
     </div>
 
diff --git a/ghost/admin/app/components/posts-list/modals/edit-posts-access.hbs b/ghost/admin/app/components/posts-list/modals/edit-posts-access.hbs
index 52aaa730d2..a4e05e5157 100644
--- a/ghost/admin/app/components/posts-list/modals/edit-posts-access.hbs
+++ b/ghost/admin/app/components/posts-list/modals/edit-posts-access.hbs
@@ -1,6 +1,6 @@
 <div class="modal-content" data-test-modal="edit-posts-access">
     <header class="modal-header">
-        <h1>Modify access for {{if @data.isSingle 'this post' (concat @data.count ' posts')}}</h1>
+        <h1>Modify access for {{if @data.selectionList.isSingle 'this post' (concat @data.selectionList.count ' posts')}}</h1>
     </header>
     <button type="button" class="close" title="Close" {{on "click" (fn @close false)}} data-test-button="close">{{svg-jar "close"}}<span class="hidden">Close</span></button>
 
diff --git a/ghost/admin/app/components/posts-list/modals/unpublish-posts.hbs b/ghost/admin/app/components/posts-list/modals/unpublish-posts.hbs
index 8c50dafe25..9111960837 100644
--- a/ghost/admin/app/components/posts-list/modals/unpublish-posts.hbs
+++ b/ghost/admin/app/components/posts-list/modals/unpublish-posts.hbs
@@ -1,12 +1,12 @@
 <div class="modal-content" data-test-modal="unpublish-posts">
     <header class="modal-header">
-        <h1>Are you sure you want to unpublish {{if @data.isSingle 'this post' (concat @data.count ' posts')}}?</h1>
+        <h1>Are you sure you want to unpublish {{if @data.selectionList.isSingle 'this post' (concat @data.selectionList.count ' posts')}}?</h1>
     </header>
     <button type="button" class="close" title="Close" {{on "click" (fn @close false)}} data-test-button="close">{{svg-jar "close"}}<span class="hidden">Close</span></button>
 
     <div class="modal-body">
         <p>
-            You're about to unpublish {{if @data.isSingle 'this post' (concat @data.count ' posts')}}.
+            You're about to unpublish <strong>{{if @data.selectionList.isSingle (concat '"' @data.selectionList.first.title '"') (concat @data.selectionList.count ' posts')}}</strong>.
         </p>
     </div>
 
diff --git a/ghost/admin/app/utils/selection-list.js b/ghost/admin/app/utils/selection-list.js
index 912662ede6..9a65e6777c 100644
--- a/ghost/admin/app/utils/selection-list.js
+++ b/ghost/admin/app/utils/selection-list.js
@@ -67,6 +67,10 @@ export default class SelectionList {
         return arr;
     }
 
+    get first() {
+        return this.availableModels[0];
+    }
+
     get isSingle() {
         return this.selectedIds.size === 1 && !this.inverted;
     }