Merge pull request #657 from ivanhrytsaim/12976-Blog-meta-description-fix

12976-Blog-meta-description-fix

Author: magefan

Model/Author.php

@@ -114,13 +114,17 @@ public function getMetaDescription()
             $desc = $this->getShortContentExtractor()->execute($this->getData('content'), 500);
         }
 
-        $stylePattern = "~\<style(.*)\>(.*)\<\/style\>~";
+        $stylePattern = "~<style\b[^>]*>.*?</style>~is";
         $desc = preg_replace($stylePattern, '', $desc);
         $desc = trim(strip_tags((string)$desc));
         $desc = str_replace(["\r\n", "\n\r", "\r", "\n"], ' ', $desc);
 
-        if (mb_strlen($desc) > 200) {
-            $desc = mb_substr($desc, 0, 200);
+        if (mb_strlen($desc) > 160) {
+            $desc = mb_substr($desc, 0, 160);
+            $lastSpace = mb_strrpos($desc, ' ');
+            if ($lastSpace !== false) {
+                $desc = mb_substr($desc, 0, $lastSpace) . '...';
+            }
         }
 
         return trim($desc);

Model/Category.php

@@ -398,13 +398,17 @@ public function getMetaDescription()
             $desc = $this->getShortContentExtractor()->execute($this->getData('content'), 500);
         }
 
-        $stylePattern = "~\<style(.*)\>(.*)\<\/style\>~";
+        $stylePattern = "~<style\b[^>]*>.*?</style>~is";
         $desc = preg_replace($stylePattern, '', $desc);
         $desc = trim(strip_tags((string)$desc));
         $desc = str_replace(["\r\n", "\n\r", "\r", "\n"], ' ', $desc);
 
-        if (mb_strlen($desc) > 200) {
-            $desc = mb_substr($desc, 0, 200);
+        if (mb_strlen($desc) > 160) {
+            $desc = mb_substr($desc, 0, 160);
+            $lastSpace = mb_strrpos($desc, ' ');
+            if ($lastSpace !== false) {
+                $desc = mb_substr($desc, 0, $lastSpace);
+            }
         }
 
         return trim($desc);

Model/Post.php

@@ -584,13 +584,17 @@ public function getMetaDescription()
                 $desc = $this->getShortFilteredContent(500);
             }
 
-            $stylePattern = "~\<style(.*)\>(.*)\<\/style\>~";
+            $stylePattern = "~<style\b[^>]*>.*?</style>~is";
             $desc = preg_replace($stylePattern, '', $desc);
             $desc = trim(strip_tags((string)$desc));
             $desc = str_replace(["\r\n", "\n\r", "\r", "\n"], ' ', $desc);
 
-            if (mb_strlen($desc) > 200) {
-                $desc = mb_substr($desc, 0, 200);
+            if (mb_strlen($desc) > 160) {
+                $desc = mb_substr($desc, 0, 160);
+                $lastSpace = mb_strrpos($desc, ' ');
+                if ($lastSpace !== false) {
+                    $desc = mb_substr($desc, 0, $lastSpace) . '...';
+                }
             }
 
             $desc = trim($desc);

Model/Tag.php

@@ -187,13 +187,17 @@ public function getMetaDescription()
             $desc = $this->getShortContentExtractor()->execute($this->getData('content'), 500);
         }
 
-        $stylePattern = "~\<style(.*)\>(.*)\<\/style\>~";
+        $stylePattern = "~<style\b[^>]*>.*?</style>~is";
         $desc = preg_replace($stylePattern, '', $desc);
         $desc = trim(strip_tags((string)$desc));
         $desc = str_replace(["\r\n", "\n\r", "\r", "\n"], ' ', $desc);
 
-        if (mb_strlen($desc) > 200) {
-            $desc = mb_substr($desc, 0, 200);
+        if (mb_strlen($desc) > 160) {
+            $desc = mb_substr($desc, 0, 160);
+            $lastSpace = mb_strrpos($desc, ' ');
+            if ($lastSpace !== false) {
+                $desc = mb_substr($desc, 0, $lastSpace) . '...';
+            }
         }
 
         return trim($desc);