tarsana / functional

src/operators.php

Braces +3 added lines, -2 removed lines

@@ -97,8 +97,9 @@
 function equals() {
     $equals = function($a, $b) {
             $type = type($a);
-            if ($type != type($b))
-                return false;
+            if ($type != type($b)) {
+                            return false;
+            }
             switch ($type) {
                 case 'Null':
                 case 'Boolean':

src/object.php

Braces +11 added lines, -8 removed lines

@@ -72,8 +72,9 @@ 
  */
 function attributes() {
     $attrs = function($object) {
-        if (is_object($object))
-            return get_object_vars($object);
+        if (is_object($object)) {
+                    return get_object_vars($object);
+        }
         return $object;
     };
     return apply(curry($attrs), func_get_args());
@@ -208,8 +209,9 @@ 
 function getPath() {
     $getPath = function($path, $object){
         return reduce(function($result, $name) {
-            if ($result !== null)
-                $result = get($name, $result);
+            if ($result !== null) {
+                            $result = get($name, $result);
+            }
             return $result;
         }, $object, $path);
     };
@@ -237,10 +239,11 @@ 
 function set() {
     $set = function($name, $value, $object) {
         $object = clone_($object);
-        if (is_object($object))
-            $object->{$name} = $value;
-        else
-            $object[$name] = $value;
+        if (is_object($object)) {
+                    $object->{$name} = $value;
+        } else {
+                    $object[$name] = $value;
+        }
         return $object;
     };
     return apply(curry($set), func_get_args());

build.php

Braces +52 added lines, -35 removed lines

@@ -148,14 +148,16 @@ 
 function write_module($module) {
     if ($module->docs) {
         $docsDir  = dirname($module->docsPath);
-        if (!is_dir($docsDir))
-            mkdir($docsDir, 0777, true);
+        if (!is_dir($docsDir)) {
+                    mkdir($docsDir, 0777, true);
+        }
         file_put_contents($module->docsPath,  $module->docs);
     }
     if ($module->tests) {
         $testsDir = dirname($module->testsPath);
-        if (!is_dir($testsDir))
-            mkdir($testsDir, 0777, true);
+        if (!is_dir($testsDir)) {
+                    mkdir($testsDir, 0777, true);
+        }
         file_put_contents($module->testsPath, $module->tests);
     }
     if ($module->streamOperations) {
@@ -252,9 +254,15 @@ 
     $tags = groupBy(get('name'), tags_of($doxBlock));
 
     $type = 'function';
-    if (has('file', $tags)) $type = 'file';
-    if (has('class', $tags)) $type = 'class';
-    if (has('method', $tags)) $type = 'method';
+    if (has('file', $tags)) {
+        $type = 'file';
+    }
+    if (has('class', $tags)) {
+        $type = 'class';
+    }
+    if (has('method', $tags)) {
+        $type = 'method';
+    }
 
     $params = map(function($tag){
         $parts = split(' ', get('value', $tag));
@@ -266,8 +274,9 @@ 
 
     $return = getPath(['return', 0, 'value'], $tags);
     $signatures = get('signature', $tags);
-    if ($signatures)
-        $signatures = map(get('value'), $signatures);
+    if ($signatures) {
+            $signatures = map(get('value'), $signatures);
+    }
     return (object) [
         'type' => $type,
         'name' => getPath(['ctx', 'name'], $doxBlock),
@@ -291,12 +300,13 @@ 
  * @return array
  */
 function tags_of($doxBlock) {
-    if ($doxBlock->tags)
-        return map(function($tag){
+    if ($doxBlock->tags) {
+            return map(function($tag){
             return (object) [
                 'name'  => $tag->type,
                 'value' => $tag->string
             ];
+    }
         }, $doxBlock->tags);
     return [];
 }
@@ -310,8 +320,9 @@ 
  */
 function generate_docs($module) {
     $module->docs = '';
-    if (startsWith('_', $module->name))
-        return $module;
+    if (startsWith('_', $module->name)) {
+            return $module;
+    }
     return apply(process_of([
         'generate_docs_header',
         'generate_docs_sommaire',
@@ -388,15 +399,17 @@ 
  * @return string
  */
 function generate_docs_contents_item($block) {
-    if ($block->type != 'function')
-        return '';
+    if ($block->type != 'function') {
+            return '';
+    }
     $params = join(', ', map(pipe(values(), join(' ')), get('params', $block)));
     $return = get('return', $block);
     $prototype = "```php\n{$block->name}({$params}) : {$return}\n```\n\n";
     $signature = '';
     $blockSignature = join("\n", $block->signatures);
-    if ($blockSignature)
-        $signature = "```\n{$blockSignature}\n```\n\n";
+    if ($blockSignature) {
+            $signature = "```\n{$blockSignature}\n```\n\n";
+    }
     return "# {$block->name}\n\n{$prototype}{$signature}{$block->description}\n\n";
 }
 
@@ -427,8 +440,9 @@ 
 function generate_tests_header($module) {
     $namespace = "Tarsana\UnitTests\Functional";
     $additionalNamespace = replace("/", "\\", remove(6, dirname($module->testsPath)));
-    if ($additionalNamespace)
-        $namespace .= "\\" . $additionalNamespace;
+    if ($additionalNamespace) {
+            $namespace .= "\\" . $additionalNamespace;
+    }
     $name = remove(-4, last(split("/", $module->testsPath)));
     $module->tests .= "<?php namespace {$namespace};\n\nuse Tarsana\Functional as F;\n\nclass {$name} extends \Tarsana\UnitTests\Functional\UnitTest {\n";
     return $module;
@@ -449,10 +463,11 @@ 
     $contents = join("\n", map(function($block) use($module) {
         return generate_tests_contents_item($block, $module);
     }, $blocks));
-    if (trim($contents) != '')
-        $module->tests .= $contents;
-    else
-        $module->tests = '';
+    if (trim($contents) != '') {
+            $module->tests .= $contents;
+    } else {
+            $module->tests = '';
+    }
     return $module;
 }
 
@@ -465,8 +480,9 @@ 
  * @return string
  */
 function generate_tests_contents_item($block, $module) {
-    if ($block->type != 'function')
-        return '';
+    if ($block->type != 'function') {
+            return '';
+    }
 
     $code = apply(pipe(
         _f('code_from_description'),
@@ -480,8 +496,9 @@ 
         join("\n")
     ), [$block]);
 
-    if ('' == trim($code))
-        return '';
+    if ('' == trim($code)) {
+            return '';
+    }
     return prepend("\tpublic function test_{$block->name}() {\n",
         append("\n\t}\n", $code)
     );
@@ -496,8 +513,9 @@ 
  */
 function code_from_description($block) {
     $description = get('description', $block);
-    if (!contains('```php', $description))
-        return '';
+    if (!contains('```php', $description)) {
+            return '';
+    }
     $code = remove(7 + indexOf('```php', $description), $description);
     return remove(-4, trim($code));
 }
@@ -513,8 +531,7 @@ 
     if (contains('; //=> ', $part)) {
         $pieces = split('; //=> ', $part);
         $part = "\$this->assertEquals({$pieces[1]}, {$pieces[0]});";
-    }
-    elseif (contains('; // throws ', $part)) {
+    } elseif (contains('; // throws ', $part)) {
         $pieces = split('; // throws ', $part);
         $variables = match('/ \$[0-9a-zA-Z_]+/', $pieces[0]);
         $use = '';
@@ -523,8 +540,7 @@ 
             $use = "use({$variables}) ";
         }
         return "\$this->assertErrorThrown(function() {$use}{\n\t$pieces[0]; \n},\n{$pieces[1]});";
-    }
-    elseif (startsWith('class ', $part) || startsWith('function ', $part)) {
+    } elseif (startsWith('class ', $part) || startsWith('function ', $part)) {
         $module->testsFooter .= $part . "\n\n";
         $part = '';
     }
@@ -539,8 +555,9 @@ 
  * @return object
  */
 function generate_tests_footer($module) {
-    if ($module->tests)
-        $module->tests .= "}\n\n{$module->testsFooter}";
+    if ($module->tests) {
+            $module->tests .= "}\n\n{$module->testsFooter}";
+    }
     return $module;
 }
 

src/list.php

Braces +45 added lines, -30 removed lines

@@ -146,8 +146,9 @@ 
 function head() {
     static $head = false;
     $head = $head ?: curry(function($list) {
-        if(is_string($list))
-            return substr($list, 0, 1);
+        if(is_string($list)) {
+                    return substr($list, 0, 1);
+        }
         return (count($list) > 0)
             ? $list[0]
             : null;
@@ -174,8 +175,9 @@ 
 function last () {
     static $last = false;
     $last = $last ?: curry(function($list) {
-        if(is_string($list))
-            return substr($list, -1);
+        if(is_string($list)) {
+                    return substr($list, -1);
+        }
         return (count($list) > 0)
             ? $list[count($list) - 1]
             : null;
@@ -203,10 +205,11 @@ 
 function init () {
     static $init = false;
     $init = $init ?: curry(function($list) {
-        if(is_string($list))
-            return (strlen($list) > 1)
+        if(is_string($list)) {
+                    return (strlen($list) > 1)
                 ? substr($list, 0, strlen($list) - 1)
                 : '';
+        }
         return (count($list) > 1)
             ? array_slice($list, 0, count($list) - 1)
             : [];
@@ -234,10 +237,11 @@ 
 function tail () {
     static $tail = false;
     $tail = $tail ?: curry(function($list) {
-        if(is_string($list))
-            return (strlen($list) > 1)
+        if(is_string($list)) {
+                    return (strlen($list) > 1)
                 ? substr($list, 1)
                 : '';
+        }
         return (count($list) > 1)
             ? array_slice($list, 1)
             : [];
@@ -359,8 +363,9 @@ 
     $concat = $concat ?: curry(function($list1, $list2) {
         $t1 = toString($list1);
         $t2 = toString($list2);
-        if (is_string($list1) && is_string($list2))
-            return $list1 . $list2;
+        if (is_string($list1) && is_string($list2)) {
+                    return $list1 . $list2;
+        }
         return array_merge($list1, $list2);
     });
     return _apply($concat, func_get_args());
@@ -443,10 +448,12 @@ 
     static $insertAll = false;
     $insertAll = $insertAll ?: curry(function($position, $items, $list) {
         $length = length($list);
-        if ($position < 0)
-            $position = $length + $position;
-        if ($position < 0)
-            $position = 0;
+        if ($position < 0) {
+                    $position = $length + $position;
+        }
+        if ($position < 0) {
+                    $position = 0;
+        }
         return ($position >= $length)
             ? concat($list, $items)
             : concatAll([take($position, $list), $items, remove($position, $list)]);
@@ -521,8 +528,9 @@ 
     static $take = false;
     $take = $take ?: curry(function($count, $list) {
         $length = length($list);
-        if ($count > $length || $count < -$length)
-            return [];
+        if ($count > $length || $count < -$length) {
+                    return [];
+        }
         if(is_string($list)) {
             return ($count >= 0)
                 ? substr($list, 0, $count)
@@ -662,8 +670,9 @@ 
     static $remove = false;
     $remove = $remove ?: curry(function($count, $list) {
         $length = length($list);
-        if ($count > $length || $count < -$length)
-            return [];
+        if ($count > $length || $count < -$length) {
+                    return [];
+        }
         return ($count > 0)
             ? take($count - $length, $list)
             : take($count + $length, $list);
@@ -812,10 +821,12 @@ 
 function slices() {
     static $slices = false;
     $slices = $slices ?: curry(function($size, $list) {
-        if(empty($list))
-            return is_string($list) ? '' : [];
-        if(length($list) <= $size)
-            return [$list];
+        if(empty($list)) {
+                    return is_string($list) ? '' : [];
+        }
+        if(length($list) <= $size) {
+                    return [$list];
+        }
         return prepend(take($size, $list), slices($size, remove($size, $list)));
     });
     return _apply($slices, func_get_args());
@@ -867,8 +878,9 @@ 
     static $findIndex = false;
     $findIndex = $findIndex ?: curry(function($predicate, $list) {
         foreach ($list as $key => $value) {
-            if ($predicate($value))
-                return $key;
+            if ($predicate($value)) {
+                            return $key;
+            }
         }
         return null;
     });
@@ -896,8 +908,9 @@ 
     static $findLastIndex = false;
     $findLastIndex = $findLastIndex ?: curry(function($predicate, $list) {
         foreach (reverse(toPairs($list)) as $pair) {
-            if($predicate($pair[1]))
-                return $pair[0];
+            if($predicate($pair[1])) {
+                            return $pair[0];
+            }
         }
         return null;
     });
@@ -1107,8 +1120,9 @@ 
     $groupBy = $groupBy ?: curry(function($fn, $list) {
         return reduce(function($result, $item) use($fn) {
             $index = $fn($item);
-            if (! isset($result[$index]))
-                $result[$index] = [];
+            if (! isset($result[$index])) {
+                            $result[$index] = [];
+            }
             $result[$index][] = $item;
             return $result;
         }, [], $list);
@@ -1137,8 +1151,9 @@ 
     $pairsFrom = $pairsFrom ?: curry(function($list1, $list2) {
         $length1 = length($list1);
         $length2 = length($list2);
-        if (0 == $length1 || 0 == $length2)
-            return [];
+        if (0 == $length1 || 0 == $length2) {
+                    return [];
+        }
         $list1 = values($list1);
         $list2 = values($list2);
         return map(function($index) use($list1, $list2) {

src/Internal/_functions.php

Braces +6 added lines, -4 removed lines

@@ -65,8 +65,9 @@ 
     $addArgument = function($currentBoundArgs, $arg) use($fnArgsCount) {
         $currentBoundArgsCount = count($currentBoundArgs);
         $placeholderPosition = 0;
-        while($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition]))
-            $placeholderPosition ++;
+        while($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition])) {
+                    $placeholderPosition ++;
+        }
         if ($currentBoundArgsCount < $fnArgsCount || $placeholderPosition == $currentBoundArgsCount) {
             $currentBoundArgs[] = $arg;
         } else { // There is a placeholder and number of bound args >= $fnArgsCount
@@ -92,8 +93,9 @@ 
         $boundArgs = _merge_args($argsCount, $boundArgs, func_get_args());
         $numberOfPlaceholders = count(array_filter($boundArgs, _f('_is_placeholder')));
         $numberOfGivenArgs = count($boundArgs) - $numberOfPlaceholders;
-        if ($numberOfGivenArgs >= $argsCount)
-            return call_user_func_array($fn, $boundArgs);
+        if ($numberOfGivenArgs >= $argsCount) {
+                    return call_user_func_array($fn, $boundArgs);
+        }
         return _curried_function($fn, $argsCount, $boundArgs);
     };
 }

src/Internal/_stream.php

Braces +12 added lines, -8 removed lines

@@ -131,8 +131,9 @@ 
  */
 function _stream_operation($name, $signature, $callable = null) {
     $callable = $callable ?: $name;
-    if (!is_callable($callable))
-        _stream_throw_error('unknown-callable', $callable);
+    if (!is_callable($callable)) {
+            _stream_throw_error('unknown-callable', $callable);
+    }
 
     return [
         'name' => $name,
@@ -163,8 +164,9 @@ 
     // Assuming $text  = 'Number|List -> Number -> String|Array -> Number'
     $parts = map(pipe(split('|'), map(pipe('trim', _stream_ensure_type($text)))), split('->', $text));
     // $parts = [['Number', 'List'], ['Number'], ['String', 'Array'], ['Number']]
-    if (length($parts) < 2)
-        _stream_throw_error('invalid-signature', $text);
+    if (length($parts) < 2) {
+            _stream_throw_error('invalid-signature', $text);
+    }
 
     return reduce(function($result, $part){
         return chain(function($item) use($result){
@@ -196,8 +198,9 @@ 
  */
 function _stream_ensure_type() {
     $ensureType = function($signature, $type) {
-        if (! contains($type, _stream_types()))
-            _stream_throw_error('invalid-signature', $signature);
+        if (! contains($type, _stream_types())) {
+                    _stream_throw_error('invalid-signature', $signature);
+        }
         return $type;
     };
     return apply(curry($ensureType), func_get_args());
@@ -553,8 +556,9 @@ 
  * @return array
  */
 function _stream_resolve($stream) {
-    if (get('resolved', $stream))
-        return $stream;
+    if (get('resolved', $stream)) {
+            return $stream;
+    }
     $transformations = get('transformations', $stream);
     $transformation = head($transformations);
     if (null === $transformation) {

src/Classes/Placeholder.php

Braces +3 added lines, -2 removed lines

@@ -11,8 +11,9 @@
     private function __construct(){}
     public static function get()
     {
-        if(static::$instance === null)
-            static::$instance = new Placeholder;
+        if(static::$instance === null) {
+                    static::$instance = new Placeholder;
+        }
         return static::$instance;
     }
     public function __toString(){

src/functions.php

Braces +9 added lines, -4 removed lines

@@ -94,8 +94,9 @@ 
  */
 function pipe() {
     $fns = func_get_args();
-    if(count($fns) < 1)
-        return identity();
+    if(count($fns) < 1) {
+            return identity();
+    }
     return curry(function () use ($fns) {
         $result = _apply(array_shift($fns), func_get_args());
         foreach ($fns as $fn) {
@@ -265,8 +266,12 @@ 
     static $comparator = false;
     $comparator = $comparator ?: curry(function($fn) {
         return function($a, $b) use($fn) {
-            if ($fn($a, $b)) return -1;
-            if ($fn($b, $a)) return 1;
+            if ($fn($a, $b)) {
+                return -1;
+            }
+            if ($fn($b, $a)) {
+                return 1;
+            }
             return 0;
         };
     });

src/common.php

Braces +12 added lines, -5 removed lines

@@ -33,9 +33,15 @@ 
 function type() {
     static $type = false;
     $type = $type ?: curry(function($data) {
-        if ($data instanceof Error) return 'Error';
-        if ($data instanceof Stream) return 'Stream';
-        if (is_callable($data)) return 'Function';
+        if ($data instanceof Error) {
+            return 'Error';
+        }
+        if ($data instanceof Stream) {
+            return 'Stream';
+        }
+        if (is_callable($data)) {
+            return 'Function';
+        }
         switch (gettype($data)) {
             case 'boolean':
                 return 'Boolean';
@@ -49,8 +55,9 @@ 
             case 'resource':
                 return 'Resource';
             case 'array':
-                if (allSatisfies('is_numeric', keys($data)))
-                    return 'List';
+                if (allSatisfies('is_numeric', keys($data))) {
+                                    return 'List';
+                }
                 return 'Array';
             case 'object':
                 return 'Object';