OpenPNE 3

概要¶

サイドバナーのRSSリーダーを利用する際，URLに特殊文字が含まれる場合に正しいRSSフィードが得られない．

確認手順¶

1. pc_backend.php/design/gadget/type/sideBannerで「RSSリーダー」のガジェットを追加する
2. 「RSSリーダー」をクリックして下記URLを登録する
   

   http://video.baidu.jp/api/search?word=バイク&output=rss&start=0&count=12&sort=dt_recent&adult_filter=1
   

3. ログイン画面等のサイドバナーが表示されるページで表示される結果を確認する
   • 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     }

修正案¶

1. $this->gadgetがsfOutputEscaperIteratorDecoratorクラスでラップせずにGadgetクラスでexecuteRssBox()に渡すようにする
2. 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に置換する．