Masahiro Chiba

Results 2 issues of Masahiro Chiba

https://github.com/tokuhirom/Amon/blob/master/lib/Amon2/Setup/Flavor/Minimum.pm#L148 ここでconfigに書き込みしてしまっている。 MyApp MyApp::PC -> MyApp MyApp::Admin -> MyApp があるときに(Flavor::Largeで作るとこうなりますね) MyApp->config; MyApp::PC->config; MyApp::Admin->config; の順に呼ぶと https://github.com/tokuhirom/Amon/blob/master/lib/Amon2.pm#L46 ここでconfigメソッドが定義されてしまう関係上 その三つの->configが返すのは同じreferenceになってしまうので 最初に提示した view_conf->{path} が、MyApp::PCとMyApp::Adminで一緒になってしまう。 で、ここはまあ事前に $view_conf = +{ %{$view_conf} }; としてあげれば解決するわけですが もし、Amonのポリシーとしてconfigは読み込み専用としないのであればsub configの実装はまずいのかなと思いました! という、テストケースもつけないバグ報告ですみません><

``` perl #!/usr/bin/env perl use Test::More; use utf8; use Text::Xslate; my $xslate = Text::Xslate->new(); is $xslate->render_string('', {string => "Ä"}) => 'Ä'; is $xslate->render_string('', {string => "\x{c4}"}) => 'Ä'; is $xslate->render_string('あ',...

bug
warnocked