Merge pull request #6 from appleple/fix/auth-file-path-env

Fix/auth file path env

Author: 1000-x-t30

app/.env.sample

@@ -1,9 +1,9 @@
-# トークンファイルの保存先ディレクトリ
-# 未設定時はドキュメントルートを使用
+# トークンファイルの保存先ファイルパス（絶対パス）
+# 未設定時は <a-blog cmsルート>/private/zoho_tokens.csv を使用
 ZOHO_TOKEN_PERSISTENCE_PATH=
 
-# Zoho SDK ログの出力先ファイルパス
-# 未設定時はドキュメントルートにphp_zoho_sdk.logが出力
+# Zoho SDK ログの出力先ファイルパス（絶対パス）
+# 未設定時は <a-blog cmsルート>/private/php_zoho_sdk.log を使用
 ZOHO_LOGGER_FILE_PATH=
 
 # Zoho API レスポンスのキャッシュ有効期間（秒）

app/POST/Zoho/OAuth2.php

@@ -47,9 +47,10 @@ public function post()
         }
         else {
             AcmsLogger::error('【Zoho plugin】OAuth2認証のためのパラメータが不足しています。');
-            return null;
+            $this->addError('クライアントID、クライアントシークレットを入力してください。');
+            return $this->Post;
         }
 
-        $this->Post;
+        return $this->Post;
     }
 }

app/Services/Zoho/Client.php

@@ -56,14 +56,13 @@ class Client
 
     public function __construct()
     {
-        $documentRoot = $_SERVER['DOCUMENT_ROOT'] ?? '';
-        $this->loggerFilePath = env('ZOHO_LOGGER_FILE_PATH', $documentRoot . '/' . $this->loggerFilePath);
-        $this->tokenPresistencePath = env('ZOHO_TOKEN_PERSISTENCE_PATH', $documentRoot);
+        $scriptDir = defined('SCRIPT_DIR') ? SCRIPT_DIR : ($_SERVER['DOCUMENT_ROOT'] ?? '');
+        $defaultTokenPath = $scriptDir . 'private/zoho_tokens.csv';
+        $defaultLogPath = $scriptDir . 'private/' . $this->loggerFilePath;
 
-        if (empty($this->tokenPresistencePath)) {
-            AcmsLogger::warning('【Zoho plugin】環境変数 ZOHO_TOKEN_PERSISTENCE_PATH が設定されていません。');
-            return;
-        }
+        $this->loggerFilePath = env('ZOHO_LOGGER_FILE_PATH', $defaultLogPath);
+        $tokenPersistencePath = env('ZOHO_TOKEN_PERSISTENCE_PATH', '');
+        $this->tokenPresistencePath = $tokenPersistencePath !== '' ? $tokenPersistencePath : $defaultTokenPath;
 
         $this->store = new CustomFileStore($this->tokenPresistencePath);
     }