Docker環境でのテンプレート開発を改善する

[kurozumi]

背景

Docker 環境でテンプレートをアップロードして適用しようとしても反映されないという問題があります。

原因

管理画面のテンプレート切り替えは .env に ECCUBE_TEMPLATE_CODE を書き込む仕組みになっています。しかし index.php は APP_ENV が環境変数として設定されている場合、.env を一切読み込みません。

// 変更前
if (!isset($_SERVER['APP_ENV'])) {
    if (file_exists(__DIR__.'/.env')) {
        (Dotenv::createUnsafeMutable(__DIR__))->load();
    }
}

Docker 環境では docker-compose.yml で APP_ENV: "dev" を設定しているため、.env が読み込まれず、管理画面からテンプレートを切り替えても反映されません。

変更内容

1. index.php の修正

APP_ENV が設定されている場合でも .env を読み込むように変更しました。createImmutable()->safeLoad() を使用するため、Docker で設定済みの環境変数は上書きしません。

// 変更後
} elseif (class_exists(Dotenv::class) && file_exists(__DIR__.'/.env')) {
    // APP_ENV が環境変数として設定されている場合（Docker など）でも .env を読み込む。
    // ただし既存の環境変数（Docker で設定済みのもの）は上書きしない。
    (Dotenv::createImmutable(__DIR__))->safeLoad();
}

これにより管理画面からテンプレートを切り替えると .env に ECCUBE_TEMPLATE_CODE が書き込まれ、次のリクエストから反映されます。Docker の再起動は不要です。

2. TemplateController に警告を追加

ECCUBE_TEMPLATE_CODE がプロセス環境変数として明示的に設定されている場合（createImmutable では上書きできないため .env の変更が反映されない）、管理画面に警告を表示します。

影響範囲

• APP_ENV が設定されていない既存の環境には影響ありません
• APP_ENV が設定されている環境では .env が追加で読み込まれますが、既存の環境変数は上書きされません

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• 新機能

  • テンプレートコード選択時、環境変数 ECCUBE_TEMPLATE_CODE が設定されている場合に警告メッセージを表示するようになりました。環境変数が設定されている場合は、.env ファイルの変更は適用されず、環境変数の更新が必要です。

• 改善

  • 環境変数 .env ファイルの読み込みロジックを改善しました。外部で設定された環境変数を保持しながら、.env ファイルの値を活用できるようになりました。

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

ウォークスルー

このプルリクエストは、環境変数 ECCUBE_TEMPLATE_CODE が設定されている場合の処理を改善します。index.php では .env ファイルの読み込みロジックを拡張し、TemplateController では警告メッセージを条件付きで表示し、対応する英語と日本語の翻訳を追加しました。

変更内容

コホート / ファイル	概要
環境変数読み込みロジック index.php	APP_ENV が既に設定されている場合、Dotenv::createImmutable() で .env ファイルを安全に読み込むための elseif ブランチを追加しました。
テンプレートコントローラー警告 src/Eccube/Controller/Admin/Store/TemplateController.php	ECCUBE_TEMPLATE_CODE 環境変数が設定されている場合に、admin.store.template.env_override_warning 警告メッセージを条件付きで表示するロジックを追加しました。
翻訳メッセージ src/Eccube/Resource/locale/messages.en.yaml, src/Eccube/Resource/locale/messages.ja.yaml	環境変数オーバーライド警告用の新しいローカライゼーションキー admin.store.template.env_override_warning を英語と日本語で追加しました。

推定コードレビュー工数

🎯 2 (Simple) | ⏱️ ~15 分

詩

🐰 テンプレートの環境、整えられて
警告メッセージで優しく導き
.env の読み込み、スムーズに続く
変数の競合、もう怖くない
ウサギも大喜び、ぴょんぴょん🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Title check	✅ Passed	プルリクエストのタイトル「Docker環境でのテンプレート開発を改善する」は、変更セットの主要な目的を明確に反映しており、Docker環境でのAPP_ENV設定時に.envが読み込まれるようにして、テンプレート開発を改善するという変更内容と一致している。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.80%. Comparing base (1395333) to head (dc22eef).
⚠️ Report is 46 commits behind head on 4.3.

Files with missing lines	Patch %	Lines
...cube/Controller/Admin/Store/TemplateController.php	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                4.3    #6712      +/-   ##
============================================
+ Coverage     78.65%   78.80%   +0.14%     
- Complexity     6824     6825       +1     
============================================
  Files           476      476              
  Lines         27078    27711     +633     
============================================
+ Hits          21299    21838     +539     
- Misses         5779     5873      +94     

Flag	Coverage Δ
Unit	78.80% <0.00%> (+0.14%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[kurozumi]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@index.php`:
- Around line 36-40: The code currently uses
Dotenv::createImmutable(__DIR__)->safeLoad(), but createImmutable does not
include PutenvAdapter so values won't be visible via getenv() (used by
TemplateController to read ECCUBE_TEMPLATE_CODE); change the call to use
Dotenv::createUnsafeImmutable(__DIR__)->safeLoad() so the loaded .env values are
exposed to getenv() while still not overwriting existing env vars.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d97261df-ff03-4721-836a-866c1bae2d35

📥 Commits

Reviewing files that changed from the base of the PR and between cc5a27e and 4081517.

📒 Files selected for processing (4)

• index.php
• src/Eccube/Controller/Admin/Store/TemplateController.php
• src/Eccube/Resource/locale/messages.en.yaml
• src/Eccube/Resource/locale/messages.ja.yaml

✅ Files skipped from review due to trivial changes (2)

• src/Eccube/Resource/locale/messages.en.yaml
• src/Eccube/Resource/locale/messages.ja.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

• src/Eccube/Controller/Admin/Store/TemplateController.php

[dotani1111]

@kurozumi
PRありがとうございます！