EC-CUBE / ec-cube

src/Eccube/Command/RouterCommand.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
             if (strtoupper($orderby) === "PATH") {
                 uasort(
                     $routes,
-                    function ($a, $b) {
+                    function($a, $b) {
                         return strcmp($a->getPath(), $b->getPath());
                     }
                 );

app/Plugin/Strategy/ServiceProvider/StrategyServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -12,14 +12,14 @@
     public function register(Container $app)
     {
         // サンプルの空Strategyをコンテナに登録
-        $app['eccube.calculate.strategy.empty'] = function () {
+        $app['eccube.calculate.strategy.empty'] = function() {
             return new EmptyStrategy();
         };
 
         // 空Strategyを追加.
         $app->extend(
             'eccube.calculate.strategies',
-            function (CalculateStrategyCollection $Collection, Container $app) {
+            function(CalculateStrategyCollection $Collection, Container $app) {
                 $Collection->add($app['eccube.calculate.strategy.empty']);
 
                 return $Collection;

app/Plugin/PurchaseProcessors/ServiceProvider/PurchaseProcessorsServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     {
         $app->extend(
             'eccube.purchase.flow.cart.item_processors',
-            function (ArrayCollection $processors, Container $app) {
+            function(ArrayCollection $processors, Container $app) {
                 $processors[] = new EmptyProcessor();
                 $processors[] = new ValidatableEmptyProcessor();
                 return $processors;

app/Plugin/FormExtension/ServiceProvider/FormExtensionServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     {
         $app->extend(
             'form.type.extensions',
-            function ($extensions) {
+            function($extensions) {
                 $extensions[] = new EntryTypeExtension();
 
                 return $extensions;

src/Eccube/Service/PurchaseFlow/PurchaseFlow.php

Spacing +6 added lines, -6 removed lines

@@ -104,7 +104,7 @@ 
      */
     protected function calculateTotal(ItemHolderInterface $itemHolder)
     {
-        $total = $itemHolder->getItems()->reduce(function ($sum, ItemInterface $item) {
+        $total = $itemHolder->getItems()->reduce(function($sum, ItemInterface $item) {
             $sum += $item->getPriceIncTax() * $item->getQuantity();
 
             return $sum;
@@ -121,7 +121,7 @@ 
     {
         $total = $itemHolder->getItems()
             ->getProductClasses()
-            ->reduce(function ($sum, ItemInterface $item) {
+            ->reduce(function($sum, ItemInterface $item) {
                 $sum += $item->getPriceIncTax() * $item->getQuantity();
 
                 return $sum;
@@ -140,7 +140,7 @@ 
     {
         $total = $itemHolder->getItems()
             ->getDeliveryFees()
-            ->reduce(function ($sum, ItemInterface $item) {
+            ->reduce(function($sum, ItemInterface $item) {
                 $sum += $item->getPriceIncTax() * $item->getQuantity();
 
                 return $sum;
@@ -155,7 +155,7 @@ 
     {
         $total = $itemHolder->getItems()
             ->getDiscounts()
-            ->reduce(function ($sum, ItemInterface $item) {
+            ->reduce(function($sum, ItemInterface $item) {
                 $sum += $item->getPriceIncTax() * $item->getQuantity();
 
                 return $sum;
@@ -171,7 +171,7 @@ 
     {
         $total = $itemHolder->getItems()
             ->getCharges()
-            ->reduce(function ($sum, ItemInterface $item) {
+            ->reduce(function($sum, ItemInterface $item) {
                 $sum += $item->getPriceIncTax() * $item->getQuantity();
 
                 return $sum;
@@ -185,7 +185,7 @@ 
     protected function calculateTax(ItemHolderInterface $itemHolder)
     {
         $total = $itemHolder->getItems()
-            ->reduce(function ($sum, ItemInterface $item) {
+            ->reduce(function($sum, ItemInterface $item) {
                 $sum += ($item->getPriceIncTax() - $item->getPrice()) * $item->getQuantity();
 
                 return $sum;

src/Eccube/Service/PurchaseFlow/PurchaseFlowResult.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
      */
     public function getErrors()
     {
-        return array_filter($this->processResults, function (ProcessResult $processResult) {
+        return array_filter($this->processResults, function(ProcessResult $processResult) {
             return $processResult->isError();
         });
     }
@@ -63,7 +63,7 @@ 
      */
     public function getWarning()
     {
-        return array_filter($this->processResults, function (ProcessResult $processResult) {
+        return array_filter($this->processResults, function(ProcessResult $processResult) {
             return $processResult->isWarning();
         });
     }

src/Eccube/Controller/AbstractController.php

Doc Comments +6 added lines

@@ -59,11 +59,17 @@
      */
     protected $formFactory;
 
+    /**
+     * @param \Eccube\Application $app
+     */
     protected function getSecurity($app)
     {
         return $this->tokenStorage;
     }
 
+    /**
+     * @param \Eccube\Application $app
+     */
     protected function isTokenValid($app)
     {
         $csrf = $this->csrfTokenManager;

src/Eccube/Controller/Admin/Content/FileController.php

Doc Comments +11 added lines

@@ -270,6 +270,10 @@ 
         return $str;
     }
 
+    /**
+     * @param string $topDir
+     * @param Request $request
+     */
     private function getTree($topDir, $request)
     {
         $finder = Finder::create()->in($topDir)
@@ -307,6 +311,10 @@ 
         return $tree;
     }
 
+    /**
+     * @param Application $app
+     * @param string $nowDir
+     */
     private function getFileList($app, $nowDir)
     {
         $topDir = $this->appConfig['user_data_realdir'];
@@ -366,6 +374,9 @@ 
         return (strpos($targetDir, $topDir) === 0);
     }
 
+    /**
+     * @return string
+     */
     private function convertStrFromServer($target)
     {
         if ($this->encode == self::SJIS) {

Spacing +5 added lines, -5 removed lines

@@ -64,7 +64,7 @@ 
     private $error = null;
     private $encode = '';
 
-    public function __construct(){
+    public function __construct() {
         $this->encode = self::UTF;
         if ('\\' === DIRECTORY_SEPARATOR) {
             $this->encode = self::SJIS;
@@ -113,7 +113,7 @@ 
         $arrFileList = $this->getFileList($app, $nowDir);
 
         $javascript = $this->getJsArrayList($tree);
-        $onload = "eccube.fileManager.viewFileTree('tree', arrTree, '" . $nowDir . "', 'tree_select_file', 'tree_status', 'move');";
+        $onload = "eccube.fileManager.viewFileTree('tree', arrTree, '".$nowDir."', 'tree_select_file', 'tree_status', 'move');";
 
         return [
             'form' => $form->createView(),
@@ -174,7 +174,7 @@ 
                 $nowDir = $this->checkDir($request->get('now_dir'), $topDir)
                     ? $this->normalizePath($request->get('now_dir'))
                     : $topDir;
-                $fs->mkdir($nowDir . '/' . $filename);
+                $fs->mkdir($nowDir.'/'.$filename);
             }
         }
     }
@@ -259,7 +259,7 @@ 
     {
         $str = "arrTree = new Array();\n";
         foreach ($tree as $key => $val) {
-            $str .= 'arrTree[' . $key . "] = new Array(" . $key . ", '" . $val['type'] . "', '" . $val['path'] . "', " . $val['rank'] . ',';
+            $str .= 'arrTree['.$key."] = new Array(".$key.", '".$val['type']."', '".$val['path']."', ".$val['rank'].',';
             if ($val['open']) {
                 $str .= "true);\n";
             } else {
@@ -310,7 +310,7 @@ 
     private function getFileList($app, $nowDir)
     {
         $topDir = $this->appConfig['user_data_realdir'];
-        $filter = function (\SplFileInfo $file) use ($topDir) {
+        $filter = function(\SplFileInfo $file) use ($topDir) {
             $acceptPath = realpath($topDir);
             $targetPath = $file->getRealPath();
             return (strpos($targetPath, $acceptPath) === 0);

src/Eccube/Controller/NonMemberShoppingController.php

Doc Comments +1 added lines, -1 removed lines

@@ -474,7 +474,7 @@
      *
      * @param Application $app
      * @param array $data リクエストパラメータ
-     * @return array
+     * @return \Symfony\Component\Validator\ConstraintViolationListInterface[]
      */
     protected function customerValidation(Application $app, array $data)
     {