[Svn-src-all:3925] [version-2_13-dev 22839] #2253 (スマートフォン 新着情報の個別記事表示が動作しない)

[Seasoft]

Subversion committed to /home/svn/open 22839
http://svn.ec-cube.net/open_trac/changeset/22839
┌────────────────────────────┐
│更新者　:  Seasoft                                      │
│更新日時:  2013-05-27 23:04:45 +0900 (月, 27  5月 2013)│
└────────────────────────────┘

Log:
--------------------------------------------------------
#2253 (スマートフォン 新着情報の個別記事表示が動作しない)

Changed:                      [U:修正，A:追加，D:削除]
--------------------------------------------------------
U   branches/version-2_13-dev/data/Smarty/templates/sphone/frontparts/bloc/news.tpl
U   branches/version-2_13-dev/data/class/helper/SC_Helper_News.php
U   branches/version-2_13-dev/data/class/pages/frontparts/bloc/LC_Page_FrontParts_Bloc_News.php

変更: branches/version-2_13-dev/data/Smarty/templates/sphone/frontparts/bloc/news.tpl
===================================================================
--- branches/version-2_13-dev/data/Smarty/templates/sphone/frontparts/bloc/news.tpl	2013-05-23 09:55:02 UTC (rev 22838)
+++ branches/version-2_13-dev/data/Smarty/templates/sphone/frontparts/bloc/news.tpl	2013-05-27 14:04:45 UTC (rev 22839)
@@ -100,8 +100,8 @@
     var loadingState = 0;
     function getNewsDetail(newsId) {
         if (loadingState == 0) {
+            loadingState = 1;
             $.mobile.showPageLoadingMsg();
-            loadingState = 1;
             $.ajax({
                 url: "<!--{$smarty.const.ROOT_URLPATH}-->frontparts/bloc/news.php",
                 type: "GET",
@@ -120,8 +120,8 @@
                         $.mobile.hidePageLoadingMsg();
                         loadingState = 0;
                     }
-                    else if (result[0] != null) {
-                        var news = result[0];
+                    else if (result != null) {
+                        var news = result;
                         var maxCnt = 0;
 
                         //件名をセット
@@ -145,10 +145,15 @@
                         $.mobile.changePage('#windowcolumn', {transition: "slideup"});
                         //ダイアログが開き終わるまで待機
                         setTimeout( function() {
+                            $.mobile.hidePageLoadingMsg();
                             loadingState = 0;
-                            $.mobile.hidePageLoadingMsg();
                         }, 1000);
                     }
+                    else {
+                        $.mobile.hidePageLoadingMsg();
+                        loadingState = 0;
+                        alert('取得できませんでした。');
+                    }
                 }
             });
         }

変更: branches/version-2_13-dev/data/class/helper/SC_Helper_News.php
===================================================================
--- branches/version-2_13-dev/data/class/helper/SC_Helper_News.php	2013-05-23 09:55:02 UTC (rev 22838)
+++ branches/version-2_13-dev/data/class/helper/SC_Helper_News.php	2013-05-27 14:04:45 UTC (rev 22839)
@@ -37,7 +37,7 @@
      * @param boolean $has_deleted 削除されたニュースも含む場合 true; 初期値 false
      * @return array
      */
-    public function getNews($news_id, $has_deleted = false)
+    static public function getNews($news_id, $has_deleted = false)
     {
         $objQuery =& SC_Query_Ex::getSingletonInstance();
         $col = '*, cast(news_date as date) as cast_news_date';

変更: branches/version-2_13-dev/data/class/pages/frontparts/bloc/LC_Page_FrontParts_Bloc_News.php
===================================================================
--- branches/version-2_13-dev/data/class/pages/frontparts/bloc/LC_Page_FrontParts_Bloc_News.php	2013-05-23 09:55:02 UTC (rev 22838)
+++ branches/version-2_13-dev/data/class/pages/frontparts/bloc/LC_Page_FrontParts_Bloc_News.php	2013-05-27 14:04:45 UTC (rev 22839)
@@ -91,7 +91,7 @@
                 $this->arrErr = $objFormParam->checkError(false);
                 if (empty($this->arrErr)) {
                     $arrData = $objFormParam->getHashArray();
-                    $json = $this->lfGetNewsDetailForJson($arrData, $objNews);
+                    $json = $this->lfGetNewsDetailForJson($arrData);
                     echo $json;
                     SC_Response_Ex::actionExit();
                 } else {
@@ -186,10 +186,10 @@
      * @param object $objNews
      * @return String $json 新着情報1件分のJSONを返す
      */
-    function lfGetNewsDetailForJson($arrData, SC_Helper_News_Ex $objNews)
+    function lfGetNewsDetailForJson($arrData)
     {
 
-        $arrNewsList = $objNews->get($arrData['news_id']);
+        $arrNewsList = SC_Helper_News_Ex::getNews($arrData['news_id']);
         $json =  SC_Utils_Ex::jsonEncode($arrNewsList);    //JSON形式
 
         return $json;