refactor: possible one liner in Helper

Author: ddevsr

system/Helpers/array_helper.php

@@ -72,12 +72,7 @@ function _array_search_dot(array $indexes, array $array)
             $answer = array_filter($answer, static fn ($value) => $value !== null);
 
             if ($answer !== []) {
-                if (count($answer) === 1) {
-                    // If array only has one element, we return that element for BC.
-                    return current($answer);
-                }
-
-                return $answer;
+                return count($answer) === 1 ? current($answer) : $answer;
             }
 
             return null;

system/Helpers/filesystem_helper.php

@@ -218,13 +218,7 @@ function get_filenames(
                 }
 
                 if ($includeDir || ! $object->isDir()) {
-                    if ($includePath === false) {
-                        $files[] = $basename;
-                    } elseif ($includePath === null) {
-                        $files[] = str_replace($sourceDir, '', $name);
-                    } else {
-                        $files[] = $name;
-                    }
+                    $files[] = $includePath === false ? $basename : ($includePath === null ? str_replace($sourceDir, '', $name) : $name);
                 }
             }
         } catch (Throwable $e) {

system/Helpers/form_helper.php

@@ -656,13 +656,7 @@ function set_radio(string $field, string $value = '', bool $default = false): st
 
         $postInput = $request->getPost($field);
 
-        if ($oldInput !== null) {
-            $input = $oldInput;
-        } elseif ($postInput !== null) {
-            $input = $postInput;
-        } else {
-            $input = $default;
-        }
+        $input = $oldInput !== null ? $oldInput : ($postInput !== null ? $postInput : $default);
 
         if (is_array($input)) {
             // Note: in_array('', array(0)) returns TRUE, do not use it

system/Helpers/html_helper.php

@@ -66,14 +66,7 @@ function _list(string $type = 'ul', $list = [], $attributes = '', int $depth = 0
         foreach ($list as $key => $val) {
             $out .= str_repeat(' ', $depth + 2) . '<li>';
 
-            if (! is_array($val)) {
-                $out .= $val;
-            } else {
-                $out .= $key
-                        . "\n"
-                        . _list($type, $val, '', $depth + 4)
-                        . str_repeat(' ', $depth + 2);
-            }
+            $out .= ! is_array($val) ? $val : $key . "\n" . _list($type, $val, '', $depth + 4) . str_repeat(' ', $depth + 2);
 
             $out .= "</li>\n";
         }
@@ -109,11 +102,7 @@ function img($src = '', bool $indexPage = false, $attributes = ''): string
 
         // Check for a relative URI
         if (! preg_match('#^([a-z]+:)?//#i', $src['src']) && strpos($src['src'], 'data:') !== 0) {
-            if ($indexPage === true) {
-                $img .= ' src="' . site_url($src['src']) . '"';
-            } else {
-                $img .= ' src="' . slash_item('baseURL') . $src['src'] . '"';
-            }
+            $img .= $indexPage === true ? ' src="' . site_url($src['src']) . '"' : ' src="' . slash_item('baseURL') . $src['src'] . '"';
 
             unset($src['src']);
         }
@@ -203,11 +192,7 @@ function script_tag($src = '', bool $indexPage = false): string
 
         foreach ($src as $k => $v) {
             if ($k === 'src' && ! preg_match('#^([a-z]+:)?//#i', $v)) {
-                if ($indexPage === true) {
-                    $script .= 'src="' . site_url($v) . '" ';
-                } else {
-                    $script .= 'src="' . slash_item('baseURL') . $v . '" ';
-                }
+                $script .= $indexPage === true ? 'src="' . site_url($v) . '" ' : 'src="' . slash_item('baseURL') . $v . '" ';
             } else {
                 // for attributes without values, like async or defer, use NULL.
                 $script .= $k . (null === $v ? ' ' : '="' . $v . '" ');
@@ -295,13 +280,7 @@ function video($src, string $unsupportedMessage = '', string $attributes = '', a
 
         $video = '<video';
 
-        if (_has_protocol($src)) {
-            $video .= ' src="' . $src . '"';
-        } elseif ($indexPage === true) {
-            $video .= ' src="' . site_url($src) . '"';
-        } else {
-            $video .= ' src="' . slash_item('baseURL') . $src . '"';
-        }
+        $video .= _has_protocol($src) ? ' src="' . $src . '"' : ($indexPage === true ? ' src="' . site_url($src) . '"' : ' src="' . slash_item('baseURL') . $src . '"');
 
         if ($attributes !== '') {
             $video .= ' ' . $attributes;
@@ -341,13 +320,7 @@ function audio($src, string $unsupportedMessage = '', string $attributes = '', a
 
         $audio = '<audio';
 
-        if (_has_protocol($src)) {
-            $audio .= ' src="' . $src . '"';
-        } elseif ($indexPage === true) {
-            $audio .= ' src="' . site_url($src) . '"';
-        } else {
-            $audio .= ' src="' . slash_item('baseURL') . $src . '"';
-        }
+        $audio .= _has_protocol($src) ? ' src="' . $src . '"' : ($indexPage === true ? ' src="' . site_url($src) . '"' : ' src="' . slash_item('baseURL') . $src . '"');
 
         if ($attributes !== '') {
             $audio .= ' ' . $attributes;

system/Helpers/text_helper.php

@@ -387,11 +387,7 @@ function word_wrap(string $str, int $charlim = 76): string
 
             // If $temp contains data it means we had to split up an over-length
             // word into smaller chunks so we'll add it back to our current line
-            if ($temp !== '') {
-                $output .= $temp . "\n" . $line . "\n";
-            } else {
-                $output .= $line . "\n";
-            }
+            $output .= $temp !== '' ? $temp . "\n" . $line . "\n" : $line . "\n";
         }
 
         // Put our markers back
@@ -430,11 +426,7 @@ function ellipsize(string $str, int $maxLength, $position = 1, string $ellipsis
         $beg      = mb_substr($str, 0, (int) floor($maxLength * $position));
         $position = ($position > 1) ? 1 : $position;
 
-        if ($position === 1) {
-            $end = mb_substr($str, 0, -($maxLength - mb_strlen($beg)));
-        } else {
-            $end = mb_substr($str, -($maxLength - mb_strlen($beg)));
-        }
+        $end = $position === 1 ? mb_substr($str, 0, -($maxLength - mb_strlen($beg))) : mb_substr($str, -($maxLength - mb_strlen($beg)));
 
         return $beg . $ellipsis . $end;
     }