webstream-framework / WebStream

WebStream/Http/Request.php

Doc Comments +1 added lines

@@ -108,6 +108,7 @@
     /**
      * SERVERパラメータ取得
      * @param string パラメータキー
+     * @param string $key
      */
     private function server($key)
     {

WebStream/Http/Response.php

Doc Comments +9 added lines

@@ -67,6 +67,7 @@ 
     /**
      * Cache-Controlを設定
      * @param String Cache-Control
+     * @param string $cacheControl
      */
     public function setCacheControl($cacheControl)
     {
@@ -76,6 +77,7 @@ 
     /**
      * Pragmaを設定
      * @param String Pragma
+     * @param string $pragma
      */
     public function setPragma($pragma)
     {
@@ -86,6 +88,7 @@ 
      * MimeTypeを設定
      * ファイルタイプにより指定
      * @param String ファイルタイプ
+     * @param string $fileType
      */
     public function setType($fileType)
     {
@@ -167,6 +170,7 @@ 
     /**
      * Content-Lengthを設定
      * @param Integer Content-Length
+     * @param integer $contentLength
      */
     public function setContentLength($contentLength)
     {
@@ -187,6 +191,7 @@ 
     /**
      * Content-Transfer-Encodingを設定
      * @param String エンコーディング方法
+     * @param string $contentTransferEncoding
      */
     public function setContentTransferEncoding($contentTransferEncoding)
     {
@@ -196,6 +201,7 @@ 
     /**
      * Expiresを設定
      * @param Integer 有効期限
+     * @param integer $expires
      */
     public function setExpires($expires)
     {
@@ -205,6 +211,7 @@ 
     /**
      * レスポンスボディを設定
      * @param String レスポンスボディ
+     * @param string $body
      */
     public function setBody($body)
     {
@@ -504,6 +511,7 @@ 
     /**
      * 指定したステータスコードのページに遷移
      * @param Integer ステータスコード
+     * @param integer $statusCode
      */
     public function move($statusCode)
     {
@@ -545,6 +553,7 @@ 
     /**
      * HTMLテンプレート
      * @param String 表示内容
+     * @param string $content
      * @return String HTML
      */
     private function bodyTemplate($content)

Indentation +1 added lines, -1 removed lines

@@ -236,7 +236,7 @@
     {
         // StatusCode
         $headerMessage = 'HTTP/' . self::HTTP_VERSION . ' ' .
-                         $this->statusCode . ' ' . $this->status[$this->statusCode];
+                            $this->statusCode . ' ' . $this->status[$this->statusCode];
         header($headerMessage);
 
         // Redirect

Spacing +18 added lines, -18 removed lines

@@ -155,11 +155,11 @@ 
     public function setStatusCode($statusCode)
     {
         if (!is_string($statusCode) && !is_int($statusCode)) {
-            throw new ConnectionException("Invalid status code format: " . strval($statusCode));
+            throw new ConnectionException("Invalid status code format: ".strval($statusCode));
         }
 
         if (!array_key_exists($statusCode, $this->status)) {
-            throw new ConnectionException("Unknown status code: " . $statusCode);
+            throw new ConnectionException("Unknown status code: ".$statusCode);
         }
         $this->statusCode = $statusCode;
     }
@@ -180,7 +180,7 @@ 
     public function setContentDisposition($filename)
     {
         if (file_exists($filename)) {
-            $this->contentDisposition = 'attachement; filename="'. basename($filename) . '"';
+            $this->contentDisposition = 'attachement; filename="'.basename($filename).'"';
         }
     }
 
@@ -235,43 +235,43 @@ 
     public function header()
     {
         // StatusCode
-        $headerMessage = 'HTTP/' . self::HTTP_VERSION . ' ' .
-                         $this->statusCode . ' ' . $this->status[$this->statusCode];
+        $headerMessage = 'HTTP/'.self::HTTP_VERSION.' '.
+                         $this->statusCode.' '.$this->status[$this->statusCode];
         header($headerMessage);
 
         // Redirect
         if (intval($this->statusCode) === 301) {
-            header('Location: ' . $this->location);
+            header('Location: '.$this->location);
         }
 
         // Content-Type
-        header('Content-Type: ' . $this->mimeType . '; charset=' . $this->charset);
+        header('Content-Type: '.$this->mimeType.'; charset='.$this->charset);
 
         // Content-Length
         if ($this->contentLength === null) {
             $this->contentLength = $this->bytelen($this->body);
         }
-        header('Content-Length: ' . $this->contentLength);
+        header('Content-Length: '.$this->contentLength);
 
         // Content-Disposition
         if ($this->contentDisposition !== null) {
-            header('Content-Disposition: ' . $this->contentDisposition);
+            header('Content-Disposition: '.$this->contentDisposition);
         }
 
         // Content-Transfer-Encoding
         if ($this->contentTransferEncoding !== null) {
-            header('Content-Transfer-Encoding: ' . $this->contentTransferEncoding);
+            header('Content-Transfer-Encoding: '.$this->contentTransferEncoding);
         }
 
         // Cache-Control
-        header('Cache-Control: ' . $this->cacheControl);
+        header('Cache-Control: '.$this->cacheControl);
 
         // Pragma
-        header('Pragma: ' . $this->pragma);
+        header('Pragma: '.$this->pragma);
 
         // Expires
         if ($this->expires !== null) {
-            header('Expires: ' . $this->expires);
+            header('Expires: '.$this->expires);
         }
 
         // X-Content-Type-Options
@@ -279,20 +279,20 @@ 
 
         // Access-Control-Allow-Origin
         if (!empty($this->accessControlAllowOrigin)) {
-            header('Access-Control-Allow-Origin: ' . implode(',', $this->accessControlAllowOrigin));
+            header('Access-Control-Allow-Origin: '.implode(',', $this->accessControlAllowOrigin));
         }
 
         // X-Frame-Options
         if ($this->xframeOptions !== null) {
-            header('X-Frame-Options: ' . $this->xframeOptions);
+            header('X-Frame-Options: '.$this->xframeOptions);
         }
 
         // X-XSS-Protection
         if ($this->xxssProtection !== null) {
-            header('X-XSS-Protection: ' . $this->xxssProtection);
+            header('X-XSS-Protection: '.$this->xxssProtection);
         }
 
-        $this->logger->info("HTTP access occured: status code " . $this->statusCode);
+        $this->logger->info("HTTP access occured: status code ".$this->statusCode);
     }
 
     /**
@@ -510,7 +510,7 @@ 
         ob_clean(); // これまでの出力バッファをクリア
         $statusCode = array_key_exists($statusCode, $this->status) ? $statusCode : 500;
         $this->setStatusCode($statusCode);
-        $bodyMessage = $statusCode . ' ' . $this->status[$statusCode];
+        $bodyMessage = $statusCode.' '.$this->status[$statusCode];
         $this->setBody($this->bodyTemplate($bodyMessage));
         $this->send();
         exit;

WebStream/Http/Session.php

Doc Comments +4 added lines

@@ -49,6 +49,10 @@
      * @param string Cookieを有効にするドメイン
      * @param boolean Secure属性を有効にする
      * @param boolean HttpOnly属性を有効にする
+     * @param string $path
+     * @param string $domain
+     * @param boolean $secure
+     * @param boolean $httpOnly
      */
     private function initialize($expire, $path, $domain, $secure, $httpOnly)
     {

WebStream/Log/Logger.php

Doc Comments +6 added lines, -2 removed lines

@@ -59,6 +59,7 @@ 
     /**
      * コンストラクタ
      * @param string ログファイルパス
+     * @param string $configPath
      */
     private function __construct($configPath)
     {
@@ -75,7 +76,7 @@ 
 
     /**
      * インスタンスを返却する
-     * @return WebStream\Module\Logger ロガーインスタンス
+     * @return Logger ロガーインスタンス
      */
     public static function getInstance()
     {
@@ -292,6 +293,7 @@ 
      * @param string ログレベル文字列
      * @param string 書きだす文字列
      * @param string スタックトレース文字列
+     * @param string $msg
      */
     public function write($level, $msg, $context = null)
     {
@@ -381,6 +383,8 @@ 
      * ローテートを実行する
      * @param integer 作成日時のUnixTime
      * @param integer 現在日時のUnixTime
+     * @param integer $from
+     * @param integer $to
      */
     private function runRotate($from, $to)
     {
@@ -450,7 +454,7 @@ 
 
     /**
      * ログローテートサイズを返却する
-     * @return string ログ出力パス
+     * @return integer ログ出力パス
      */
     public function getLogRotateSize()
     {

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace WebStream\Log;
 
 use WebStream\Module\Utility\FileUtils;
-use WebStream\Module\Container;
 use WebStream\DI\ServiceLocator;
 use WebStream\Exception\Extend\LoggerException;
 

Spacing +10 added lines, -10 removed lines

@@ -139,14 +139,14 @@ 
 
         // 設定ファイルが存在するかどうか
         if ($log === null) {
-            throw new LoggerException("Log config file does not exist: " . $configPath);
+            throw new LoggerException("Log config file does not exist: ".$configPath);
         }
 
         // ログレベル取得
         $logLevel = $this->toLogLevelValue($log["level"]);
         // 妥当なログレベルかどうか
         if ($logLevel === 0) {
-            throw new LoggerException("Invalid log level: " . $log["level"]);
+            throw new LoggerException("Invalid log level: ".$log["level"]);
         }
         $this->logLevel = $logLevel;
 
@@ -155,16 +155,16 @@ 
         // 絶対パスでのチェック
         if (!realpath(dirname($path))) {
             $container = ServiceLocator::getInstance()->getContainer();
-            $path = $container->applicationInfo->applicationRoot . "/" . $log["path"];
+            $path = $container->applicationInfo->applicationRoot."/".$log["path"];
             // プロジェクトルートからの相対パスでのチェック
             if (!file_exists(dirname($path))) {
-                throw new LoggerException("Log directory does not exist: " . dirname($path));
+                throw new LoggerException("Log directory does not exist: ".dirname($path));
             }
         }
         $this->logPath = $path;
 
         // ステータスファイルパスを設定
-        $this->statusPath = preg_replace_callback('/(.*)\..+/', function ($matches) {
+        $this->statusPath = preg_replace_callback('/(.*)\..+/', function($matches) {
             return "$matches[1].status";
         }, $this->logPath);
 
@@ -173,7 +173,7 @@ 
             $rotateCycle = $this->cycle2value($log["rotate_cycle"]);
             // 妥当なローテートサイクルか
             if ($rotateCycle === 0) {
-                throw new LoggerException("Invalid log rotate cycle: " . $log["rotate_cycle"]);
+                throw new LoggerException("Invalid log rotate cycle: ".$log["rotate_cycle"]);
             }
             $this->rotateCycle = $rotateCycle;
         }
@@ -183,7 +183,7 @@ 
             $rotateSize = intval($log["rotate_size"]);
             // ローテートサイズが不正の場合(正の整数以外の値が設定された場合)
             if ($rotateSize <= 0) {
-                throw new LoggerException("Invalid log rotate size: " . $log["rotate_size"]);
+                throw new LoggerException("Invalid log rotate size: ".$log["rotate_size"]);
             }
             $this->rotateSize = $rotateSize;
         }
@@ -262,7 +262,7 @@ 
         date_default_timezone_set('Asia/Tokyo');
         $msec = sprintf("%2d", floatval(microtime()) * 100);
 
-        return strftime("%Y-%m-%d %H:%M:%S") . "," . $msec;
+        return strftime("%Y-%m-%d %H:%M:%S").",".$msec;
     }
 
     /**
@@ -280,7 +280,7 @@ 
                 continue;
             }
             $msg .= "\n";
-            $msg .= "\t#" . trim($stacktraceLine);
+            $msg .= "\t#".trim($stacktraceLine);
         }
 
         return $msg;
@@ -341,7 +341,7 @@ 
         $content = fread($handle, $size);
         fclose($handle);
         if (!preg_match('/^\d{10}$/', $content)) {
-            throw new LoggerException("Invalid log state file contents: " . $content);
+            throw new LoggerException("Invalid log state file contents: ".$content);
         }
 
         return intval($content);

WebStream/Log/LoggerFormatter.php

Doc Comments +4 added lines, -1 removed lines

@@ -30,6 +30,7 @@ 
     /**
      * コンストラクタ
      * @param string 設定ファイルパス
+     * @param string $configPath
      */
     public function __construct($configPath)
     {
@@ -41,7 +42,7 @@ 
      * フォーマット済みメッセージを返却する
      * @param  string メッセージ
      * @param  string ログレベル
-     * @return フォーマット済みメッセージ
+     * @return string
      */
     public function getFormattedMessage($message, $logLevel)
     {
@@ -97,6 +98,7 @@ 
      * アプリケーション名項目を埋め込む
      * @param  string メッセージ
      * @param  string アプリケーション名
+     * @param string $message
      * @return 埋め込み済みメッセージ
      */
     private function compileApplicationName($message, $applicationName)
@@ -113,6 +115,7 @@ 
     /**
      * 日付項目を埋め込む
      * @param  string メッセージ
+     * @param string $message
      * @return 埋め込み済みメッセージ
      */
     private function compileDateTime($message)

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
         // メッセージ
         $formattedMessage = preg_replace('/%m/', $message, $formattedMessage);
 
-        return $formattedMessage . "\n";
+        return $formattedMessage."\n";
     }
 
     /**
@@ -69,7 +69,7 @@ 
 
         // 設定ファイルが存在するかどうか
         if ($log === null) {
-            throw new LoggerException("Log config file does not exist: " . $configPath);
+            throw new LoggerException("Log config file does not exist: ".$configPath);
         }
 
         // ログアプリケーション名

WebStream/Module/Cache.php

Doc Comments +2 added lines

@@ -22,6 +22,7 @@ 
     /**
      * コンストラクタ
      * @param string キャッシュの保存ディレクトリパス
+     * @param string $savePath
      */
     public function __construct($savePath = null)
     {
@@ -34,6 +35,7 @@ 
     /**
      * キャッシュを取得する
      * @param string キャッシュID
+     * @param string $id
      * @return string キャッシュデータ
      */
     public function get($id)

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
     {
         $cache = $this->cache($id);
         if ($cache !== null) {
-            $path = $this->savePath . "/" . $id . '.cache';
+            $path = $this->savePath."/".$id.'.cache';
             $cachePath = realpath($path);
             $this->logger->info("Get cache: ${cachePath}");
 
@@ -74,7 +74,7 @@ 
      */
     private function cache($id)
     {
-        $path = $this->savePath . "/" . $id . '.cache';
+        $path = $this->savePath."/".$id.'.cache';
         $cachePath = realpath($path);
         if ($cachePath !== false && is_file($cachePath)) {
             $data = $this->decode(file_get_contents($cachePath));
@@ -114,7 +114,7 @@ 
 
         // キャッシュディレクトリが存在するか
         if (is_dir($this->savePath)) {
-            $cachePath = realpath($this->savePath) . '/' . basename($id) . '.cache';
+            $cachePath = realpath($this->savePath).'/'.basename($id).'.cache';
             // キャッシュファイルがない場合またはキャッシュファイルが存在するが、
             // 上書きする場合はキャッシュを新規作成する
             if (!is_file($cachePath) || (is_file($cachePath) && $overwrite === true)) {
@@ -126,11 +126,11 @@ 
 
                     return true;
                 } else { // ファイル書き込みに失敗した場合
-                    throw new IOException("Can't create cache: " . $cachePath);
+                    throw new IOException("Can't create cache: ".$cachePath);
                 }
             }
         } else {
-            $this->logger->error("Invalid cache directory: " . $this->savePath);
+            $this->logger->error("Invalid cache directory: ".$this->savePath);
         }
 
         return false;
@@ -142,7 +142,7 @@ 
      */
     public function delete($id)
     {
-        $cachePath = realpath($this->savePath . "/" . $id . '.cache');
+        $cachePath = realpath($this->savePath."/".$id.'.cache');
         if ($cachePath) {
             $this->logger->debug("Cache delete success: ${cachePath}");
 

WebStream/Module/ClassLoader.php

Doc Comments +2 added lines

@@ -27,6 +27,7 @@ 
     /**
      * クラスをロードする
      * @param string|array クラス名
+     * @param string $className
      * @return array<string> ロード済みクラスリスト
      */
     public function load($className)
@@ -38,6 +39,7 @@ 
      * ファイルをインポートする
      * @param string ファイルパス
      * @param callable フィルタリング無名関数 trueを返すとインポート
+     * @param string $filepath
      * @return boolean インポート結果
      */
     public function import($filepath, callable $filter = null)

Spacing +18 added lines, -18 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 namespace WebStream\Module;
 
-require_once dirname(__FILE__) . '/Utility/FileUtils.php';
-require_once dirname(__FILE__) . '/../DI/Injector.php';
+require_once dirname(__FILE__).'/Utility/FileUtils.php';
+require_once dirname(__FILE__).'/../DI/Injector.php';
 
 use WebStream\Module\Utility\FileUtils;
 use WebStream\DI\Injector;
@@ -43,13 +43,13 @@ 
     public function import($filepath, callable $filter = null)
     {
         $rootDir = $this->applicationInfo->applicationRoot;
-        $includeFile = $rootDir . "/" . $filepath;
+        $includeFile = $rootDir."/".$filepath;
         if (is_file($includeFile)) {
             $ext = pathinfo($includeFile, PATHINFO_EXTENSION);
             if ($ext === 'php') {
                 if ($filter === null || (is_callable($filter) && $filter($includeFile) === true)) {
                     include_once $includeFile;
-                    $this->logger->debug($includeFile . " import success.");
+                    $this->logger->debug($includeFile." import success.");
                 }
             }
 
@@ -68,7 +68,7 @@ 
     public function importAll($dirPath, callable $filter = null)
     {
         $rootDir = $this->applicationInfo->applicationRoot;
-        $includeDir = realpath($rootDir . "/" . $dirPath);
+        $includeDir = realpath($rootDir."/".$dirPath);
         if (is_dir($includeDir)) {
             $iterator = $this->getFileSearchIterator($includeDir);
             $isSuccess = true;
@@ -81,11 +81,11 @@ 
                     if ($ext === 'php') {
                         if ($filter === null || (is_callable($filter) && $filter($filepath) === true)) {
                             include_once $filepath;
-                            $this->logger->debug($filepath . " import success.");
+                            $this->logger->debug($filepath." import success.");
                         }
                     }
                 } else {
-                    $this->logger->warn($filepath . " import failure.");
+                    $this->logger->warn($filepath." import failure.");
                     $isSuccess = false;
                 }
             }
@@ -111,22 +111,22 @@ 
         // まずcoreディレクトリを検索
         // coreディレクトリは名前空間とディレクトリパスが
         // 紐づいているのでそのまま連結して読ませる
-        $includeFile = $rootDir . "/core/" . $className . ".php";
+        $includeFile = $rootDir."/core/".$className.".php";
         if (is_file($includeFile)) {
             include_once $includeFile;
-            $this->logger->debug($includeFile . " load success. (search from " . $rootDir . "/core/)");
+            $this->logger->debug($includeFile." load success. (search from ".$rootDir."/core/)");
 
             return [$includeFile];
         }
 
         // さらに見つからなかったらappディレクトリを名前空間付きで全検索し、マッチするもの全てをincludeする
-        $iterator = $this->getFileSearchIterator($rootDir . "/app");
+        $iterator = $this->getFileSearchIterator($rootDir."/app");
         $includeList = [];
         foreach ($iterator as $filepath => $fileObject) {
-            if (strpos($filepath, $className . ".php") !== false) {
+            if (strpos($filepath, $className.".php") !== false) {
                 include_once $filepath;
                 $includeList[] = $filepath;
-                $this->logger->debug($filepath . " load success. (search from " . $rootDir . "/app/)");
+                $this->logger->debug($filepath." load success. (search from ".$rootDir."/app/)");
             }
         }
         if (!empty($includeList)) {
@@ -139,12 +139,12 @@ 
             $classNameWithoutNamespace = $matches[1];
             // この処理が走るケースはapp配下のクラスがディレクトリ構成と名前空間が一致していない
             // 場合以外ない(テスト用クラス除く)ので、app配下の検索を優先する
-            $iterator = $this->getFileSearchIterator($rootDir . "/app");
+            $iterator = $this->getFileSearchIterator($rootDir."/app");
             foreach ($iterator as $filepath => $fileObject) {
-                if (strpos($filepath, $classNameWithoutNamespace . ".php") !== false) {
+                if (strpos($filepath, $classNameWithoutNamespace.".php") !== false) {
                     include_once $filepath;
                     $includeList[] = $filepath;
-                    $this->logger->debug($filepath . " load success. (full search)");
+                    $this->logger->debug($filepath." load success. (full search)");
                 }
             }
             if (!empty($includeList)) {
@@ -152,12 +152,12 @@ 
             }
 
             // ここに到達するのはテスト用クラスのみ
-            $iterator = $this->getFileSearchIterator($rootDir . "/core");
+            $iterator = $this->getFileSearchIterator($rootDir."/core");
             foreach ($iterator as $filepath => $fileObject) {
-                if (strpos($filepath, $classNameWithoutNamespace . ".php") !== false) {
+                if (strpos($filepath, $classNameWithoutNamespace.".php") !== false) {
                     include_once $filepath;
                     $includeList[] = $filepath;
-                    $this->logger->debug($filepath . " load success. (full search, use in test)");
+                    $this->logger->debug($filepath." load success. (full search, use in test)");
                 }
             }
             if (!empty($includeList)) {

WebStream/Module/Container.php

Doc Comments -2 removed lines

@@ -72,8 +72,6 @@
 
     /**
      * キーの値を設定する
-     * @param  string $name      メソッド名
-     * @param  array  $arguments 引数リスト
      * @return void
      */
     public function set($key, $value)

WebStream/Module/Functions.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
  */
 
 use WebStream\Module\Security;
-use WebStream\DI\ServiceLocator;
 use WebStream\Log\Logger;
 
 /**

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Functions
- * @author Ryuichi TANAKA.
- * @since 2013/09/04
- * @version 0.7
- */
+     * Functions
+     * @author Ryuichi TANAKA.
+     * @since 2013/09/04
+     * @version 0.7
+     */
 
 use WebStream\Module\Security;
 use WebStream\DI\ServiceLocator;

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
     {
         // ログ処理
         if ($error = error_get_last()) {
-            $errorMsg = $error['message'] . " " . $error['file'] . "(" . $error['line'] . ")";
+            $errorMsg = $error['message']." ".$error['file']."(".$error['line'].")";
             switch ($error['type']) {
                 case E_ERROR:
                 case E_CORE_ERROR:
@@ -92,7 +92,7 @@ 
 if (!function_exists('safetyOutJSON')) {
     function safetyOutJSON($data)
     {
-        return json_encode($data, JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT);
+        return json_encode($data, JSON_HEX_TAG|JSON_HEX_AMP|JSON_HEX_APOS|JSON_HEX_QUOT);
     }
 }
 
@@ -102,6 +102,6 @@ 
 if (!function_exists('safetyOutJSONP')) {
     function safetyOutJSONP($data, $callback)
     {
-        return safetyOutJavaScript($callback) . "(" . safetyOutJSON($data) . ");";
+        return safetyOutJavaScript($callback)."(".safetyOutJSON($data).");";
     }
 }