Project

General

Profile

Bug(バグ) #3350

レイアウトファイル内ローカルナビゲーションのコンポーネント呼び出しでオプションが考慮されていない

Added by Yuya Watanabe about 9 years ago. Updated over 2 years ago.

Status:
New(新規)
Priority:
Low(低め)
Assignee:
-
Target version:
Start date:
2013-05-13
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

概要

レイアウト部分では下記の様に localNavOptions が設定されているが,この設定をコンポーネント側で考慮していない.
実害としてはいまのところ特に無さそうだが,レイアウト自身をカスタマイズしたり別のレイアウトを用いている場合に影響がある可能性がある. localNav コンポーネントで指定するか,コンポーネント呼び出し時に指定するかなど統一しておいたほうがよさそう.

apps/pc_frontend/templates/_layout.php
 33 $localNavOptions = array(
 34   'is_secure' => opToolkit::isSecurePage(),
 35   'type'      => sfConfig::get('sf_nav_type', sfConfig::get('mod_'.$module.'_default_nav', 'default')),
 36   'culture'   => $context->getUser()->getCulture(),
 37 );
 38 if ('default' !== $localNavOptions['type'])
 39 {
 40   $localNavOptions['nav_id'] = sfConfig::get('sf_nav_id', $context->getRequest()->getParameter('id'));
 41 }
 42 include_component('default', 'localNav', $localNavOptions);

コンポーネント部分.

apps/pc_frontend/modules/default/actions/components.class.php
 22   public function executeLocalNav()
 23   {
 24     if (!opToolkit::isSecurePage()) {
 25       return sfView::NONE;
 26     }
 27 
 28     $context = sfContext::getInstance();
 29     $module = $context->getActionStack()->getLastEntry()->getModuleName();
 30     $action = $context->getActionStack()->getLastEntry()->getActionName();
 31 
 32     $this->type = sfConfig::get('sf_nav_type', sfConfig::get('mod_' . $module . '_default_nav', 'default'));
 33 
 34     $this->navs = Doctrine::getTable('Navigation')->retrieveByType($this->type);
 35 
 36     if ('default' !== $this->type)
 37     {
 38       $this->navId = sfConfig::get('sf_nav_id', $context->getRequest()->getParameter('id'));
 39     }
 40   }

History

#1 Updated by Chiharu Nakajima over 7 years ago

  • Priority changed from Normal(通常) to Low(低め)

#2 Updated by Chiharu Nakajima over 7 years ago

このチケットの内容は下記の項目に該当する不具合であるため、一旦優先度を下げます。
・未対応でもユーザーに影響が少ない不具合(回避可能な不具合である等)

#3 Updated by kaoru n over 2 years ago

  • Target version set to OpenPNE 3.10.x

Also available in: Atom PDF