byrokrat / autogiro

spec/Exception/ParserExceptionSpec.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 
 use byrokrat\autogiro\Exception\ParserException;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class ParserExceptionSpec extends ObjectBehavior
 {

spec/ParserSpec.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use byrokrat\autogiro\Tree\Node;
 use byrokrat\autogiro\Exception\ParserException;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class ParserSpec extends ObjectBehavior
 {

spec/Processor/BankgiroProcessorSpec.php

Unused Use Statements -3 removed lines

@@ -5,9 +5,7 @@ 
 namespace spec\byrokrat\autogiro\Processor;
 
 use byrokrat\autogiro\Processor\BankgiroProcessor;
-use byrokrat\autogiro\Exception;
 use byrokrat\autogiro\Tree\OpeningNode;
-use byrokrat\autogiro\Tree\ClosingNode;
 use byrokrat\autogiro\Tree\RequestMandateCreationNode;
 use byrokrat\autogiro\Tree\RequestMandateAcceptanceNode;
 use byrokrat\autogiro\Tree\RequestMandateRejectionNode;
@@ -16,7 +14,6 @@ 
 use byrokrat\autogiro\Tree\MandateResponseNode;
 use byrokrat\autogiro\Tree\BankgiroNode;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class BankgiroProcessorSpec extends ObjectBehavior
 {

spec/Processor/FileProcessorSpec.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use byrokrat\autogiro\Tree\OpeningNode;
 use byrokrat\autogiro\Tree\BankgiroNode;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class FileProcessorSpec extends ObjectBehavior
 {

spec/Tree/MandateResponseNodeSpec.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use byrokrat\autogiro\Tree\IdNode;
 use byrokrat\autogiro\Tree\MessageNode;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class MandateResponseNodeSpec extends ObjectBehavior
 {

src/Grammar.php

Doc Comments +6 added lines

@@ -3022,6 +3022,9 @@ 
         return '"' . substr($this->string, $this->position) . '"';
     }
 
+    /**
+     * @param string $expecting
+     */
     protected function report($position, $expecting)
     {
         if ($this->cut) {
@@ -3044,6 +3047,9 @@ 
         return implode(', ', end($this->warnings));
     }
 
+    /**
+     * @param string $_string
+     */
     public function parse($_string)
     {
         $this->string = $_string;

Unused Use Statements -5 removed lines

@@ -11,14 +11,9 @@
 use byrokrat\autogiro\Tree\RequestMandateRejectionNode;
 use byrokrat\autogiro\Tree\RequestMandateUpdateNode;
 use byrokrat\autogiro\Tree\RequestMandateDeletionNode;
-use byrokrat\autogiro\Tree\RequestIncomingTransactionNode;
-use byrokrat\autogiro\Tree\RequestOutgoingTransactionNode;
-use byrokrat\autogiro\Tree\RequestTransactionDeletionNode;
-use byrokrat\autogiro\Tree\RequestTransactionUpdateNode;
 use byrokrat\autogiro\Tree\MandateResponseNode;
 use byrokrat\autogiro\Tree\PersonalIdNode;
 use byrokrat\autogiro\Tree\OrganizationIdNode;
-use byrokrat\autogiro\Tree\AmountNode;
 use byrokrat\autogiro\Tree\AccountNode;
 use byrokrat\autogiro\Tree\BankgiroNode;
 use byrokrat\autogiro\Tree\MessageNode;

Spacing +24 added lines, -24 removed lines

@@ -79,7 +79,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$layout) {
+            $this->value = call_user_func(function() use (&$layout) {
                 return $layout;
             });
         }
@@ -120,7 +120,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () {
+            $this->value = call_user_func(function() {
                 $this->currentLineNr = 0;
             });
         }
@@ -277,7 +277,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$date, &$layout, &$custNr, &$bg) {
+            $this->value = call_user_func(function() use (&$date, &$layout, &$custNr, &$bg) {
                 return new OpeningNode(rtrim($layout), $date, $custNr, $bg, $this->currentLineNr);
             });
         }
@@ -376,7 +376,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$layouts) {
+            $this->value = call_user_func(function() use (&$layouts) {
                 return new FileNode(...$layouts);
             });
         }
@@ -511,7 +511,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$date, &$custNr, &$bg) {
+            $this->value = call_user_func(function() use (&$date, &$custNr, &$bg) {
                 // TODO layout_request kan vara ''
                     // kanske lägga in ett test i visitor som kontrollerar att layoutId är ett känt värde... ??
                 return new OpeningNode('', $date, $custNr, $bg, $this->currentLineNr);
@@ -642,7 +642,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$opening, &$records) {
+            $this->value = call_user_func(function() use (&$opening, &$records) {
                 $opening->setAttribute('layout_name', Layouts::LAYOUT_MANDATE_REQUEST);
 
                 return new LayoutNode(
@@ -781,7 +781,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$bg, &$payerNr, &$account, &$id) {
+            $this->value = call_user_func(function() use (&$bg, &$payerNr, &$account, &$id) {
                 return new RequestMandateCreationNode($bg, $payerNr, $account, $id, $this->currentLineNr);
             });
         }
@@ -924,7 +924,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$bg, &$payerNr, &$reject) {
+            $this->value = call_user_func(function() use (&$bg, &$payerNr, &$reject) {
                 return $reject == 'AV'
                     ? new RequestMandateRejectionNode($bg, $payerNr, $this->currentLineNr)
                     : new RequestMandateAcceptanceNode($bg, $payerNr, $this->currentLineNr);
@@ -1021,7 +1021,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$oldBg, &$oldPayerNr, &$newBg, &$newPayerNr) {
+            $this->value = call_user_func(function() use (&$oldBg, &$oldPayerNr, &$newBg, &$newPayerNr) {
                 return new RequestMandateUpdateNode($oldBg, $oldPayerNr, $newBg, $newPayerNr, $this->currentLineNr);
             });
         }
@@ -1096,7 +1096,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$bg, &$payerNr) {
+            $this->value = call_user_func(function() use (&$bg, &$payerNr) {
                 return new RequestMandateDeletionNode($bg, $payerNr, $this->currentLineNr);
             });
         }
@@ -1193,7 +1193,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$opening, &$records) {
+            $this->value = call_user_func(function() use (&$opening, &$records) {
                 $opening->setAttribute('layout_name', Layouts::LAYOUT_PAYMENT_REQUEST);
 
                 return new LayoutNode(
@@ -1296,7 +1296,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$opening, &$records) {
+            $this->value = call_user_func(function() use (&$opening, &$records) {
                 $opening->setAttribute('layout_name', Layouts::LAYOUT_AMENDMENT_REQUEST);
 
                 return new LayoutNode(
@@ -1401,7 +1401,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$opening, &$mandates, &$closing) {
+            $this->value = call_user_func(function() use (&$opening, &$mandates, &$closing) {
                 return new FileNode(new LayoutNode($opening, $closing, ...$mandates));
             });
         }
@@ -1496,7 +1496,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$date, &$bg) {
+            $this->value = call_user_func(function() use (&$date, &$bg) {
                 return new OpeningNode('AG-MEDAVI', $date, '', $bg, $this->currentLineNr);
             });
         }
@@ -1705,7 +1705,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$bg, &$payerNr, &$accountNr, &$id, &$info, &$comment, &$date) {
+            $this->value = call_user_func(function() use (&$bg, &$payerNr, &$accountNr, &$id, &$info, &$comment, &$date) {
                 // TODO fundera om jag kan dela upp denna i flera R_ så att jag kan använda ACCOUNT ...
                 $accountNr = trim(ltrim($accountNr, '0'));
 
@@ -1824,7 +1824,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$date, &$nrOfPosts) {
+            $this->value = call_user_func(function() use (&$date, &$nrOfPosts) {
                 return new ClosingNode($date, intval($nrOfPosts), $this->currentLineNr);
             });
         }
@@ -1887,7 +1887,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$number) {
+            $this->value = call_user_func(function() use (&$number) {
                 return new AccountNode($this->currentLineNr, ltrim($number, '0'));
             });
         }
@@ -1924,7 +1924,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$number) {
+            $this->value = call_user_func(function() use (&$number) {
                 return new BankgiroNode($this->currentLineNr, ltrim($number, '0'));
             });
         }
@@ -2016,7 +2016,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$nr) {
+            $this->value = call_user_func(function() use (&$nr) {
                 return ltrim($nr, '0');
             });
         }
@@ -2085,7 +2085,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$date) {
+            $this->value = call_user_func(function() use (&$date) {
                 return new \DateTimeImmutable($date . '000000');
             });
         }
@@ -2160,10 +2160,10 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$century, &$number) {
+            $this->value = call_user_func(function() use (&$century, &$number) {
                 return in_array($century, ['00', '99'])
                     ? new OrganizationIdNode($this->currentLineNr, $number)
-                    : new PersonalIdNode($this->currentLineNr, $century.$number);
+                    : new PersonalIdNode($this->currentLineNr, $century . $number);
             });
         }
 
@@ -2225,7 +2225,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () use (&$nr) {
+            $this->value = call_user_func(function() use (&$nr) {
                 return new PayerNumberNode($this->currentLineNr, ltrim($nr, '0'));
             });
         }
@@ -2925,7 +2925,7 @@ 
         }
 
         if ($_success) {
-            $this->value = call_user_func(function () {
+            $this->value = call_user_func(function() {
                 $this->currentLineNr++;
             });
         }

spec/GrammarSpec.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     {
         return array_reduce(
             explode("\n", $content),
-            function ($carry, $line) {
+            function($carry, $line) {
                 if (empty(trim($line))) {
                     return $carry;
                 }

src/ParserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
      */
     public function createParser(int $flags = 0): Parser
     {
-        $flag = function (int $needle) use ($flags) {
+        $flag = function(int $needle) use ($flags) {
             return ($needle & $flags) == $needle;
         };
 

src/Tree/LayoutNode.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         $this->setChild('opening', $opening);
 
         foreach ($content as $key => $node) {
-            $this->setChild('content_'.++$key, $node);
+            $this->setChild('content_' . ++$key, $node);
         }
 
         $this->setChild('closing', $closing);