固定ページ オプション欄のアイキャッチが保存されない

[kaburk]

概要

固定ページ オプション欄のアイキャッチが保存されないようです。
環境依存かとも思いましたので、Docker、MAMPそれぞれで試してみましたが保存されていないようでした。

ファイルとDB両方空のようです。

エラーログなどログが出なくてちょっと保存されない理由が不明でした。

ただ、ブログ設定のアイキャッチなども同様だったのでcontentsテーブルの保存処理あたりが怪しそうです。

baserCMS version : 5.0.7

[ryuring]

@kaburk 問題自体は解決しましたが、仕様的な課題が残っているためオープンします。

@seto1 こちらの件ですが、以前、BcFileUploader に、_bc_upload_id をエンティティに組込処理があったと思いますが、その際、エンティティの $_accessible プロパティに、* または、_bc_upload_id を明示していないと、配列をエンティティ化する際に、消えてしまうようです。

仕様的にわかりにくいので、他にいい方法ないですかね？？

[seto1]

@ryuring なるほど... ブログ記事のアイキャッチしか動作確認できてませんでした。

未検証ですがぱっと思いついたのは以下です。
Bが実現性が高そうです。

• A. BcUploadBehaviorからエンティティの_accessibleに動的に_bc_upload_idを追加する
  • エンティティに変更を加える方法が問題
    • beforeMarshalから$event->getSubject()->getEntityClass()やsetEntityClass?
• B. BcUploadBehaviorのbeforeMarshalからafterMarshalに一時変数使って_bc_upload_idを渡す
• C. リクエストデータやエンティティに_bc_upload_idをもたせるのをやめる
  • その場合はキーにする値が問題
    • 更新時はidを使えるが作成時はidが存在しない

[ryuring]

@seto1 悩ましいですね

• A：_accessible へ外部から介入するのが気持ち悪い
• B：バグ生みそう
• C：そもそもの問題

実行時というより実装時の問題なので、実装時に、_bc_upload_id にアクセスできるようになってなかったら例外投げるってのはどうでしょう？

「BcUploderBehahavior を利用するには、ファイル保存対象のエンティティについて、_bc_upload_id フィールドをアクセスできるようにしてください。」など

[seto1]

@ryuring あまりルールは増やしたくないですね...

BcUploadBehavior->beforeMarshal に以下の変更を加えると対応できそうなのですがいかがでしょう。

• 引数追加
  • ArrayObject $options
• 一行追加
  • $options['accessibleFields']['_bc_upload_id'] = true;

[ryuring]

@seto1 おー、マジっすか、それでいけるなら一番良さそうですね。お手隙でいいのでお願いしていいですか