Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix code issues

Browse Source 
[MAILPOET-1145]
This commit is contained in:
Pavel Dohnal
2018-02-20 09:39:14 +00:00
parent 989baf2aa4
commit adb579a26c
4 changed files with 18 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
assets/js/src/handlebars_helpers.js
View File

@@ -24,7 +24,7 @@ define('handlebars_helpers', ['handlebars'], function (Handlebars) {
// set date format
f = block.hash.format || 'MMM Do, YYYY';
// check if we passed a timestamp
if (/^\d+$/.test(timestamp)) {
if (/^\s*\d+\s*$/.test(timestamp)) {
return window.moment.unix(timestamp).format(f);
}
return window.moment.utc(timestamp).format(f);

12
assets/js/src/mp2migrator.js
View File

@@ -46,8 +46,10 @@ define('mp2migrator', ['mailpoet', 'jquery'], function (mp, jQuery) {
jQuery('#logger').append('<span class="error_msg">' + MailPoet.MP2Migrator.fatal_error + '</span>' + '<br />\n');
}).always(function () {
if (MailPoet.MP2Migrator.is_logging) {
MailPoet.MP2Migrator.displayLogs_timeout
= setTimeout(MailPoet.MP2Migrator.displayLogs, 1000);
MailPoet.MP2Migrator.displayLogs_timeout = setTimeout(
MailPoet.MP2Migrator.displayLogs,
1000
);
}
});
},
@@ -71,8 +73,10 @@ define('mp2migrator', ['mailpoet', 'jquery'], function (mp, jQuery) {
jQuery('#logger-container').show();
}
if (MailPoet.MP2Migrator.is_logging) {
MailPoet.MP2Migrator.updateProgressbar_timeout
= setTimeout(MailPoet.MP2Migrator.updateProgressbar, 1000);
MailPoet.MP2Migrator.updateProgressbar_timeout = setTimeout(
MailPoet.MP2Migrator.updateProgressbar,
1000
);
}
});
},

5
assets/js/src/newsletter_editor/behaviors/ContainerDropZoneBehavior.js
View File

@@ -20,9 +20,10 @@ define([
columnLimit: 3
},
onRender: function () {
var dragAndDropDisabled =
var dragAndDropDisabled = (
_.isObject(this.view.options.renderOptions)
&& this.view.options.renderOptions.disableDragAndDrop === true;
&& this.view.options.renderOptions.disableDragAndDrop === true
);
if (!dragAndDropDisabled) {
this.addDropZone();
}

9
assets/js/src/subscribers/importExport/import.js
View File

@@ -247,9 +247,12 @@ define(
// if the header options is set, there should be at least
// 2 data rows, otherwise at least 1 data row
if (
(processedSubscribers && (isHeaderFound && processedSubscribers.length >= 2))
||
(!isHeaderFound && processedSubscribers.length >= 1)
processedSubscribers &&
(
(isHeaderFound && processedSubscribers.length >= 2)
||
(!isHeaderFound && processedSubscribers.length >= 1)
)
) {
// since we assume that the header line is always present, we need
// to detect the header by checking if it contains a valid e-mail address