jasny / twig-extensions

tests/TextExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -160,6 +160,6 @@
      */
     public function testWithNull($filter)
     {
-        $this->assertRender('-', '{{ null|' . $filter . '("//")|default("-") }}');
+        $this->assertRender('-', '{{ null|'.$filter.'("//")|default("-") }}');
     }    
 }

src/TextExtension.php

Spacing +17 added lines, -17 removed lines

@@ -43,7 +43,7 @@ 
             return null;
         }
         
-        return '<p>' . preg_replace(['~\n(\s*)\n\s*~', '~(?<!</p>)\n\s*~'], ["</p>\n\$1<p>", "<br>\n"], trim($value)) .
+        return '<p>'.preg_replace(['~\n(\s*)\n\s*~', '~(?<!</p>)\n\s*~'], ["</p>\n\$1<p>", "<br>\n"], trim($value)).
             '</p>';
     }
 
@@ -80,7 +80,7 @@ 
         }
         
         $pos = stripos($value, $break);
-        return $pos === false ? $value : substr($value, 0, $pos) . $replace;
+        return $pos === false ? $value : substr($value, 0, $pos).$replace;
     }
 
     /**
@@ -97,7 +97,7 @@ 
             return null;
         }
         
-        return strlen($value) <= $length ? $value : substr($value, 0, $length - strip_tags($replace)) . $replace;
+        return strlen($value) <= $length ? $value : substr($value, 0, $length - strip_tags($replace)).$replace;
     }
     
     /**
@@ -115,12 +115,12 @@ 
             ? '~(?:(https?)://([^\s<>]+)|(?<!\w@)\b(www\.[^\s<>]+?\.[^\s<>]+))(?<![\.,:;\?!\'"\|])~i'
             : '~(?:(https?)://([^\s<>]+)|(?<!\w@)\b([^\s<>@]+?\.[^\s<>]+)(?<![\.,:]))~i';
         
-        return preg_replace_callback($regexp, function ($match) use ($protocol, &$links, $attr) {
+        return preg_replace_callback($regexp, function($match) use ($protocol, &$links, $attr) {
             if ($match[1]) $protocol = $match[1];
             $link = $match[2] ?: $match[3];
             
-            return '<' . array_push($links, '<a' . $attr . ' href="' . $protocol . '://' . $link  . '">'
-                . rtrim($link, '/') . '</a>') . '>';
+            return '<'.array_push($links, '<a'.$attr.' href="'.$protocol.'://'.$link.'">'
+                . rtrim($link, '/').'</a>').'>';
         }, $text);
     }
     
@@ -135,8 +135,8 @@ 
     {
         $regexp = '~([^\s<>]+?@[^\s<>]+?\.[^\s<>]+)(?<![\.,:;\?!\'"\|])~';
         
-        return preg_replace_callback($regexp, function ($match) use (&$links, $attr) {
-            return '<' . array_push($links, '<a' . $attr . ' href="mailto:' . $match[1]  . '">' . $match[1] . '</a>')
+        return preg_replace_callback($regexp, function($match) use (&$links, $attr) {
+            return '<'.array_push($links, '<a'.$attr.' href="mailto:'.$match[1].'">'.$match[1].'</a>')
                 . '>';
         }, $text);
     }
@@ -154,16 +154,16 @@ 
     protected function linkifyOther($protocol, $text, array &$links, $attr, $mode)
     {
         if (strpos($protocol, ':') === false) {
-            $protocol .= in_array($protocol, ['ftp', 'tftp', 'ssh', 'scp'])  ? '://' : ':';
+            $protocol .= in_array($protocol, ['ftp', 'tftp', 'ssh', 'scp']) ? '://' : ':';
         }
         
         $regexp = $mode != 'all'
-            ? '~' . preg_quote($protocol, '~') . '([^\s<>]+)(?<![\.,:;\?!\'"\|])~i'
+            ? '~'.preg_quote($protocol, '~').'([^\s<>]+)(?<![\.,:;\?!\'"\|])~i'
             : '~([^\s<>]+)(?<![\.,:])~i';
         
-        return preg_replace_callback($regexp, function ($match) use ($protocol, &$links, $attr) {
-            return '<' . array_push($links, '<a' . $attr . ' href="' . $protocol . $match[1]  . '">' . $match[1]
-                . '</a>') . '>';
+        return preg_replace_callback($regexp, function($match) use ($protocol, &$links, $attr) {
+            return '<'.array_push($links, '<a'.$attr.' href="'.$protocol.$match[1].'">'.$match[1]
+                . '</a>').'>';
         }, $text);
     }
     
@@ -185,14 +185,14 @@ 
         // Link attributes
         $attr = '';
         foreach ($attributes as $key => $val) {
-            $attr .= ' ' . $key . '="' . htmlentities($val) . '"';
+            $attr .= ' '.$key.'="'.htmlentities($val).'"';
         }
         
         $links = [];
         
         // Extract existing links and tags
-        $text = preg_replace_callback('~(<a .*?>.*?</a>|<.*?>)~i', function ($match) use (&$links) {
-            return '<' . array_push($links, $match[1]) . '>';
+        $text = preg_replace_callback('~(<a .*?>.*?</a>|<.*?>)~i', function($match) use (&$links) {
+            return '<'.array_push($links, $match[1]).'>';
         }, $value);
         
         // Extract text links for each protocol
@@ -206,7 +206,7 @@ 
         }
         
         // Insert all link
-        return preg_replace_callback('/<(\d+)>/', function ($match) use (&$links) {
+        return preg_replace_callback('/<(\d+)>/', function($match) use (&$links) {
             return $links[$match[1] - 1];
         }, $text);
     }

Braces +3 added lines, -1 removed lines

@@ -116,7 +116,9 @@
             : '~(?:(https?)://([^\s<>]+)|(?<!\w@)\b([^\s<>@]+?\.[^\s<>]+)(?<![\.,:]))~i';
         
         return preg_replace_callback($regexp, function ($match) use ($protocol, &$links, $attr) {
-            if ($match[1]) $protocol = $match[1];
+            if ($match[1]) {
+                $protocol = $match[1];
+            }
             $link = $match[2] ?: $match[3];
             
             return '<' . array_push($links, '<a' . $attr . ' href="' . $protocol . '://' . $link  . '">'

src/DateExtension.php

Doc Comments +2 added lines, -1 removed lines

@@ -31,7 +31,7 @@ 
     /**
      * Callback for Twig to get all the filters.
      * 
-     * @return \Twig_Filter[]
+     * @return \Twig_SimpleFilter[]
      */
     public function getFilters()
     {
@@ -211,6 +211,7 @@ 
      * Split duration into seconds, minutes, hours, days, weeks and years.
      * 
      * @param int $seconds
+     * @param integer $max
      * @return array
      */
     protected function splitDuration($seconds, $max)

Spacing +10 added lines, -10 removed lines

@@ -112,7 +112,7 @@ 
         if (
             (is_int($datetype) && $datetype !== \IntlDateFormatter::NONE) ||
             (is_int($timetype) && $timetype !== \IntlDateFormatter::NONE)
-        ){
+        ) {
             $pattern = \IntlDateFormatter::create(
                 \Locale::getDefault(),
                 is_int($datetype) ? $datetype : \IntlDateFormatter::NONE,
@@ -125,11 +125,11 @@ 
         }
         
         if (is_string($datetype)) {
-            $pattern = trim($datetype . ' ' . $pattern);
+            $pattern = trim($datetype.' '.$pattern);
         }
 
         if (is_string($timetype)) {
-            $pattern = trim($pattern . ' ' . $timetype);
+            $pattern = trim($pattern.' '.$timetype);
         }
         
         return preg_replace('/\byy?\b/', 'yyyy', $pattern);
@@ -180,7 +180,7 @@ 
      * @param string              $calendar  'gregorian' or 'traditional'
      * @return string
      */
-    public function localTime($date, $format='short', $calendar='gregorian')
+    public function localTime($date, $format = 'short', $calendar = 'gregorian')
     {
         return $this->formatLocal($date, false, $format, $calendar);
     }
@@ -270,27 +270,27 @@ 
         
         $duration = '';
         if (isset($years) && isset($units[5])) {
-            $duration .= $seperator . $years . $units[5];
+            $duration .= $seperator.$years.$units[5];
         }
         
         if (isset($weeks) && isset($units[4])) {
-            $duration .= $seperator . $weeks . $units[4];
+            $duration .= $seperator.$weeks.$units[4];
         }
         
         if (isset($days) && isset($units[3])) {
-            $duration .= $seperator . $days . $units[3];
+            $duration .= $seperator.$days.$units[3];
         }
         
         if (isset($hours) && isset($units[2])) {
-            $duration .= $seperator . $hours . $units[2];
+            $duration .= $seperator.$hours.$units[2];
         }
         
         if (isset($minutes) && isset($units[1])) {
-            $duration .= $seperator . $minutes . $units[1];
+            $duration .= $seperator.$minutes.$units[1];
         }
         
         if (isset($seconds) && isset($units[0])) {
-            $duration .= $seperator . $seconds . $units[0];
+            $duration .= $seperator.$seconds.$units[0];
         }
         
         return trim($duration, $seperator);

src/ArrayExtension.php

Indentation +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
      */
     public function product($array)
     {
-       return isset($array) ? array_product((array)$array) : null;
+        return isset($array) ? array_product((array)$array) : null;
     }
     
     /**
@@ -63,7 +63,7 @@ 
      */
     public function values($array)
     {
-       return isset($array) ? array_values((array)$array) : null;
+        return isset($array) ? array_values((array)$array) : null;
     }
     
     /**

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
                 $value = $key;
             }
             
-            $str .= ' ' . $key . '="' . addcslashes($value, '"') . '"';
+            $str .= ' '.$key.'="'.addcslashes($value, '"').'"';
         }
         
         return trim($str);

tests/DateExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -160,6 +160,6 @@
      */
     public function testWithNull($filter)
     {
-        $this->assertRender('-', '{{ null|' . $filter . '("//")|default("-") }}');
+        $this->assertRender('-', '{{ null|'.$filter.'("//")|default("-") }}');
     }    
 }

tests/PcreExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -160,6 +160,6 @@
      */
     public function testWithNull($filter)
     {
-        $this->assertRender('-', '{{ null|' . $filter . '("//")|default("-") }}');
+        $this->assertRender('-', '{{ null|'.$filter.'("//")|default("-") }}');
     }    
 }

tests/ArrayExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -160,6 +160,6 @@
      */
     public function testWithNull($filter)
     {
-        $this->assertRender('-', '{{ null|' . $filter . '("//")|default("-") }}');
+        $this->assertRender('-', '{{ null|'.$filter.'("//")|default("-") }}');
     }    
 }