diff --git a/assets/js/src/newsletters/listings/standard.jsx b/assets/js/src/newsletters/listings/standard.jsx index 021d0ba92e..f6e9569e4d 100644 --- a/assets/js/src/newsletters/listings/standard.jsx +++ b/assets/js/src/newsletters/listings/standard.jsx @@ -97,6 +97,17 @@ const bulk_actions = [ }, ]; +const confirmEdit = (newsletter) => { + if( + !newsletter.queue + || newsletter.status != 'sending' + || newsletter.queue.status !== null + || window.confirm(MailPoet.I18n.t('confirmEdit')) + ) { + window.location.href = `?page=mailpoet-newsletter-editor&id=${ newsletter.id }`; + } +}; + let newsletter_actions = [ { name: 'view', @@ -111,15 +122,7 @@ let newsletter_actions = [ { name: 'edit', label: MailPoet.I18n.t('edit'), - onClick: (newsletter) => { - if( - !newsletter.queue - || newsletter.queue.status != 'scheduled' - || window.confirm(MailPoet.I18n.t('confirmEdit')) - ) { - window.location.href = `?page=mailpoet-newsletter-editor&id=${ newsletter.id }`; - } - }, + onClick: confirmEdit, }, { name: 'duplicate', @@ -175,7 +178,8 @@ const NewsletterListStandard = React.createClass({ confirmEdit(newsletter)} >{ newsletter.queue.newsletter_rendered_subject || newsletter.subject } { actions } diff --git a/assets/js/src/newsletters/send.jsx b/assets/js/src/newsletters/send.jsx index 5094aab958..82b5aeb50b 100644 --- a/assets/js/src/newsletters/send.jsx +++ b/assets/js/src/newsletters/send.jsx @@ -274,7 +274,9 @@ define( />   { - this.state.item.queue && this.state.item.queue.status == 'paused' + this.state.item.status == 'sending' + && this.state.item.queue + && this.state.item.queue.status == 'paused' ?