fix: マスタEntityをtraitで拡張するとマスタデータ管理から消える問題を修正

Entity/Master 配下のマスタEntityをプラグインのtraitで拡張すると、
マスタデータ管理ページのプルダウンから該当テーブルが消え、さらに
プラグインインストール時のschema updateがカラムを自動追加しない問題を修正する。

原因は TraitProxyAttributeDriver / ReloadSafeAttributeDriver の
getAllClassNames() で、Proxyパス生成時に basename() を用いて
Entity/Master 等のサブディレクトリ情報を落としていたため。
$sourceFile のパス構造ごと proxies ディレクトリへ変換するよう修正した。

- 親ドライバ TraitProxyAttributeDriver: プルダウン表示 (#5400)
- 子ドライバ ReloadSafeAttributeDriver: install時の自動schema update (#6273)
  の両方を修正 (annotation時代の #5401 + #6274 を4.4で統合的に解決)

回帰テストを Codeception から Playwright へ移植:
- MasterEntityExtension プラグイン (テストデータ)
- MasterDataManagePage / MasterEntityExtensionLocal
- plugin-extend.spec.ts に test_master_entity_extension を追加
- plugin-test.yml の plugin-extend マトリクスに追加

Closes #5400
Closes #6273
Refs #5401 #6274

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: nanasess

.github/workflows/plugin-test.yml

@@ -371,6 +371,7 @@ jobs:
           - test_extend_same_table_disabled_remove_local
           - test_extend_same_table_crossed_store
           - test_extend_same_table_crossed_local
+          - test_master_entity_extension
         include:
           - db: pgsql
             database_url: postgres://postgres:password@127.0.0.1:5432/eccube_db

codeception/_data/plugins/MasterEntityExtension-1.0.0/Entity/DeviceTypeTrait.php

@@ -0,0 +1,30 @@
+<?php
+
+/*
+ * This file is part of EC-CUBE
+ *
+ * Copyright(c) EC-CUBE CO.,LTD. All Rights Reserved.
+ *
+ * http://www.ec-cube.co.jp/
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Plugin\MasterEntityExtension\Entity;
+
+use Doctrine\DBAL\Types\Types;
+use Doctrine\ORM\Mapping as ORM;
+use Eccube\Attribute\EntityExtension;
+use Eccube\Entity\Master\DeviceType;
+
+/**
+ * マスタデータのEntity(Entity/Master配下)をtraitで拡張するサンプル.
+ * Issue #5400 / #6273 の回帰テスト用.
+ */
+#[EntityExtension(DeviceType::class)]
+trait DeviceTypeTrait
+{
+    #[ORM\Column(name: 'notes', type: Types::STRING, nullable: true)]
+    public ?string $notes = null;
+}

codeception/_data/plugins/MasterEntityExtension-1.0.0/composer.json

@@ -0,0 +1 @@
+{"name":"ec-cube/masterentityextension","description":"エンティティ拡張機能(Master)のサンプル","version":"1.0.0","type":"eccube-plugin","require":{"ec-cube/plugin-installer":"^2.0@dev"},"extra":{"code":"MasterEntityExtension","id":10000}}

e2e/models/plugins/master-entity-extension-local.ts

@@ -0,0 +1,22 @@
+import { type Page } from '@playwright/test';
+import { type DbClient } from '../../helpers/db-client';
+import { type PluginTestConfig } from '../../fixtures/plugin-test';
+import { LocalPlugin } from '../local-plugin';
+
+/**
+ * マスタデータのEntity(Entity/Master配下)をtraitで拡張するローカルプラグイン。
+ * Issue #5400 / #6273 の回帰テスト用。
+ * Codeception の MasterEntityExtension_Local に相当。
+ */
+export class MasterEntityExtensionLocal extends LocalPlugin {
+  constructor(page: Page, db: DbClient, config: PluginTestConfig) {
+    super(page, db, config, 'MasterEntityExtension');
+    // Entity/Master 配下のEntityを拡張するtrait。
+    // 有効化でこのtraitがproxyに注入され、mtb_device_type がプルダウンに残ることを検証する。
+    this.traits.set('Plugin\\MasterEntityExtension\\Entity\\DeviceTypeTrait', 'src/Eccube/Entity/Master/DeviceType');
+  }
+
+  static async start(page: Page, db: DbClient, config: PluginTestConfig): Promise<MasterEntityExtensionLocal> {
+    return new MasterEntityExtensionLocal(page, db, config);
+  }
+}

e2e/pages/master-data-manage.page.ts

@@ -0,0 +1,67 @@
+import { type Page, expect } from '@playwright/test';
+
+/**
+ * マスタデータ管理ページ
+ * Codeception の Page\Admin\MasterDataManagePage に相当。
+ * Issue #5400 / #6273: Entity/Master 配下のマスタEntityをtraitで拡張しても
+ * プルダウンから消えないことを検証する。
+ */
+export class MasterDataManagePage {
+  /** マスタデータ選択プルダウン (form1) */
+  static readonly SELECT_SELECTOR = '#admin_system_masterdata_masterdata';
+  /** 更新完了フラッシュ */
+  static readonly ALERT_SUCCESS_SELECTOR = '.c-contentsArea .alert-success';
+
+  constructor(private readonly page: Page) {}
+
+  static async go(page: Page, adminRoute = 'admin'): Promise<MasterDataManagePage> {
+    await page.goto(`/${adminRoute}/setting/system/masterdata`);
+    await expect(page.locator(MasterDataManagePage.SELECT_SELECTOR)).toBeVisible({ timeout: 30_000 });
+    return new MasterDataManagePage(page);
+  }
+
+  /**
+   * プルダウンに指定ラベル(テーブル名)の選択肢が存在するか。
+   * バグがある場合は拡張したマスタのテーブル名が選択肢から消えるため false になる。
+   */
+  async 選択肢が存在する(label: string): Promise<boolean> {
+    const count = await this.page
+      .locator(MasterDataManagePage.SELECT_SELECTOR)
+      .locator('option', { hasText: label })
+      .count();
+    return count > 0;
+  }
+
+  /**
+   * マスタデータを選択して form1 を submit し、編集フォーム(form2)を表示する。
+   * 選択肢が存在しない場合は selectOption がタイムアウトして回帰を検知する。
+   */
+  async 選択(label: string): Promise<this> {
+    await expect(
+      this.page.locator(MasterDataManagePage.SELECT_SELECTOR).locator('option', { hasText: label }),
+      `マスタデータのプルダウンに ${label} が存在する`,
+    ).toHaveCount(1);
+
+    await this.page.locator(MasterDataManagePage.SELECT_SELECTOR).selectOption({ label });
+    await this.page.locator('#form1 button[type="submit"]').click();
+    await this.page.waitForLoadState('load');
+
+    // 選択後は編集フォーム(form2)が表示される
+    await expect(this.page.locator('#form2')).toBeVisible({ timeout: 30_000 });
+
+    return this;
+  }
+
+  /**
+   * 編集フォーム(form2)を保存し、更新完了メッセージを確認する。
+   */
+  async 保存(): Promise<this> {
+    await this.page.locator('#form2 .c-conversionArea button[type="submit"]').click();
+    await this.page.waitForLoadState('load');
+    await expect(this.page.locator(MasterDataManagePage.ALERT_SUCCESS_SELECTOR)).toContainText('保存しました', {
+      timeout: 30_000,
+    });
+
+    return this;
+  }
+}

e2e/tests/plugin-extend.spec.ts

@@ -3,6 +3,8 @@ import { HorizonStore } from '../models/plugins/horizon-store';
 import { HorizonLocal } from '../models/plugins/horizon-local';
 import { BoomerangStore } from '../models/plugins/boomerang-store';
 import { BoomerangLocal } from '../models/plugins/boomerang-local';
+import { MasterEntityExtensionLocal } from '../models/plugins/master-entity-extension-local';
+import { MasterDataManagePage } from '../pages/master-data-manage.page';
 import { emptyDir } from '../helpers/tar-helper';
 
 test.describe('Plugin Extend', () => {
@@ -111,4 +113,22 @@ test.describe('Plugin Extend', () => {
     await boomerang.無効化();
     await boomerang.削除();
   });
+
+  // Issue #5400 / #6273: Entity/Master 配下のマスタEntityをtraitで拡張すると、
+  // マスタデータ管理ページのプルダウンから拡張したテーブルが消えてしまう回帰の防止。
+  test('test_master_entity_extension', async ({ page, db, config }) => {
+    const plugin = await MasterEntityExtensionLocal.start(page, db, config);
+
+    await plugin.インストール();
+    await plugin.有効化();
+
+    // 拡張したマスタデータ(mtb_device_type)がプルダウンに残っており、選択・保存できること
+    const masterPage = await MasterDataManagePage.go(page, config.adminRoute);
+    expect(await masterPage.選択肢が存在する('mtb_device_type'), 'mtb_device_type がプルダウンに存在する').toBe(true);
+    await masterPage.選択('mtb_device_type');
+    await masterPage.保存();
+
+    await plugin.無効化();
+    await plugin.削除();
+  });
 });

src/Eccube/Doctrine/ORM/Mapping/Driver/ReloadSafeAttributeDriver.php

@@ -99,9 +99,11 @@ public function getAllClassNames(): ?array
                     $projectDir = str_replace('\\', '/', $projectDir);
                 }
 
-                // Replace /path/to/ec-cube to proxies path
-                $proxyFile = str_replace($projectDir, $this->trait_proxies_directory, $path).'/'.basename((string) $sourceFile);
-                if (file_exists($proxyFile)) {
+                // Replace /path/to/ec-cube to proxies path.
+                // Entity/Master 等のサブディレクトリ配下のクラスでもProxyパスが正しく解決されるよう、
+                // basename()でファイル名だけに丸めず$sourceFileのパス構造ごと変換する（Issue #5400 / #6273）.
+                $proxyFile = str_replace($projectDir, $this->trait_proxies_directory, (string) $sourceFile);
+                if ($proxyFile !== $sourceFile && file_exists($proxyFile)) {
                     $sourceFile = $proxyFile;
                 }
 

src/Eccube/Doctrine/ORM/Mapping/Driver/TraitProxyAttributeDriver.php

@@ -78,9 +78,11 @@ public function getAllClassNames(): ?array
                     $sourceFile = str_replace('\\', '/', $sourceFile);
                     $projectDir = str_replace('\\', '/', $projectDir);
                 }
-                // Replace /path/to/ec-cube to proxies path
-                $proxyFile = str_replace($projectDir, $this->trait_proxies_directory, $path).'/'.basename((string) $sourceFile);
-                if (file_exists($proxyFile)) {
+                // Replace /path/to/ec-cube to proxies path.
+                // Entity/Master 等のサブディレクトリ配下のクラスでもProxyパスが正しく解決されるよう、
+                // basename()でファイル名だけに丸めず$sourceFileのパス構造ごと変換する（Issue #5400 / #6273）.
+                $proxyFile = str_replace($projectDir, $this->trait_proxies_directory, (string) $sourceFile);
+                if ($proxyFile !== $sourceFile && file_exists($proxyFile)) {
                     require_once $proxyFile;
 
                     $sourceFile = $proxyFile;