Skip to content

Symfony 7 / EC-CUBE 4.4 対応#150

Open
ji-eunsoo wants to merge 6 commits into
4.4from
feat-4.4
Open

Symfony 7 / EC-CUBE 4.4 対応#150
ji-eunsoo wants to merge 6 commits into
4.4from
feat-4.4

Conversation

@ji-eunsoo

@ji-eunsoo ji-eunsoo commented Jul 10, 2026

Copy link
Copy Markdown

概要

EC-CUBE 4.4(Symfony 7.4 / Doctrine ORM 3.0 / PHP 8.2+)にメルマガ管理プラグインを対応させます。
公式の関連商品プラグイン EC-CUBE/related-product-plugin#91
4.4 マイグレーションドキュメント EC-CUBE/doc4.ec-cube.net#346 に倣った移行です。

4.3 と 4.4 はアノテーション必須/属性必須の違いで非互換のため、4.2 ブランチ(MailMagazine42)とは別に
4.4 ブランチ(MailMagazine44 )で保守します。本 PR は 4.4 ブランチをベースにしています。

主な変更点

Symfony 7 対応

  • Controller: @Route アノテーション → #[Route] 属性へ移行、戻り値型・引数型を明示
  • Form / Service / Nav: buildForm(): void 等の戻り値型・引数型を明示

コード名リネーム MailMagazine42MailMagazine42

  • namespace(Plugin\MailMagazine42\...)/ composer 名(ec-cube/mailmagazine44)/ extra.code / version 4.4.0
  • Twig 名前空間(@MailMagazine42)/ 管理ナビキー / CI plugin_code

PHPUnit 11 対応

  • phpunit.xml.dist を新形式へ(<source> / <extensions>DAMA\...\PHPUnitExtension
  • テストメソッドに : void / 引数型を付与

CI / 開発ツール一式(PR#91 と同等)

  • main.yml: EC-CUBE 4.4 / PHP 8.2-8.5 / MySQL8・PostgreSQL マトリクス、static-analysis ジョブ(php-cs-fixer / rector / phpstan)を追加
  • release.yml: 開発・テスト用ファイルを配布パッケージから除外
  • docker-compose(SQLite/MySQL/PostgreSQL)/ dockerbuild / phpstan.neon.dist / Resource/rector.php / Resource/.php-cs-fixer.dist.php / CLAUDE.md / .gitignore を追加
  • README に Docker Compose でのテスト手順を追記

動作確認(EC-CUBE 4.4 / PHP 8.2 / SQLite、docker-compose)

  • ✅ PHPUnit: 69 tests, 62 assertions, 17 skipped 成功
  • ✅ php-cs-fixer(--dry-run): 差分ゼロ
  • ✅ rector(--dry-run): 変更ゼロ
  • ✅ phpstan level 6: エラーゼロ
  • ✅ プラグインの install / enable / 管理画面ナビ表示・3 画面(配信/テンプレート設定/配信履歴)をテストで確認

🤖 Generated with Cursor

Summary by CodeRabbit

  • 新機能

    • プラグインを EC-CUBE 4.4 対応版へ更新し、管理画面のメルマガ配信・テンプレート・配信履歴機能を引き続き利用できるようにしました。
    • Docker Compose による開発・テスト環境の起動手順を追加し、SQLite/MySQL/PostgreSQL での実行をしやすくしました。
  • 改善

    • ルーティングや画面関連の内部実装を最新形式に整理し、動作の安定性と保守性を向上しました。
  • ドキュメント

    • セットアップ、テスト実行、静的解析の手順を追記しました。

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 48c7820c-c9e6-49a7-a602-2f080ad4e8fd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat-4.4

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant