Remove unnecessary usages of $subscriber for preview links
[MAILPOET-2430]
This commit is contained in:
committed by
Jack Kitterhing
parent
b22198f599
commit
fb7e87706b
@@ -127,8 +127,7 @@ class Newsletters extends APIEndpoint {
|
|||||||
(object)[
|
(object)[
|
||||||
'id' => $newsletter->getId(),
|
'id' => $newsletter->getId(),
|
||||||
'hash' => $newsletter->getHash(),
|
'hash' => $newsletter->getHash(),
|
||||||
],
|
]
|
||||||
Subscriber::getCurrentWPUser()
|
|
||||||
);
|
);
|
||||||
|
|
||||||
$response = $this->wp->applyFilters('mailpoet_api_newsletters_get_after', $response);
|
$response = $this->wp->applyFilters('mailpoet_api_newsletters_get_after', $response);
|
||||||
@@ -157,8 +156,7 @@ class Newsletters extends APIEndpoint {
|
|||||||
(object)[
|
(object)[
|
||||||
'id' => $newsletter->getId(),
|
'id' => $newsletter->getId(),
|
||||||
'hash' => $newsletter->getHash(),
|
'hash' => $newsletter->getHash(),
|
||||||
],
|
]
|
||||||
Subscriber::getCurrentWPUser()
|
|
||||||
);
|
);
|
||||||
return $this->successResponse($response);
|
return $this->successResponse($response);
|
||||||
} else {
|
} else {
|
||||||
@@ -281,7 +279,7 @@ class Newsletters extends APIEndpoint {
|
|||||||
$this->wp->doAction('mailpoet_api_newsletters_save_after', $newsletter);
|
$this->wp->doAction('mailpoet_api_newsletters_save_after', $newsletter);
|
||||||
$this->authorizedEmailsController->onNewsletterUpdate($newsletter, $oldNewsletter);
|
$this->authorizedEmailsController->onNewsletterUpdate($newsletter, $oldNewsletter);
|
||||||
|
|
||||||
$previewUrl = NewsletterUrl::getViewInBrowserUrl($newsletter, Subscriber::getCurrentWPUser());
|
$previewUrl = NewsletterUrl::getViewInBrowserUrl($newsletter);
|
||||||
return $this->successResponse($newsletter->asArray(), ['preview_url' => $previewUrl]);
|
return $this->successResponse($newsletter->asArray(), ['preview_url' => $previewUrl]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user