fix #4220 【ブログ】特定条件でコピーした記事がコピーできない件を修正

[kaburk]

よろしくお願いします。

[Copilot]

Pull request overview

ブログ記事のコピー処理において、スラッグ（name）やタイトル（title）の長さ制限・重複に起因してコピーに失敗するケース（#4220）を回避するための調整を行うPRです。

Changes:

• コピー時に name / title を _copy 付与前提で長さを切り詰め、255文字制限を超えないように修正
• name が重複する場合にサフィックス（_1〜）を付与してユニーク化するロジックを追加
• PersistenceFailedException の再帰リトライ処理を削除

Comments suppressed due to low confidence (2)

plugins/bc-blog/src/Model/Table/BlogPostsTable.php:683

• 重複チェックが最大10回で打ち切られているため、*_copy / *_copy_1 ... *_copy_10 が既に存在する場合は重複したまま saveOrFail() に進み、コピーが失敗します。上限を設けない（もしくは既存の最大サフィックスを探索して次を採番する）か、上限に達した場合は明示的に失敗（false返却/例外で理由提示）する処理を入れてください。

            for ($i = 1; $i <= 10 && $this->find()->where(['BlogPosts.name' => $data->name])->count(); $i++) {
                $suffix = '_' . $i;
                $data->name = mb_substr($baseName, 0, 255 - mb_strlen($suffix)) . $suffix;
            }

plugins/bc-blog/src/Model/Table/BlogPostsTable.php:725

• saveOrFail() はバリデーション失敗時に PersistenceFailedException を投げますが、ここでは捕捉していないため（以前は name エラー時にリトライしていた）、スラッグ競合が解決できないケースや競合レースで例外が上位へ伝播します。少なくとも name のユニークエラーについては PersistenceFailedException を捕捉して再採番/再試行するか、呼び出し側が扱いやすい形（false返却 or 400相当の扱いにできる例外）に揃えることを検討してください。

        try {
            $result = $this->saveOrFail($this->patchEntity($this->newEmptyEntity(), $arrayData));
            if ($eyeCatch) {
                $result->eye_catch = $eyeCatch;
                $this->renameToBasenameFields($result, true);
                $result = $this->save($result);
            }

            // EVENT BlogPosts.afterCopy
            $this->dispatchLayerEvent('afterCopy', [
                'id' => $result->id,
                'data' => $result,
                'oldId' => $id,
                'oldData' => $oldData,
            ]);
            return $result;
        } catch (BcException $e) {
            throw $e;
        }