baserproject/bcaddonmigrator

baserCMS Add-On Migrator

bcaddonmigrator's Issues

テーマ名がCamelCaseでないテーマをマイグレーションしてもファイルの中身まで置換されない

baserCMS 3 betaの動作検証で利用させていただいています。

テーマのマイグレーションを実行した際の問題です。
テーマ名がアッパーキャメルケースに沿っているテーマは正常にマイグレーションされますが、そうでないテーマはヘルパーファイルとビューファイルの中身の置換が動きませんでした。

原因

    public function migrateTheme($theme) {

            $this->migrateThemeStructure($theme);
            $newTheme = Inflector::camelize($theme);

            $themePath = WWW_ROOT . 'theme' . DS . $newTheme;                
            $this->migrateHelper($themePath . DS . 'Helper');
            $this->migrateView($themePath);

    }

BcAddonMigrator3Componentの上記メソッドを見ると、
新しいテーマ名$newThemeは元のテーマ名$themeをアッパーキャメルケースに変換したものになっています。
しかし、元のテーマ名と新しいテーマ名が異なる場合（＝元のテーマ名がアッパーキャメルケースではない場合）でも、テーマフォルダ自体はリネームされていないように見受けられます。

この結果、migrateHelperとmigrateViewの引数に存在しないディレクトリのパスが渡されてしまいます。

マイグレーション対象のテーマフォルダをアッパーキャメルケースにリネームするか、新しいテーマのパスを使わず元のテーマのパスを引数に渡すように変更する必要があるかと思いますが、いかがでしょうか？

また、CakePHP2系ではテーマ名もアッパーキャメルケースが標準だったと記憶しています。baserCMS 3からテーマ名はアッパーキャメルケースに統一することになるのでしょうか？