Merge pull request #238 from uzulla/issue140/improve-js-error-handling-in-admin-page

fc2dev · web-flow · commit 9d76db45be79 · 2021-02-28T15:13:42.000+08:00
管理画面操作時のAjax通信エラーについて、エラーダイアログ等をできるだけ出すように修正
diff --git a/app/src/Web/Controller/Admin/CategoriesController.php b/app/src/Web/Controller/Admin/CategoriesController.php
@@ -138,7 +138,7 @@ public function delete(Request $request)
   }
 
   /**
-   * ajax用のカテゴリ追加
+   * ajax用のカテゴリ追加 admin/entries/create からインクルード
    * @param Request $request
    * @return string
    */
diff --git a/app/src/Web/Controller/Admin/FilesController.php b/app/src/Web/Controller/Admin/FilesController.php
@@ -13,7 +13,7 @@ class FilesController extends AdminController
 {
 
   /**
-   * 一覧表示
+   * 一覧表示 /admin/files/upload からPartial読み込み
    * @param Request $request
    * @return string
    */
diff --git a/app/twig_templates/admin/blog_plugins/index.twig b/app/twig_templates/admin/blog_plugins/index.twig
@@ -77,7 +77,11 @@
                         display: $(this).prop('checked') ? 1 : 0,
                         sig: "{{ sig }}"
                     }),
-                    cache: false
+                    cache: false,
+                    error: function (data, status, xhr){
+                        alert("エラーが発生しました、ページをリロードしてやり直してください。\n" +
+                          "An error occurred, please reload page and try again.");
+                    }
                 });
             });
 
diff --git a/app/twig_templates/admin/categories/ajax_add.twig b/app/twig_templates/admin/categories/ajax_add.twig
@@ -1,3 +1,4 @@
+{# include from admin/entries/create #}
 <table>
     <tbody>
     <tr>
@@ -152,6 +153,13 @@
                     $('#sys-category-add-name').val('');
                     $('#sys-category-add').removeAttr('disabled');
                     $('#sys-category-add').val('{{ _('Add') }}');
+                },
+                error: function (data, status, xhr){
+                    alert("カテゴリ追加時にエラーが発生しました、時間をおいてから再度試行してください。\n" +
+                      "An error occurred when adding the category, please wait a while and try again.");
+                    $('#sys-category-add-name').val('');
+                    $('#sys-category-add').removeAttr('disabled');
+                    $('#sys-category-add').val('{{ _('Add') }}');
                 }
             });
             return false;
diff --git a/app/twig_templates/admin/categories/ajax_add_sp.twig b/app/twig_templates/admin/categories/ajax_add_sp.twig
@@ -1,3 +1,4 @@
+{# include from admin/entries/create #}
 <h3><span class="h3_inner">{{ _('Categories') }}</span></h3>
 
 <div class="checkbox_list">
@@ -137,6 +138,13 @@
                         $('#sys-category-add-name').val('');
                         $('#sys-category-add').removeAttr('disabled');
                         $('#sys-category-add').val('{{ _('Add') }}');
+                    },
+                    error: function (data, status, xhr){
+                        alert("カテゴリ追加時にエラーが発生しました、時間をおいてから再度試行してください。\n" +
+                         "An error occurred when adding the category, please wait a while and try again.");
+                        $('#sys-category-add-name').val('');
+                        $('#sys-category-add').removeAttr('disabled');
+                        $('#sys-category-add').val('{{ _('Add') }}');
                     }
                 });
                 return false;
diff --git a/app/twig_templates/admin/comments/index.twig b/app/twig_templates/admin/comments/index.twig
@@ -143,7 +143,12 @@
             };
             $('#sys-comment-reply-dialog').dialog(option);
             $('#sys-comment-reply-dialog').html('now loading..');
-            $('#sys-comment-reply-dialog').load(common.fwURL('Comments', 'ajax_reply', {id: id}))
+            $('#sys-comment-reply-dialog').load(common.fwURL('Comments', 'ajax_reply', {id: id}), function(data, status, xhr){
+              if(status === "error") {
+                alert("エラーが発生しました、ページをリロードしてください。\n" +
+                  "An error occurred, please reload page and try again.");
+              }
+            });
             reply.updateReadStatus(is_private);
         };
         // コメントを閉じる処理
@@ -170,6 +175,10 @@
                     $('#sys-reply-message').hide();
                     $('#sys-reply-error').html(json.error);
                     $('#sys-reply-error').show();
+                },
+                error: function (data, status, xhr){
+                    alert("エラーが発生しました、ページをリロードしてください。\n" +
+                    "An error occurred, please reload page and try again.");
                 }
             });
         };
@@ -193,6 +202,10 @@
                         $('#sys-open-status-' + id + ' > *').show();
                         $('#sys-comment-approval').show();
                         alert(json.error);
+                    },
+                    error: function (data, status, xhr){
+                        alert("エラーが発生しました、ページをリロードしてやり直してください。\n" +
+                          "An error occurred, please reload page and try again.");
                     }
                 });
             });
diff --git a/app/twig_templates/admin/entries/ajax_media_load.twig b/app/twig_templates/admin/entries/ajax_media_load.twig
@@ -1,3 +1,4 @@
+{# 新規記事追加ページ /admin/entries/create から読み込み #}
 {% if files %}
 
     <ul class="sys-form-add-media">
diff --git a/app/twig_templates/admin/entries/ajax_media_load_sp.twig b/app/twig_templates/admin/entries/ajax_media_load_sp.twig
@@ -1,3 +1,4 @@
+{# 新規記事追加ページ /admin/entries/create SP から読み込み #}
 {% if files %}
 
     <div class="form_contents">
diff --git a/app/twig_templates/admin/files/ajax_index.twig b/app/twig_templates/admin/files/ajax_index.twig
@@ -1,3 +1,4 @@
+{# /admin/files/upload からPartial読み込み #}
 <h3 id="entry_count">
     {{ _('File search') }}[{{ _('Hits') }}&nbsp;{{ paging.count }}{{ _(' results') }}]
     {{ input(req, 'limit', 'select', {'options': page_list_file, 'default': file_default_limit}) }}
@@ -79,7 +80,7 @@
             return false;
         });
 
-        // ファイル削除ボタン
+        // 複数ファイル削除ボタン
         $('#sys-delete-button').click(function () {
             if (!confirm('{{ _('Are you sure you want to delete?') }}')) {
                 return;
@@ -107,6 +108,15 @@
                     // 削除完了後検索処理を実行
                     isAjaxSubmit = isPageChange = true;
                     ajaxSubmit();
+                },
+                error: function( response, status, xhr ) {
+                  if (status === "error") {
+                    alert("エラーが発生しました、ページをリロードしてください。\n" +
+                      "Request failed. Please reload page and retry.");
+                    // 失敗しているが、画面を復帰させるために検索処理を実行
+                    isAjaxSubmit = isPageChange = true;
+                    ajaxSubmit();
+                  }
                 }
             });
             return false;
diff --git a/app/twig_templates/admin/files/upload.twig b/app/twig_templates/admin/files/upload.twig
@@ -42,7 +42,7 @@
             common.formEnterNonSubmit('sys-file-form');
 
             // ajaxで一覧情報をロード
-            $('#sys-ajax-files-index').load('{{ url(req, 'Files', 'ajax_index') }}')
+            $('#sys-ajax-files-index').load('{{ url(req, 'Files', 'ajax_index') }}', writeErrToSysAjaxFilesIndex)
         });
 
         // ページ数初期化有無フラグ
@@ -64,7 +64,8 @@
                     success: function (res) {
                         $('#sys-ajax-files-index').html(res);
                         isAjaxSubmit = true;
-                    }
+                    },
+                    error: writeErrToSysAjaxFilesIndex
                 });
                 return;
             }
@@ -85,7 +86,8 @@
                 success: function (res) {
                     $('#sys-ajax-files-index').html(res);
                     isAjaxSubmit = true;
-                }
+                },
+                error: writeErrToSysAjaxFilesIndex
             });
         }
 
@@ -102,5 +104,13 @@
             $('input[name=order]').val(order);
             ajaxSubmit();
         }
+
+        function writeErrToSysAjaxFilesIndex( response, status, xhr ) {
+          if ( status === "error" ) {
+            $('#sys-ajax-files-index')
+              .html("<div style='text-align: center; font-size: 2em; font-weight:bold'>エラーが発生しました、ページをリロードしてください。<br>" +
+                "Loading error. Please reload page.</div>");
+          }
+        }
     </script>
 {% endblock %}
diff --git a/public/js/admin/entry_editor.js b/public/js/admin/entry_editor.js
@@ -1,15 +1,15 @@
 // メディア追加関数
 var addMedia = {
   target_id: null,
-  elrte: true,    // elRTEの使用,不使用
+  elrte: true, // elRTEの使用,不使用
   init: function(){
     if (this.target_id!=null) {
       return ;
     }
     // 検索ボタン
     $('#sys-add-media-search-button').on('click', function(){
       var keyword = $('#sys-add-media-search-keyword').val();
-      $('#sys-add-media-load').load(addMedia.load({keyword: keyword}));
+      addMedia.load({keyword: keyword});
     });
     // Enterキーで検索
     $('#sys-add-media-search-keyword').keypress(function(e){
@@ -22,18 +22,27 @@ var addMedia = {
       $('#sys-add-media-dialog').dialog('option', {width: $(window).width() - 100});
     });
   },
-  load: function(params){
-    $('#sys-add-media-load').fadeOut('fast', function(){
-      $('#sys-add-media-load').load(common.fwURL('Entries', 'ajax_media_load', params), function(){
-        $('#sys-add-media-load').fadeIn('fast');
-        $('#sys-add-media-load').find('input[type=checkbox]').on('click', function(){
-          if ($(this).prop('checked')) {
-            $(this).closest('li').addClass('selected');
+  load: function (params) {
+    $('#sys-add-media-load').fadeOut('fast', function () {
+      $('#sys-add-media-load').load(
+        common.fwURL('Entries', 'ajax_media_load', params),
+        function (response, status, xhr) {
+          if (status === "error") {
+            alert("エラーが発生しました、ページをリロードしてください。\n" +
+              "Loading error. Please reload page.");
+
           } else {
-            $(this).closest('li').removeClass('selected');
+            $('#sys-add-media-load').fadeIn('fast');
+            $('#sys-add-media-load').find('input[type=checkbox]').on('click', function () {
+              if ($(this).prop('checked')) {
+                $(this).closest('li').addClass('selected');
+              } else {
+                $(this).closest('li').removeClass('selected');
+              }
+            });
           }
-        })
-      });
+        }
+      );
     });
   },
   open: function(key, config){
@@ -62,7 +71,7 @@ var addMedia = {
     }
     $('#sys-add-media-dialog').dialog(option);
     $('#sys-add-media-load').html('<p>Now loading...</p>');
-    $('#sys-add-media-load').load(addMedia.load({}));
+    addMedia.load({});
   },
   add: function(){
     var textarea_id = addMedia.target_id;
@@ -91,7 +100,7 @@ var addMedia = {
   // 位置情報取得
   getSelection: function(dom) {
     var pos = {};
-    if (/*@cc_on!@*/false) {
+    if (/*@cc_on!@*/false) { // TODO remove, It's hack for IE<=10 #235
       dom.focus();
       var range = document.selection.createRange();
       var clone = range.duplicate();
diff --git a/tests/cli_generate_sample_comment.php b/tests/cli_generate_sample_comment.php
@@ -6,6 +6,7 @@
 use Fc2blog\Tests\LoaderHelper;
 
 define("TEST_APP_DIR", __DIR__ . "/../app");
+define("THIS_IS_TEST", true);
 
 require_once __DIR__ . "/../app/vendor/autoload.php";
 
diff --git a/tests/cli_generate_sample_upload_image.php b/tests/cli_generate_sample_upload_image.php
@@ -6,6 +6,7 @@
 use Fc2blog\Tests\LoaderHelper;
 
 define("TEST_APP_DIR", __DIR__ . "/../app");
+define("THIS_IS_TEST", true);
 
 require_once __DIR__ . "/../app/vendor/autoload.php";
 

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ public function delete(Request $request)`
`138`	`138`	`}`
`139`	`139`
`140`	`140`	`/**`
`141`		`- * ajax用のカテゴリ追加`
	`141`	`+ * ajax用のカテゴリ追加 admin/entries/create からインクルード`
`142`	`142`	`* @param Request $request`
`143`	`143`	`* @return string`
`144`	`144`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ class FilesController extends AdminController`
`13`	`13`	`{`
`14`	`14`
`15`	`15`	`/**`
`16`		`- * 一覧表示`
	`16`	`+ * 一覧表示 /admin/files/upload からPartial読み込み`
`17`	`17`	`* @param Request $request`
`18`	`18`	`* @return string`
`19`	`19`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+{# 新規記事追加ページ /admin/entries/create から読み込み #}`
`1`	`2`	`{% if files %}`
`2`	`3`
`3`	`4`	`<ul class="sys-form-add-media">`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+{# 新規記事追加ページ /admin/entries/create SP から読み込み #}`
`1`	`2`	`{% if files %}`
`2`	`3`
`3`	`4`	`<div class="form_contents">`