madapaja/madapaja.twigmodule

BEAR.Sunday Module

madapaja.twigmodule's Issues

loadTemmplate is @internal method

It should be avoided to use it.

Version number 2.0

Why release v2.0.0 ? I think it should be 1.x. (=1.2.0).
But wait, we need some other discussion for stable release.

2.xをデフォルトブランチに

@madapaja settingから変更お願いします

拡張子

.twigの方が.html.twigより一般的ではないかと思ったのですがどうでしょうか？

$ro->templatePath ? テンプレートの指定方法

    /**
     * return template file full path
     */
    private function getTemplatePath(ResourceObject $ro) : string
    {
        if (isset($ro->templatePath) && ! empty($ro->templatePath)) {
            return $ro->templatePath;
        }

何かのプロパティ('templatePath')が特定のコンポーネント(Twig)にとって特別な意味がある設計というのはどうでしょうか？ 特定プロパティに特別な意味を持たせて、外部がそれを目印に振る舞いを変えるのはCakePHPやLaravelなどでもありますがそれはどうでしょうか？

テンプレートの場所

テンプレートはどこに置けばいいのだろうか、、、
https://twitter.com/madapaja/status/552062047580475392

0.xと同じようにクラスファイルと同じ場所はどうでしょうか。

ProdModule

今のTwigModuleはデフォルトでdebugオプションがオンになってますが、これをオフにしたprodコンテキスト用のモジュールが必要だと思いますがどうでしょうか。

Twig extension を追加したい場合、どのようにすればいいか？

TwigModule をinstallすると、Twig_Environment がDIされるようになるので、そのインスタンスを取得して、そこからextensionの追加などできそうですが、具体的にどこにどのように記述すればいいか

Madapaja.TwigModule下のpageリソースで Link アトリビュートが機能しない

再現方法

BEAR.Sundayのスケルトンから始めて再現するまでのコミットを、次のリポジトリへ積みました。
Kimita/twigmodule-issue@7546748

状況

BEAR.Sunday のチュートリアル２:ticketsリソース で例示されているような Linkアトリビュート は、Madapaja.TwigModule の導入によってpageリソースのレンダラがHTML出力に変わると、pageリソース側では効かなくなるもよう。

考察

• どうやら、BEAR\Resource\HalRenderer::renderHal() 内で行われているような処理がMadapaja\TwigModule\TwigRenderer では行われていないことによるもののようだ。
• ひとまず、このリンク先にて積んだコミットで問題を回避してみている。
• 上述の回避策では現状、自作の AnnotationResolver で _embedded への埋め込みまでやってしまっている。しかし、BEAR.Sundayのチュートリアルで示されている内容からすると、Embedアトリビュートには触れずに「Linkアトリビュートを _links へ埋め込む」だけを処理対象とすべきなのかも？

上記で示した回避策がもし妥当ならば、今回自作した AnnotationResolver は次のように配置＆使用されるのが理想か？

• BEAR.Resource パッケージで AnnotationResolver (に相当するクラス)を提供する
• Madapaja.TwigModule のようなプレゼンテーション層向けのパッケージ側で、レンダリング処理を行うクラスにて適宜 AnnotationResolver (に相当するクラス)を任意使用する