refactor: Rework character_limiter()

Author: neznaika0

system/Helpers/text_helper.php

@@ -43,35 +43,40 @@ function word_limiter(string $str, int $limit = 100, string $endChar = '…'
     /**
      * Character Limiter
      *
-     * Limits the string based on the character count.  Preserves complete words
+     * Limits the string based on the character count. Preserves complete words
      * so the character count may not be exactly as specified.
      *
      * @param string $endChar the end character. Usually an ellipsis
      */
-    function character_limiter(string $str, int $n = 500, string $endChar = '…'): string
+    function character_limiter(string $string, int $limit = 500, string $endChar = '…'): string
     {
-        if (mb_strlen($str) < $n) {
-            return $str;
+        if (mb_strlen($string) < $limit) {
+            return $string;
         }
 
         // a bit complicated, but faster than preg_replace with \s+
-        $str = preg_replace('/ {2,}/', ' ', str_replace(["\r", "\n", "\t", "\x0B", "\x0C"], ' ', $str));
+        $string       = preg_replace('/ {2,}/', ' ', str_replace(["\r", "\n", "\t", "\x0B", "\x0C"], ' ', $string));
+        $stringLength = mb_strlen($string);
 
-        if (mb_strlen($str) <= $n) {
-            return $str;
+        if ($stringLength <= $limit) {
+            return $string;
         }
 
-        $out = '';
+        $output       = '';
+        $outputLength = 0;
+        $words        = explode(' ', trim($string));
 
-        foreach (explode(' ', trim($str)) as $val) {
-            $out .= $val . ' ';
-            if (mb_strlen($out) >= $n) {
-                $out = trim($out);
+        foreach ($words as $word) {
+            $output .= $word . ' ';
+            $outputLength = mb_strlen($output);
+
+            if ($outputLength >= $limit) {
+                $output = trim($output);
                 break;
             }
         }
 
-        return (mb_strlen($out) === mb_strlen($str)) ? $out : $out . $endChar;
+        return ($outputLength === $stringLength) ? $output : $output . $endChar;
     }
 }
 
@@ -721,9 +726,9 @@ function excerpt(string $text, ?string $phrase = null, int $radius = 100, string
         $beforeWords = explode(' ', mb_substr($text, 0, $phrasePosition));
         $afterWords  = explode(' ', mb_substr($text, $phrasePosition + $phraseLength));
 
-        $firstPartOutput  = ' ';
-        $endPartOutput    = ' ';
-        $count = 0;
+        $firstPartOutput = ' ';
+        $endPartOutput   = ' ';
+        $count           = 0;
 
         foreach (array_reverse($beforeWords) as $beforeWord) {
             $beforeWordLength = mb_strlen($beforeWord);

user_guide_src/source/helpers/text_helper.rst

@@ -166,11 +166,11 @@ The following functions are available:
 
     .. literalinclude:: text_helper/012.php
 
-.. php:function:: word_limiter($str[, $limit = 100[, $end_char = '…']])
+.. php:function:: word_limiter($str[, $limit = 100[, $endChar = '…']])
 
     :param    string    $str: Input string
     :param    int    $limit: Limit
-    :param    string    $end_char: End character (usually an ellipsis)
+    :param    string    $endChar: End character (usually an ellipsis)
     :returns:    Word-limited string
     :rtype:    string
 
@@ -181,11 +181,11 @@ The following functions are available:
     The third parameter is an optional suffix added to the string. By
     default it adds an ellipsis.
 
-.. php:function:: character_limiter($str[, $n = 500[, $end_char = '…']])
+.. php:function:: character_limiter($string[, $limit = 500[, $endChar = '…']])
 
-    :param    string    $str: Input string
-    :param    int    $n: Number of characters
-    :param    string    $end_char: End character (usually an ellipsis)
+    :param    string    $string: Input string
+    :param    int    $limit: Number of characters
+    :param    string    $endChar: End character (usually an ellipsis)
     :returns:    Character-limited string
     :rtype:    string