課題 #13834

eventのエラーハンドリング

- 清末直約2年前に追加. 3ヶ月前に更新.

ステータス:担当開始日:
優先度:急いで期日:
担当者:-進捗率:

0%

カテゴリ:イベント
対象バージョン:-

説明

主に、ajaxまわり (特に contents)の部分に係わってくると思うのですが
eventで起きたエラーを、大元で取得できなく、ユーザに適切なエラーメッセージを
出力できません。

たとえば、固定ページの追加を ajax経由で行うと。
/admin/pages/ajax_add
がコールされるのですが、このmethodの中で、page table に insert される前に
contents table へ insert されたりします。これがeventで登録されているので、
Content.php modelで起きたエラーを Page.php で取得するすべが現状ないと思います。

PagesController.php のロジックには
$this->ajaxError(500, $this->Page->validationErrors);
とか書かれていて、validationErrorを返そうとしてるのですが、event Contentの方で
起きたエラーはこちらにデータが入ってこないので、default の 「Internal Server Error」の
表示になります。

このような状況の場合、errorを、throwして、大元で、catchするのがいいかと思うのですが
(それ以外思い浮かばない)、恐らく全体的に改修を入れると多岐にわたりそうなのと
根本的な設計の問題なので、先に相談でチケット書いてます。

取りあえずは、解消の方法だけ決めておけば今後これに類似するエラーは同様の方向性で
解消できるとおもうので、方向性だけ決めたいのですがどれがいいでしょうか?
(どれがいいと言いつつ、1つしか提案してないですが、、)

履歴

#1 - 清末直約2年前に更新

  • 関連している TODO #13827: コンテンツ一覧からのajax経由でコンテンツ名の制御が必要 を追加

#2 江頭 竜二約2年前に更新

  • 担当者江頭 竜二 から - 清末直 に変更

別チケットにも書いたのですが、
BcContentsBehavior::beforeValidate() でバリデーションエラーの引き継ぎ処理をやる方向では難しいでしょうか?

#3 - 清末直約2年前に更新

おぉ、やってみます。

#4 江頭 竜二約2年前に更新

追記です。

今回、コア側をプラグインでラップするような仕様となってますので、コアで発生したものをプラグインでキャッチする仕様にするとプラグイン側の開発負担が増えるんですよね。

方針としては、プラグインの開発負担を減らす為に、できるだけコアで担う方向でいきたいと考えています。

#5 - 清末直約2年前に更新

  • トラッカーTODO から 課題 に変更
  • プロジェクトbaserCMSベータテスト から baserCMS に変更
  • カテゴリバグ から イベント に変更
  • 対象バージョン正式版リリース準備 から 4.0.1 に変更

一端、チケットを未来にむけなおしておきます。
課題として。

#6 - 清末直約2年前に更新

  • 優先度今すぐ から 通常 に変更

#7 江頭 竜二約2年前に更新

追記です。

プラグインだけでなく、固定ページや、コンテンツフォルダ、コンテンツリンク等、コンテンツ管理で管理されるコアのコンテンツも仕様を合わせないといけないです。

#8 江頭 竜二約2年前に更新

  • 対象バージョン4.0.1 から 4.0.0 に変更

すんません、こちら、一旦戻させてください。

#9 江頭 竜二約2年前に更新

  • 担当者- 清末直 から 江頭 竜二 に変更

#10 江頭 竜二約2年前に更新

  • 対象バージョン4.0.0 から 4.0.1 に変更

こちらやっぱり未来向けの課題という事で。

#11 江頭 竜二約2年前に更新

  • 担当者 を削除 (江頭 竜二)

#12 江頭 竜二ほぼ2年前に更新

  • 対象バージョン4.0.1 から 4.0.2 に変更

#13 江頭 竜二1年以上前に更新

  • 対象バージョン4.0.2 から 4.0.3 に変更

#14 江頭 竜二1年以上前に更新

  • 対象バージョン4.0.3 から 4.0.4 に変更

#15 江頭 竜二1年以上前に更新

  • 対象バージョン4.0.4 から 4.0.5 に変更

#16 江頭 竜二1年以上前に更新

  • 対象バージョン4.0.5 から 4.0.6 に変更

取り急ぎ、見送ります。

#17 江頭 竜二約1年前に更新

  • 対象バージョン4.0.6 から 188 に変更

#18 江頭 竜二約1年前に更新

  • 対象バージョン188 から 4.1.0 に変更

#19 江頭 竜二3ヶ月前に更新

  • 優先度通常 から 急いで に変更

#20 江頭 竜二3ヶ月前に更新

  • 対象バージョン を削除 (4.1.0)

他の形式にエクスポート: Atom PDF