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

Use WP functions to get content and excerpt

Browse Source 
[MAILPOET-3671]
This commit is contained in:
Jan Lysý
2021-10-01 09:46:29 +02:00
committed by Veljko V
parent 47bba6799f
commit 4a3aecbf5c
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
lib/Newsletter/Editor/PostContentManager.php
View File

@@ -14,11 +14,14 @@ class PostContentManager {
/** @var WooCommerceHelper */ /** @var WooCommerceHelper */
private $woocommerceHelper; private $woocommerceHelper;
/** @var WPFunctions */
private $wp;
public function __construct( public function __construct(
WooCommerceHelper $woocommerceHelper = null WooCommerceHelper $woocommerceHelper = null
) { ) {
$wp = new WPFunctions; $this->wp = new WPFunctions;
$this->maxExcerptLength = $wp->applyFilters('mailpoet_newsletter_post_excerpt_length', $this->maxExcerptLength); $this->maxExcerptLength = $this->wp->applyFilters('mailpoet_newsletter_post_excerpt_length', $this->maxExcerptLength);
$this->woocommerceHelper = $woocommerceHelper ?: new WooCommerceHelper(); $this->woocommerceHelper = $woocommerceHelper ?: new WooCommerceHelper();
} }
@@ -26,19 +29,19 @@ class PostContentManager {
if ($displayType === 'titleOnly') { if ($displayType === 'titleOnly') {
return ''; return '';
} }
if ($this->woocommerceHelper->isWooCommerceActive() && $post->post_type === 'product') { // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps if ($this->woocommerceHelper->isWooCommerceActive() && $this->wp->getPostType($post) === 'product') {
$product = $this->woocommerceHelper->wcGetProduct($post->ID); $product = $this->woocommerceHelper->wcGetProduct($post->ID);
if ($product) { if ($product) {
return $this->getContentForProduct($product, $displayType); return $this->getContentForProduct($product, $displayType);
} }
} }
if ($displayType === 'excerpt') { if ($displayType === 'excerpt') {
if (!empty($post->post_excerpt)) { // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps if ($this->wp->hasExcerpt($post)) {
return self::stripShortCodes($post->post_excerpt); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps return self::stripShortCodes($this->wp->getTheExcerpt($post));
} }
return $this->generateExcerpt($post->post_content); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps return $this->generateExcerpt($this->wp->getTheContent(null, false, $post));
} }
return self::stripShortCodes($post->post_content); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps return self::stripShortCodes($this->wp->getTheContent(null, false, $post));
} }
public function filterContent($content, $displayType, $withPostClass = true) { public function filterContent($content, $displayType, $withPostClass = true) {