Project

General

Profile

Backport(バックポート) #2772

サイドバナーのRSSリーダーで読み込めない時がある

Added by Yuma Sakata over 7 years ago. Updated over 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2010-10-15
Due date:
% Done:

100%


Description

現象

(http://sns.openpne.jp/communityTopic/6318 より転記)

サイドバナーへRSSリーダーを登録しているのですが、
読み込み先によって、
画面の表示が「サーバーが混雑しています~」のような表示になってしまいます。

再現環境

サーバー:

  • CentOS Linux 5.5
  • MySQL version 5.0.91
  • php 5.2.13

OpenPNEのバージョン:

  • OpenPNE3.4.6.2

再現手順

こちらのRSSを登録する(8月28日23時ごろ)
http://video.baidu.jp/api/search?word=%E3%83%90%E3%82%A4%...

最初はずっと表示されていましたので、時間がたつと内容が変わりますので、表示されるようになるかもしれません。

サーバーのエラーログ:

PHP Fatal error: Uncaught exception 'sfException' with message 'Impossible to parse date "" with format "yyyy-MM-dd HH:mm:ss".'
 in /www/lib/vendor/symfony/lib/i18n/sfDateFormat.class.php:186\n
 Stack trace:\n
 #0 /www/lib/vendor/symfony/lib/i18n/sfDateFormat.class.php(219): sfDateFormat->getDate('', NULL)\n
 #1 /www/lib/vendor/symfony/lib/helper/DateHelper.php(60): sfDateFormat->format('', 'MM???dd???', NULL, 'utf-8')\n
 #2 /www/lib/helper/opUtilHelper.php(333): format_date('', 'MM???dd???', 'ja_JP', NULL)\n
 #3 /www/apps/pc_frontend/modules/default/templates/_rssBox.php(7): op_format_date('', 'XShortDateJa')\n
 #4 /www/cache/apache/pc_frontend/prod/config/config_core_compile.yml.php(3810): require('...')\n
 #5 /www/lib/vendor/symfony/lib/view/sfPartialView.class.php(110): sfPHPView->renderFile('...')\n
 #6 /www/lib/vendor/symfony/lib/helper/PartialHelper.php(155): sfPartialView->render()\n
 #7 /var/www/r in /www/lib/vendor/symfony/lib/i18n/sfDateFormat.class.php on line 186

原因

規定外のRSSを受信した場合などに、パースに失敗し例外により処理が中断されるため。

修正内容

ガジェット側ですべての例外を catch して、適切な表示になるように修正するべきです。

具体的には、note-3 にあるように

diff --git a/apps/pc_frontend/modules/default/actions/components.class.php b/apps/pc_frontend/modules/def
index 097703c..ad79046 100644
--- a/apps/pc_frontend/modules/default/actions/components.class.php
+++ b/apps/pc_frontend/modules/default/actions/components.class.php
@@ -95,11 +95,17 @@ class defaultComponents extends sfComponents

   public function executeRssBox()
   {
-    $fetcher = new opRssFetcher('UTF-8');
-    $this->result = @$fetcher->fetch($this->gadget->getConfig('url'), true);
-    if ($this->result)
+    try
+    {
+      $fetcher = new opRssFetcher('UTF-8');
+      $this->result = @$fetcher->fetch($this->gadget->getConfig('url'), true);
+      if ($this->result)
+      {
+        $this->result[1] = array_slice($this->result[1], 0, 5);
+      }
+    }
+    catch (Exception $e)
     {
-      $this->result[1] = array_slice($this->result[1], 0, 5);
     }
   }
のような変更で十分かと思われます。


Related issues

Related to OpenPNE 3 - Bug(バグ) #1684: サイドバナーのRSSリーダーで読み込めない時がある Fixed(完了) 2010-10-15

Associated revisions

Revision cd3f99d4 (diff)
Added by Naoya Tozuka over 7 years ago

catch all fatal exceptions thrown in RSS-reader gadget (refs #2772, BP from #1684)

BP from #1684
ceae6c568fb1c1ece7c60e1fa40f7aef80cfeee2

Revision a27e2686 (diff)
Added by Yuya Watanabe over 7 years ago

(fixes #2772, BP from #1684) fixed to show sidebar also when RSSBox got an empty string

BP from #1684
1d880d6c61e8b6cf4c1200170c6eb0d3e01b0da9

Conflicts:

apps/pc_frontend/i18n/messages.ja.xml

History

#1 Updated by Yuya Watanabe over 7 years ago

  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#3 Updated by Kousuke Ebihara over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#4 Updated by Yuma Sakata over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

Also available in: Atom PDF