bake, CakePHP 本体のコード変更に追随する

[mamy1326]

課題

twig や Class など、vendor 配下からコピーしてきて継承しているクラスがある。
これらのファイルは、bake や CakePHP のバージョンアップでの変更に追随する必要がある。

起きていること

以下のような不都合が生じる。
#10

対応案

CI で 以下を検出できないか。

• vendor 配下の対象 twig, class と、plugin のファイルの差分検知
• 差分があれば CI を通さない

[isanasan]

@mamy1326
これは僕も #10 やりながら考えてましたー
僕のアイデアとしては以下かなと

1. entity.twigとtable.twigはvendor配下を直接利用する
2. CIの定期実行で本家bakeのtemplateと差分チェックして差分があれば自動で更新PRを出す

[mamy1326]

@isanasan
「1.」が実現可能ならそれが一番いいですねー。そうなると Class 側で調整になるかな。
それでも、Class に差分でたら吸収する必要はあるんですよね。

例えば以下の bakeExtendedTable メソッドは
https://github.com/LancersDevTeam/cakephp-generation-gap-pattern-model-baker/blob/master/src/Command/ExtendedModelCommand.php#L52

vendor/cakephp/bake/src/Command/ModelCommand.php の bakeTable メソッドをコピーして機能追加してたりします。
同じことが bakeEntity にも言えます。

template は確か、twig を操作する必要があって plugin 配下に持っていったんですが、そうなると Bake の Class と twig で管理対象増えちゃうので、できれば Class だけにしたいです。

この辺は先々自分がやらねば…と思っていたヤツなだけに、いさなさんに任せちゃってるのが申し訳ないっす。

[isanasan]

@mamy1326

なるほどー
baked配下に作成するファイルは本家のメソッドを呼んでいるだけだったのでこいつを継承してtemplateのパスを変えてやればいいかと思っていたのですが、そうすると結局Class側で差分が出た時に困るという話ですね？
いたちごっこになってしまうわけですねー。うーむ。

こちらとしてはモーマンタイです！
やれることをガンガンやっていく所存です！

[mamy1326]

@isanasan

baked配下に作成するファイルは本家のメソッドを呼んでいるだけだったのでこいつを継承してtemplateのパスを変えてやればいいかと思っていた

そこはそのままで合ってるんですけど、baked を継承した Model を自動生成する以下の部分でメソッドを丸々 bakeTable -> bakeExtendedTable, bakeEntity -> bakeExtendedEntity とコピーしてるんですよね。
https://github.com/LancersDevTeam/cakephp-generation-gap-pattern-model-baker/blob/master/src/Command/ExtendedModelCommand.php#L39-L40

で、vendor 配下の bakeTable メソッドとの差分は実は以下くらいしかないんです。

https://github.com/LancersDevTeam/cakephp-generation-gap-pattern-model-baker/blob/master/src/Command/ExtendedModelCommand.php#L84

        // template 定義
        $out = $renderer->generate('Model/extended_table');

これを既存をオーバーライドまたはパスをオブジェクトのプロパティに指定で実現したかったけどやり方がわからなかったのでメソッドコピーして実装したんですよねー。

結局Class側で差分が出た時に困るという話ですね？

ですです。その通りです。実装時に気付いていたので Issue 化しとくべきでした。
初めてプラグイン作って公開してインストールして動かして、ってやってて満足してしまってたかもしれないw

ほんと頼りにしてるしいさなさんのコードや対応は勉強になるので可能な限りしっかり見ていきます！
たまに見落としてたら急かしてくださいw