Fix: CodeRabbitレビュー指摘事項を修正(3回目)

- docblockの「トークン解析」を「正規表現でパース」に修正
- setUp()にDATA_REALDIRの定数チェックを追加
- tearDown()にtmpDir null guardを追加(markTestSkipped時の安全性)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: nobuhiko

data/class/batch/SC_Batch_Update.php

@@ -263,7 +263,7 @@ public function makeDistInfo($bkupDistInfoArray)
     /**
      * distinfo.php を安全にパースして $distinfo 配列を返す.
      *
-     * include を使わず、ファイル内容をトークン解析することで
+     * include を使わず、ファイル内容を正規表現でパースすることで
      * 任意コード実行を防止する.
      *
      * @param string $path distinfo.php のパス

tests/class/batch/SC_Batch_Update_parseDistInfoTest.php

@@ -13,7 +13,7 @@ class SC_Batch_Update_parseDistInfoTest extends PHPUnit_Framework_TestCase
 
     protected function setUp(): void
     {
-        if (!defined('MODULE_REALDIR') || !defined('HTML_REALDIR')) {
+        if (!defined('MODULE_REALDIR') || !defined('HTML_REALDIR') || !defined('DATA_REALDIR')) {
             $this->markTestSkipped('EC-CUBE constants are not defined.');
         }
         $this->batch = new SC_Batch_Update();
@@ -27,9 +27,11 @@ protected function setUp(): void
 
     protected function tearDown(): void
     {
-        // テンポラリファイルを削除
-        array_map('unlink', glob($this->tmpDir.'/*'));
-        rmdir($this->tmpDir);
+        // テンポラリファイルを削除 (setUp がスキップした場合は tmpDir が null のため guard が必要)
+        if ($this->tmpDir !== null && is_dir($this->tmpDir)) {
+            array_map('unlink', glob($this->tmpDir.'/*') ?: []);
+            rmdir($this->tmpDir);
+        }
         if (file_exists('/tmp/pwned.txt')) {
             unlink('/tmp/pwned.txt');
         }