From a4061704a4864d454e264a71686948b6ec58c836 Mon Sep 17 00:00:00 2001
From: Kevin Ansfield <kevin@lookingsideways.co.uk>
Date: Tue, 27 Mar 2018 20:57:59 +0100
Subject: [PATCH] Fixed authors filter for posts with multiple authors (#993)

no issue
- we need `?filter=authors:foo` to match all posts where `foo` is an author, the previous `?filter=author:foo` will only match posts where `foo` is the primary author
---
 ghost/admin/app/routes/posts.js              | 6 +++---
 ghost/admin/tests/acceptance/content-test.js | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/ghost/admin/app/routes/posts.js b/ghost/admin/app/routes/posts.js
index 4aca9331f3..cd0dd43912 100644
--- a/ghost/admin/app/routes/posts.js
+++ b/ghost/admin/app/routes/posts.js
@@ -46,13 +46,13 @@ export default AuthenticatedRoute.extend(InfinityRoute, {
 
             if (user.get('isAuthor')) {
                 // authors can only view their own posts
-                filterParams.author = user.get('slug');
+                filterParams.authors = user.get('slug');
             } else if (user.get('isContributor')) {
                 // Contributors can only view their own draft posts
-                filterParams.author = user.get('slug');
+                filterParams.authors = user.get('slug');
                 queryParams.status = 'draft';
             } else if (params.author) {
-                filterParams.author = params.author;
+                filterParams.authors = params.author;
             }
 
             let filter = this._filterString(filterParams);
diff --git a/ghost/admin/tests/acceptance/content-test.js b/ghost/admin/tests/acceptance/content-test.js
index 12ca6c4015..4665c7d778 100644
--- a/ghost/admin/tests/acceptance/content-test.js
+++ b/ghost/admin/tests/acceptance/content-test.js
@@ -112,7 +112,7 @@ describe('Acceptance: Content', function () {
             expect(lastRequest.queryParams.status, '"all" request status param').to.equal('all');
             expect(lastRequest.queryParams.staticPages, '"all" request staticPages param').to.equal('all');
             expect(lastRequest.queryParams.filter, '"editor" request filter param')
-                .to.equal(`author:${editor.slug}`);
+                .to.equal(`authors:${editor.slug}`);
 
             // Displays editor post
             // TODO: implement "filter" param support and fix mirage post->author association
@@ -151,7 +151,7 @@ describe('Acceptance: Content', function () {
 
             // API request includes author filter
             let [lastRequest] = server.pretender.handledRequests.slice(-1);
-            expect(lastRequest.queryParams.filter).to.equal(`author:${author.slug}`);
+            expect(lastRequest.queryParams.filter).to.equal(`authors:${author.slug}`);
 
             // only author's post is shown
             expect(find('[data-test-post-id]').length, 'post count').to.equal(1);
@@ -189,7 +189,7 @@ describe('Acceptance: Content', function () {
 
             // API request includes author filter
             let [lastRequest] = server.pretender.handledRequests.slice(-1);
-            expect(lastRequest.queryParams.filter).to.equal(`author:${contributor.slug}`);
+            expect(lastRequest.queryParams.filter).to.equal(`authors:${contributor.slug}`);
 
             // only contributor's post is shown
             expect(find('[data-test-post-id]').length, 'post count').to.equal(1);