diff --git a/assets/js/src/newsletter_editor/components/save.js b/assets/js/src/newsletter_editor/components/save.js index 0c61d4276d..8d62556d99 100644 --- a/assets/js/src/newsletter_editor/components/save.js +++ b/assets/js/src/newsletter_editor/components/save.js @@ -266,7 +266,6 @@ define([ }, validateNewsletter: function (jsonObject) { var body = ''; - var contents; if (!App._contentContainer.isValid()) { this.showValidationError(App._contentContainer.validationError); return; @@ -282,10 +281,9 @@ define([ return; } - contents = JSON.stringify(jsonObject); if ((App.getNewsletter().get('type') === 'notification') && - contents.indexOf('"type":"automatedLatestContent"') < 0 && - contents.indexOf('"type":"automatedLatestContentLayout"') < 0 + body.indexOf('"type":"automatedLatestContent"') < 0 && + body.indexOf('"type":"automatedLatestContentLayout"') < 0 ) { this.showValidationError(MailPoet.I18n.t('automatedLatestContentMissing')); return; diff --git a/tests/javascript/newsletter_editor/components/save.spec.js b/tests/javascript/newsletter_editor/components/save.spec.js index 6eb7178289..3efb5b5caa 100644 --- a/tests/javascript/newsletter_editor/components/save.spec.js +++ b/tests/javascript/newsletter_editor/components/save.spec.js @@ -129,10 +129,12 @@ define([ var newsletter = { get: sinon.stub().withArgs('type').returns('notification') }; EditorApplication.getNewsletter = sinon.stub().returns(newsletter); view.validateNewsletter({ - content: { - blocks: [ - { type: 'automatedLatestContent' } - ] + body: { + content: { + blocks: [ + { type: 'automatedLatestContent' } + ] + } } }); expect(hideValidationErrorStub.callCount).to.be.equal(1);