ブロック管理：ファイル名に「/」を含むブロックのファイル名を変更したときに500エラーになるケースがある

[ryo-endo]

概要(Overview)

ブロック管理ページ。
現状ファイル名に「/」を含むファイルを作成することができます。(例) ec/ec.twig
このときファイル名のスラッシュを増やして ec//ec.twig と変更して保存をすると、500エラーが発生して、twigファイルも消えてしまいます。

期待する内容(Expect) or 要望 (Requirement)

• 連続した「/」は不正なファイル名としてバリデーションエラーになること。
  or
• 連続した「/」は単一の「/」に自動で変換されること。
  or
• ファイル名に「/」を含めたい要件がないなら「/」を使えないようにする。(要互換性考慮)

再現手順(Procedure)

1. ブロック管理ページで新規ブロックを作成開始
2. ファイル名を「ec/ec」にする。他の項目は任意の入力で保存。
3. 上記で作成したブロックを編集開始する。
4. ファイル名を「ec//ec」にして保存。 → 500エラー,このタイミングでtwigファイルが消えている
5. 上記で作成したブロックを再度編集 → 500エラー(twigファイルが存在しないので)

↓gifは再現手順のNo.4まで

ちょっと調べてみたこと

500エラーはファイルが消えたことに起因しています。

[Eccube\Log\Logger:log:64] - システムエラーが発生しました。
 ["Unable to find template \"Block/ec/ec.twig\" (looked into: /var/www/html/app/template/default, /var/www/html/src/Eccube/Resource/template/default, /var/www/html/app/Plugin, /var/www/html/vendor/symfony/twig-bridge/Resources/views/Form).","/var/www/html/vendor/twig/twig/src/Loader/FilesystemLoader.php",250,"
#0 /var/www/html/vendor/twig/twig/src/Loader/FilesystemLoader.php(141): Twig\\Loader\\FilesystemLoader->findTemplate()\n
#1 /var/www/html/src/Eccube/Controller/Admin/Content/BlockController.php(115):

新しいの名前でファイル作成→古い名前のファイル削除、という処理になっていますが、連続した「//」の扱いが上手くできていないため、新旧ファイルパスが同一となりファイルが消えているんじゃないかと推測。

ec-cube/src/Eccube/Controller/Admin/Content/BlockController.php

Line 151 in ff66d0a

// 更新でファイル名を変更した場合、以前のファイルを削除

環境 (environment)

• EC-CUBE: 4.2-beta2
  ※公式のdocker環境で確認

関連情報 (Ref)

なし

[chihiro-adachi]

@ryo-endo
ありがとうございます！
事象確認しました

[chihiro-adachi]

#5567 で対応