OpenPNE 3

note-8で指摘された問題の原因¶

本チケットに記載されたRSSフィードのURLにおいては，下記部分で$entry['date']で空文字列が与えられていることが原因です．
ここで$entry['date']が空文字列であった場合でも適切な文字列をop_format_date()に与えることで解決できると思われます．

apps/pc_frontend/modules/default/templates/_rssBox.php

  6 <?php foreach ($result[1] as $entry): ?>
  7 <li><span class="date"><?php echo op_format_date($entry['date'], 'XShortDateJa') ?></span>
  8 <?php echo link_to($entry['title'], $entry['link']) ?>

その他問題に関して¶

このチケットの例で得られるRSSフィードがおかしいので以下の部分でRSSフィードをフェッチしている部分に注目してみました．

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     }

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

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

メモ¶

apps/pc_frontend/modules/default/actions/components.class.php:101で得られる$this->gadgetをget_class($this->gadget)した結果．

sfOutputEscaperIteratorDecorator

その他問題追記¶

サイドバーガジェット設定画面（pc_backend.php/design/gadget/type/sideBanner）において，ガジェットの設定画面において「設定変更」部分に「URL」の欄が存在します．
この「URL」の欄にも気になる点があったので調べてみました．

確認内容¶

ガジェットの設定画面のRSSリーダーガジェット設定における「設定変更」部分「URL」の文字列がデコードされているかどうか．

確認手順¶

1. サイドバーガジェット設定画面でRSSリーダーを２つ追加
2. それぞれのRSSリーダーにおいて「URL」に以下の２つのURLを入力して設定を保存
   • RSSリーダー１

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

   • RSSリーダー２

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

3. その後「設定変更」部分に「URL」の欄を確認
   • RSSリーダー１

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

   • RSSリーダー２

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

4. 下記のようにコードを編集して確認．
   lib/form/doctrine/GadgetConfigForm.class.php:43行目に値を出力するコードを挿入．
   

    40       $config = Doctrine::getTable('GadgetConfig')->retrieveByGadgetIdAndName($gadget->getId(), $key);
    41       if ($config)
    42       {
    43         var_dump($config->getValue());
    44         $this->setDefault($key, $config->getValue());
    45       }
   

   • RSSリーダー１

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

   • RSSリーダー２

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

まとめ¶

以上の結果より，RSSリーダー設定部の「URL」の欄は，入力され保存されているデータそのままを表示していないということが考えられます．よってnote-10で述べたその他問題に関して修正を施したとしても適切な動作の確認を行えない可能性があります．

修正案¶

管理画面ガジェット設定のガジェット編集部においてデコードを行わない実装を行う．

下記のような修正を行った際にURL1，URL2でRSSは違う結果が表示されるが管理画面では同じURLに見える現象が生じたことを確認．

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);

• URL1
  

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

• URL2
  

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

note-11の問題は #2407 にチケットを作成しました．テスト実施者は入力するURLに注意してテストを実施してください．

更新履歴 7a31a02c817da282a82ba6b2fb6075371b2d55c1 で適用されました。

note-10のその他問題に関して，本チケットの趣旨とは違うと思われるため別チケット #2408 で対応することにします．そのため本チケットで対応した 7a31a02c についてはrevertを行います．

更新履歴 45a62891d1d70fb9cfe88757ba4b6843aa492446 で適用されました。