webstream-framework / WebStream

WebStream/Annotation/ExceptionHandler.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace WebStream\Annotation;
 
-use WebStream\Core\CoreInterface;
 use WebStream\Annotation\Base\Annotation;
 use WebStream\Annotation\Base\IAnnotatable;
 use WebStream\Annotation\Base\IRead;

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     /**
      * {@inheritdoc}
      */
-    public function onMethodInject(IAnnotatable &$instance, Container $container, \ReflectionMethod $method)
+    public function onMethodInject(IAnnotatable&$instance, Container $container, \ReflectionMethod $method)
     {
         $this->injectedLog($this);
 

WebStream/Annotation/Query.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace WebStream\Annotation;
 
-use WebStream\Core\CoreInterface;
 use WebStream\Annotation\Base\Annotation;
 use WebStream\Annotation\Base\IAnnotatable;
 use WebStream\Annotation\Base\IRead;

Spacing +4 added lines, -4 removed lines

@@ -52,11 +52,11 @@ 
     /**
      * {@inheritdoc}
      */
-    public function onMethodInject(IAnnotatable &$instance, Container $container, \ReflectionMethod $method)
+    public function onMethodInject(IAnnotatable&$instance, Container $container, \ReflectionMethod $method)
     {
         $this->injectedLog($this);
 
-        $key = $method->class . "#" . $method->name;
+        $key = $method->class."#".$method->name;
         if ($this->injectedContainer->{$key} === null) {
             $this->injectedContainer->{$key} = new AnnotationListContainer();
         }
@@ -65,13 +65,13 @@ 
             $files = [$files];
         }
 
-        $this->injectedContainer->{$key}->pushAsLazy(function () use ($files) {
+        $this->injectedContainer->{$key}->pushAsLazy(function() use ($files) {
             $xmlObjectList = [];
             foreach ($files as $file) {
                 if (file_exists($file)) {
                     $xmlObject = simplexml_load_file($file);
                     if ($xmlObject === false) {
-                        throw new DatabaseException("Failded to parse query file: " . $file);
+                        throw new DatabaseException("Failded to parse query file: ".$file);
                     }
                     $xmlObjectList[] = $xmlObject;
                 }

WebStream/Annotation/Reader/AnnotationReader.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     /**
      * constructor
      * @param IAnnotatable アノテーション使用可能インスタンス
-     * @param Container 依存コンテナ
+     * @param Container Container
      */
     public function __construct(IAnnotatable &$instance, Container $container)
     {

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
      * @param IAnnotatable アノテーション使用可能インスタンス
      * @param Container 依存コンテナ
      */
-    public function __construct(IAnnotatable &$instance, Container $container)
+    public function __construct(IAnnotatable&$instance, Container $container)
     {
         $this->instance = $instance;
         $this->container = $container;
@@ -252,7 +252,7 @@ 
                         $annotation->onPropertyInject($this->instance, $this->container, $property);
                     } catch (\Exception $e) {
                         if ($this->exception === null) {
-                            $this->exception = function () use ($e) {
+                            $this->exception = function() use ($e) {
                                 throw $e;
                             };
                         }

WebStream/Annotation/Template.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace WebStream\Annotation;
 
-use WebStream\Core\CoreInterface;
 use WebStream\Annotation\Base\Annotation;
 use WebStream\Annotation\Base\IAnnotatable;
 use WebStream\Annotation\Base\IMethod;

Spacing +9 added lines, -9 removed lines

@@ -53,7 +53,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function onMethodInject(IAnnotatable &$instance, Container $container, \ReflectionMethod $method)
+    public function onMethodInject(IAnnotatable&$instance, Container $container, \ReflectionMethod $method)
     {
         $this->injectedLog($this);
 
@@ -64,17 +64,17 @@ 
 
         if (PHP_OS === "WIN32" || PHP_OS === "WINNT") {
             if (preg_match("/^.*[. ]|.*[\p{Cntrl}\/:*?\"<>|].*|(?i:CON|PRN|AUX|CLOCK\$|NUL|COM[1-9]|LPT[1-9])(?:[.].+)?$/", $filename)) {
-                throw new AnnotationException("Invalid string contains in @Template('" . safetyOut($filename) . "')");
+                throw new AnnotationException("Invalid string contains in @Template('".safetyOut($filename)."')");
             }
         } else {
             if (preg_match("/:|\.\.\/|\.\.\\\\/", $filename)) {
-                throw new AnnotationException("Invalid string contains in @Template('" . safetyOut($filename) . "')");
+                throw new AnnotationException("Invalid string contains in @Template('".safetyOut($filename)."')");
             }
         }
 
         if ($filename === null) {
-            $errorMsg = "Invalid argument of @Template('" . safetyOut($filename) . "'). ";
-            $errorMsg.= "There is no specification of the base template.";
+            $errorMsg = "Invalid argument of @Template('".safetyOut($filename)."'). ";
+            $errorMsg .= "There is no specification of the base template.";
             throw new AnnotationException($errorMsg);
         }
 
@@ -83,8 +83,8 @@ 
         if ($engine === "twig") {
             if (!is_bool($debug)) {
                 if ($debug !== null) {
-                    $errorMsg = "Invalid argument of @Template('" . safetyOut($filename) . "'). ";
-                    $errorMsg.= "'debug' attribute bool only be specified.";
+                    $errorMsg = "Invalid argument of @Template('".safetyOut($filename)."'). ";
+                    $errorMsg .= "'debug' attribute bool only be specified.";
                     throw new AnnotationException($errorMsg);
                 }
                 $debug = false;
@@ -115,7 +115,7 @@ 
 
                 $cacheTime = intval($cacheTime);
                 if ($cacheTime <= 0) {
-                    $errorMsg = "Expire value is out of integer range: @Template(cacheTime=" . strval($cacheTime) . ")";
+                    $errorMsg = "Expire value is out of integer range: @Template(cacheTime=".strval($cacheTime).")";
                     throw new AnnotationException($errorMsg);
                 } elseif ($cacheTime >= PHP_INT_MAX) {
                     $this->logger->warn("Expire value converted the maximum of PHP Integer.");
@@ -126,7 +126,7 @@ 
 
             $this->injectedContainer->engine = new Basic($container);
         } else {
-            $errorMsg = "Invalid 'engine' attribute of @Template('" . safetyOut($filename) . "'.";
+            $errorMsg = "Invalid 'engine' attribute of @Template('".safetyOut($filename)."'.";
             throw new AnnotationException($errorMsg);
         }
     }

WebStream/Annotation/Validate.php

Unused Use Statements -2 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 namespace WebStream\Annotation;
 
-use WebStream\Core\CoreInterface;
 use WebStream\Annotation\Base\Annotation;
 use WebStream\Annotation\Base\IAnnotatable;
 use WebStream\Annotation\Base\IMethod;
@@ -10,7 +9,6 @@ 
 use WebStream\Module\Utility\CommonUtils;
 use WebStream\Module\Utility\ApplicationUtils;
 use WebStream\Module\ClassLoader;
-use WebStream\DI\ServiceLocator;
 use WebStream\Exception\Extend\ValidateException;
 use WebStream\Exception\Extend\AnnotationException;
 

Spacing +12 added lines, -12 removed lines

@@ -44,7 +44,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function onMethodInject(IAnnotatable &$instance, Container $container, \ReflectionMethod $method)
+    public function onMethodInject(IAnnotatable&$instance, Container $container, \ReflectionMethod $method)
     {
         $this->injectedLog($this);
 
@@ -55,7 +55,7 @@ 
         $this->injectedContainer->isValid = false;
 
         if ($method !== null && !$this->inArray($method, ["get", "post", "put", "delete"])) {
-            $errorMsg = "Invalid method attribute is specified: " . safetyOut($method);
+            $errorMsg = "Invalid method attribute is specified: ".safetyOut($method);
             throw new AnnotationException($errorMsg);
         }
 
@@ -68,35 +68,35 @@ 
                         ->inject('applicationInfo', $container->applicationInfo);
 
             // デフォルトバリデーションルールのパス
-            $filepath = $container->applicationInfo->validateRuleDir . $className . ".php";
+            $filepath = $container->applicationInfo->validateRuleDir.$className.".php";
             if (!$classLoader->import($filepath)) {
                 $loadList = $classLoader->load($className);
                 // バリデーションルールのクラス名が複数指定されている場合は適用判断不可能なのでエラー
                 if (count($loadList) >= 2) {
-                    $errorMsg = "Class load failed because the same class name has been identified: " . $className . "";
+                    $errorMsg = "Class load failed because the same class name has been identified: ".$className."";
                     throw new ValidateException($errorMsg);
                 }
 
                 if (count($loadList) === 0) {
-                    $errorMsg = "Invalid Validate class filepath: " . $filepath . "";
+                    $errorMsg = "Invalid Validate class filepath: ".$filepath."";
                     throw new ValidateException($errorMsg);
                 }
 
                 $namespace = $this->getNamespace($loadList[0]);
-                $classpath = $namespace . "\\" . $className;
+                $classpath = $namespace."\\".$className;
             }
 
             $root = $container->applicationInfo->applicationRoot;
-            $classpath = $classpath ?: $this->getNamespace($root . "/" . $filepath) . "\\" . $className;
+            $classpath = $classpath ?: $this->getNamespace($root."/".$filepath)."\\".$className;
 
             if (!class_exists($classpath)) {
-                $errorMsg = "Invalid Validate class's classpath: " . $classpath;
+                $errorMsg = "Invalid Validate class's classpath: ".$classpath;
                 throw new AnnotationException($errorMsg);
             }
 
             $validateInstance = new $classpath();
             if ($validateInstance instanceof WebStream\Validate\IValidate) {
-                $errorMsg = get_class($validateInstance) . " must be IValidate instance.";
+                $errorMsg = get_class($validateInstance)." must be IValidate instance.";
                 throw new AnnotationException($errorMsg);
             }
 
@@ -118,7 +118,7 @@ 
                     $params = $container->request->delete;
                 }
             } else {
-                $errorMsg = "Unsupported method is specified: " . safetyOut($method);
+                $errorMsg = "Unsupported method is specified: ".safetyOut($method);
                 throw new AnnotationException($errorMsg);
             }
 
@@ -127,11 +127,11 @@ 
             $value = is_array($params) && array_key_exists($key, $params) ? $params[$key] : null;
 
             if (!$validateInstance->isValid($value, $rule)) {
-                $errorMsg = "Validation rule error. Rule is '$rule', value is " . (safetyOut($value) ?: "null");
+                $errorMsg = "Validation rule error. Rule is '$rule', value is ".(safetyOut($value) ?: "null");
                 throw new ValidateException($errorMsg);
             }
         } else {
-            $errorMsg = "Invalid validation rule definition: " . $rule;
+            $errorMsg = "Invalid validation rule definition: ".$rule;
             throw new ValidateException($errorMsg);
         }
     }

WebStream/Core/Application.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
     /**
      * アプリケーション共通で使用するクラスを初期化する
-     * @param Container DIコンテナ
+     * @param Container Container
      */
     public function __construct(Container $container)
     {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use WebStream\Exception\ApplicationException;
 use WebStream\Exception\SystemException;
 use WebStream\Exception\DelegateException;
-use WebStream\DI\ServiceLocator;
 
 /**
  * Applicationクラス

WebStream/Core/CoreModel.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use WebStream\Annotation\Filter;
 use WebStream\Annotation\Base\IAnnotatable;
 use WebStream\Database\DatabaseManager;
-use WebStream\Database\Result;
 use WebStream\Exception\Extend\DatabaseException;
 use WebStream\Exception\Extend\MethodNotFoundException;
 

Spacing +7 added lines, -7 removed lines

@@ -164,7 +164,7 @@ 
                         $result = $this->manager->query($sql)->{$method}();
                     }
                 } else {
-                    throw new DatabaseException("Invalid SQL or bind parameters: " . $sql .", " . strval($bind));
+                    throw new DatabaseException("Invalid SQL or bind parameters: ".$sql.", ".strval($bind));
                 }
             } else {
                 $bind = null;
@@ -185,7 +185,7 @@ 
                     }
                 }
 
-                $queryKey = $this->classpath . "#" . $modelMethod;
+                $queryKey = $this->classpath."#".$modelMethod;
                 $queryId = $method;
 
                 $refClass = new \ReflectionClass($this);
@@ -224,7 +224,7 @@ 
 
                 if ($entityClassPath !== null) {
                     if (!class_exists($entityClassPath)) {
-                        throw new DatabaseException("Entity classpath is not found: " . $entityClassPath);
+                        throw new DatabaseException("Entity classpath is not found: ".$entityClassPath);
                     }
 
                     switch ($method) {
@@ -236,9 +236,9 @@ 
                                     $result = $this->manager->query($sql)->select()->toEntity($entityClassPath);
                                 }
                             } else {
-                                $errorMessage = "Invalid SQL or bind parameters: " . $sql;
+                                $errorMessage = "Invalid SQL or bind parameters: ".$sql;
                                 if (is_array($bind)) {
-                                    $errorMessage .= ", " . strval($bind);
+                                    $errorMessage .= ", ".strval($bind);
                                 }
 
                                 throw new DatabaseException($errorMessage);
@@ -262,9 +262,9 @@ 
                             $result = $this->manager->query($sql)->{$method}();
                         }
                     } else {
-                        $errorMessage = "Invalid SQL or bind parameters: " . $sql;
+                        $errorMessage = "Invalid SQL or bind parameters: ".$sql;
                         if (is_array($bind)) {
-                            $errorMessage .= ", " . strval($bind);
+                            $errorMessage .= ", ".strval($bind);
                         }
 
                         throw new DatabaseException($errorMessage);

WebStream/Database/ConnectionManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 
     /**
      * constructor
-     * @param Container 依存コンテナ
+     * @param Container Container
      */
     public function __construct(Container $container)
     {
@@ -53,7 +53,7 @@ 
 
     /**
      * 初期処理
-     * @param Container 依存コンテナ
+     * @param Container Container
      */
     private function initialize(Container $container)
     {

Spacing +2 added lines, -2 removed lines

@@ -77,14 +77,14 @@
             $dsnHash = "";
             $databaseConfigContainer = new Container(false);
             foreach ($config as $key => $value) {
-                $dsnHash .= $key . $value;
+                $dsnHash .= $key.$value;
                 $databaseConfigContainer->set($key, $value);
             }
             $dsnHash = md5($dsnHash);
 
             $this->classpathMap[$container->filepath] = $dsnHash;
 
-            $this->connectionContainer->{$dsnHash} = function () use ($driverClassPath, $databaseConfigContainer, $logger) {
+            $this->connectionContainer->{$dsnHash} = function() use ($driverClassPath, $databaseConfigContainer, $logger) {
                 $driver = new $driverClassPath($databaseConfigContainer);
                 $driver->inject('logger', $logger);
 

WebStream/Database/DatabaseManager.php

Doc Comments +2 added lines, -1 removed lines

@@ -37,7 +37,7 @@ 
 
     /**
      * constructor
-     * @param Container 依存コンテナ
+     * @param Container Container
      */
     public function __construct(Container $container)
     {
@@ -183,6 +183,7 @@ 
      * クエリを設定する
      * @param string SQL
      * @param array<string> パラメータ
+     * @param string $sql
      */
     public function query($sql, array $bind = [])
     {