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

Fix various regressions introduced by camel case conversion

Browse Source 
[MAILPOET-2635]
This commit is contained in:
Jan Jakeš
2020-01-16 14:26:19 +01:00
committed by Jack Kitterhing
parent d955a9105a
commit 2ef41b56e0
5 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Config/Updater.php
View File

@@ -37,7 +37,7 @@ class Updater {
if (version_compare($this->version, $latestVersion->new_version, '<')) { // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
$updateTransient->response[$this->plugin] = $latestVersion;
}
$updateTransient->lastChecked = time();
$updateTransient->last_checked = time(); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
$updateTransient->checked[$this->plugin] = $this->version;
}

2
lib/Newsletter/ViewInBrowser.php
View File

@@ -22,7 +22,7 @@ class ViewInBrowser {
public function view($data) {
$wpUserPreview = (
($data->subscriber && $data->subscriber->isWPUser() && $data->preview) ||
($data->preview && $data->newsletterHash)
($data->preview && $data->newsletter_hash) // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
);
return $this->renderNewsletter(
$data->newsletter,

4
tests/integration/Config/UpdaterTest.php
View File

@@ -38,7 +38,7 @@ class UpdaterTest extends \MailPoetTest {
public function testItChecksForUpdates() {
$updateTransient = new \stdClass;
$updateTransient->lastChecked = time();
$updateTransient->last_checked = time(); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
$updater = Stub::construct(
$this->updater,
[
@@ -61,7 +61,7 @@ class UpdaterTest extends \MailPoetTest {
$this
);
$result = $updater->checkForUpdate($updateTransient);
expect($result->lastChecked)->greaterOrEquals($updateTransient->lastChecked);
expect($result->last_checked)->greaterOrEquals($updateTransient->last_checked); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
expect($result->checked[$this->pluginName])->equals($this->version);
expect($result->response[$this->pluginName]->slug)->equals($this->slug);
expect($result->response[$this->pluginName]->plugin)->equals($this->pluginName);

2
tests/integration/Cron/Workers/SendingQueue/Tasks/PostsTest.php
View File

@@ -48,7 +48,7 @@ class PostsTest extends \MailPoetTest {
$postId = 10;
$newsletter = (object)[
'id' => 2,
'parent_id' => 1,
'parentId' => 1,
'type' => Newsletter::TYPE_WELCOME,
];
$renderedNewsletter = [

2
tests/integration/Router/Endpoints/ViewInBrowserTest.php
View File

@@ -126,7 +126,7 @@ class ViewInBrowserTest extends \MailPoetTest {
public function testItFailsValidationWhenPreviewIsEnabledButNewsletterHashNotProvided() {
$data = (object)$this->browserPreviewData;
$data->newsletterHash = false;
$data->newsletter_hash = false; // phpcs:ignore Squiz.NamingConventions.ValidVariableName.NotCamelCaps
$data->preview = true;
expect($this->viewInBrowser->_validateBrowserPreviewData($data))->false();
}