課題 #17626

getContentsName() が返す文字列について

. tecking10ヶ月前に追加. 10ヶ月前に更新.

ステータス:終了開始日:
優先度:通常期日:
担当者:江頭 竜二進捗率:

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や表示画像の処理を切り分ける実装が想定されるので、テーマ制作者にとっても親切なのではと考えます。


関係しているリビジョン

リビジョン 4604a41e
江頭 竜二10ヶ月前に追加

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

#2 江頭 竜二10ヶ月前に更新

あ、もしかして管理画面側という事でしょうか?
もし、そうであれば、getContentsName() は、フロントエンド用の関数であり、管理画面はサポートしてないです。

#3 . tecking10ヶ月前に更新

あ、もしかして管理画面側という事でしょうか?
もし、そうであれば、getContentsName() は、フロントエンド用の関数であり、管理画面はサポートしてないです。

フロント側で getContentsName() を使ったときの話なのですが、もう一度こちらで環境つくって確認してみますね。

#4 江頭 竜二10ヶ月前に更新

お願いしますー

#5 . tecking10ヶ月前に更新

  • 担当者. tecking から 江頭 竜二 に変更

リポジトリから git clone してVagrant環境で新規インストール、デフォルトテーマ(bc_sample)の /Layouts/default.php に

<?php echo $this->BcBaser->getContentsName() ?>

を追記して確認した結果、下記のようになりました。

《/service/ がコンテンツフォルダのみの場合(インストール直後)》
  • /service/ …… ContentFolders
    • /service/service1 …… Service
《/service/ 配下に固定ページ index を追加した場合》
  • /service/ …… Service
    • /service/service1 …… Service

コンテンツフォルダのみの場合に「ContentFolders」を返すのは仕様どおりとも解釈できるのですが、固定ページ index を作らない運用も想定される(*)のでオプション値などで対応できた方がよいのかなと思いました。

(*)/lib/Baser/View/ContentFolders/default.php を当てて下層ページのリストを自動生成する など

#6 江頭 竜二10ヶ月前に更新

/service/ …… ContentFolders

これは仕様どおりではないですね。あくまで、URLをベースとして生成される仕様です。
バグだと思いますので、再度調査しますね。

#7 江頭 竜二10ヶ月前に更新

  • カテゴリその他 にセット
  • 対象バージョン4.0.5 にセット

再現しました。これからなおします。

#8 江頭 竜二10ヶ月前に更新

  • ステータス担当 から 解決 に変更
  • 進捗率0 から 100 に変更

更新履歴 basercms|commit:4604a41eaaf355fc511c6bb3436e0db54b87ef16 で適用されました。

#9 江頭 竜二10ヶ月前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF