ryuring
ryuring
@katokaisya この件、やりたい事はわかりますが、後に置くのか、前に置くのかは、デフォルトのテンプレートに依存するだけで、組み方によってはどちらにも置けるんですよね。 組み方が逆になった場合に今度また矛盾が発生してしまいます。 少し話し合って決めなければなりませんが、一旦、その前提にてコードをレビューしますね。
@katokaisya こちらの件、一度、一緒に時間作って考えたいですね。一旦マイルストーンを更新します。
admin-third にも変更が必要
@katokaisya だいぶ前のプルリクではありますが、一旦、レビューは入れてますので、ご確認ください。
> フロント用の関数はBcBaserHelper経由でも呼び出せるようにとの話なので対応方法を検討 こちらに関しては、BlogBaserHelper と BlogHelper の関係のように次の構成でできたらと思います。 https://miro.com/app/board/uXjVOr6f8uM=/?share_link_id=818933006789 ただ、現在は、プラグインしかマジックメソッドで呼び出せないので、コア側はヘルパの登録の仕組みが必要となりそうです。 @seto1
@seto1 これは面白い実装ですね。 設定ファイルのキー名が散らからないようちょっと考えたいところですね。 `BcHelper.callFunction.BcBaserHelper` ？ そういえば、bc_form.php もあるな。これは、Cakeの仕組みでヘルパのオプション指定で読み出してる感じですね。 `BcAdminAppView` に定義があります。
BcBaserHelper::crumbs() は、BcCrumbs::render() に移行 crumbs関連の処理実体は BcCrumbsHelper に持っていく
@kaburk Docker環境では、RewriteBase書かなくても動作するみたいですね。知りませんでした。 削除の方向でお願いします。
@kaburk 報告ありがとうございます！まだまだ問題ありますね、、、 空配列に調整する方向でお願いします〜
@GUSSAN-BEEDAN こちらのコミットで、設定をCSS側に移動しました。改善できるか確認頂けますか？