操作
Bug（バグ） #2408
完了
サイドバナーのRSSリーダーでURLに特殊文字が含まれる場合に正しいRSSフィードが得られない
開始日:
2011-09-17
期日:
進捗率:
100%
予定工数:
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
説明
概要¶
サイドバナーのRSSリーダーを利用する際，URLに特殊文字が含まれる場合に正しいRSSフィードが得られない．
確認手順¶
- pc_backend.php/design/gadget/type/sideBannerで「RSSリーダー」のガジェットを追加する
- 「RSSリーダー」をクリックして下記URLを登録する
http://video.baidu.jp/api/search?word=バイク&output=rss&start=0&count=12&sort=dt_recent&adult_filter=1
- ログイン画面等のサイドバナーが表示されるページで表示される結果を確認する
- Firefox等の別のRSSリーダーで表示される結果と違う
原因¶
下記コード部において$this->gadgetがGadgetクラスをsfOutputEscaperIteratorDecoratorクラスでラップされているため，特殊文字がエスケープされた文字列が取得されてしまう．
apps/pc_frontend/modules/default/actions/components.class.php
98 try
99 {
100 $fetcher = new opRssFetcher('UTF-8');
101 $this->result = @$fetcher->fetch($this->gadget->getConfig('url'), true);
102 if ($this->result)
103 {
104 $this->result[1] = array_slice($this->result[1], 0, 5);
105 }
106 }
修正案¶
- $this->gadgetがsfOutputEscaperIteratorDecoratorクラスでラップせずにGadgetクラスでexecuteRssBox()に渡すようにする
- executeRssBox()内で$this->gadgetのsfOutputEscaperIteratorDecoratorのラップを剥がして値を取得する
実装案¶
修正案1の実装方法が確認できなかったため修正案2について実装案を提示．おそらく以下の修正で修正案2に相当すると思われる．
diff --git a/apps/pc_frontend/modules/default/actions/components.class.php b/apps/pc_frontend/modules/default/actions/components.class.php
index ad79046..06b97c4 100644
--- a/apps/pc_frontend/modules/default/actions/components.class.php
+++ b/apps/pc_frontend/modules/default/actions/components.class.php
@@ -98,7 +98,7 @@ class defaultComponents extends sfComponents
try
{
$fetcher = new opRssFetcher('UTF-8');
- $this->result = @$fetcher->fetch($this->gadget->getConfig('url'), true);
+ $this->result = @$fetcher->fetch(sfOutputEscaper::unescape($this->gadget)->getConfig('url'), true);
if ($this->result)
{
$this->result[1] = array_slice($this->result[1], 0, 5);
確認バージョン¶
OpenPNE 3.7.0-dev, 3.6beta13
実際の修正¶
各アプリケーションのconfig/settings.ymlのescaping_strategy部分を'on'からtrue，'off'からfalseに置換する．
操作