diff --git a/ghost/admin/app/models/newsletter.js b/ghost/admin/app/models/newsletter.js index fedb47ef6c..1777e589e2 100644 --- a/ghost/admin/app/models/newsletter.js +++ b/ghost/admin/app/models/newsletter.js @@ -38,7 +38,7 @@ export default class Newsletter extends Model.extend(ValidationEngine) { * The filter that we should use to filter out members that are subscribed to this newsletter */ get recipientFilter() { - const idFilter = 'newsletters.id:' + this.id; + const idFilter = 'newsletters.slug:' + this.slug; if (this.visibility === 'paid') { return idFilter + '+status:-free'; } diff --git a/ghost/admin/tests/acceptance/editor-test.js b/ghost/admin/tests/acceptance/editor-test.js index 394db334a5..60f4a86d0c 100644 --- a/ghost/admin/tests/acceptance/editor-test.js +++ b/ghost/admin/tests/acceptance/editor-test.js @@ -878,8 +878,7 @@ describe('Acceptance: Editor', function () { // Enable stripe to also show paid members breakdown enableStripe(this.server); - // Note: we need to set the ID of a newsletter to some string value because of how NQL filters work. - const newsletter = this.server.create('newsletter', {status: 'active', name: 'test newsletter', id: 'test-newsletter'}); + const newsletter = this.server.create('newsletter', {status: 'active', name: 'test newsletter', slug: 'test-newsletter'}); this.server.createList('member', 4, {status: 'free', newsletters: [newsletter]}); this.server.createList('member', 2, {status: 'paid', newsletters: [newsletter]});