課題 #17626
getContentsName() が返す文字列について
| ステータス: | 終了 | 開始日: | ||
|---|---|---|---|---|
| 優先度: | 通常 | 期日: | ||
| 担当者: | 江頭 竜二 | 進捗率: | 100% | |
| カテゴリ: | その他 | |||
| 対象バージョン: | 4.0.5 |
説明
例えば、baserCMSインストール直後のように
- サービス（example.com/service/）
- サービス1（example.com/service/service1）
- サービス2（example.com/service/service2）
- サービス3（example.com/service/service3）
という構造となっているケースにおいて。
「サービス」を表示するテンプレート（デフォルトでは /lib/Baser/View/ContentFolders/default.php ）で
getContentsName() を使ったときに文字列 ContentFolders でなくフォルダ名を返した方がよいのではないでしょうか。
フォルダごとにCSSや表示画像の処理を切り分ける実装が想定されるので、テーマ制作者にとっても親切なのではと考えます。
関係しているリビジョン
fix #17626 BcBaserHelper::getContentsName() 改善
- フロントエンドでフォルダにアクセスした際、想定する挙動になっていなかった為
履歴
#1 江頭 竜二 が10ヶ月前に更新
- 担当者 を . tecking にセット
確認したのですが、/service/ と /service/service1 にアクセスしている際、どちらも、関数が返す値は、「Service」 になってたようでした。
もしかしたら問題点が伝わってないかもしれません。
ちなみに、この関数のテストケースが以下より確認できます。
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Test/Case/View/Helper/BcBaserHelperTest.php#L1176
#5 . tecking が10ヶ月前に更新
- 担当者 を . tecking から 江頭 竜二 に変更
リポジトリから git clone してVagrant環境で新規インストール、デフォルトテーマ（bc_sample）の /Layouts/default.php に
<?php echo $this->BcBaser->getContentsName() ?>
を追記して確認した結果、下記のようになりました。
《/service/ がコンテンツフォルダのみの場合（インストール直後）》- /service/ …… ContentFolders
- /service/service1 …… Service
- /service/ …… Service
- /service/service1 …… Service
コンテンツフォルダのみの場合に「ContentFolders」を返すのは仕様どおりとも解釈できるのですが、固定ページ index を作らない運用も想定される（*）のでオプション値などで対応できた方がよいのかなと思いました。
（*）/lib/Baser/View/ContentFolders/default.php を当てて下層ページのリストを自動生成する など