TODO #13935

ブログのコメントが送信できない

安部 友啓約2年前に追加. 約2年前に更新.

ステータス:解決開始日:
優先度:通常期日:
担当者:滝下 真玄進捗率:

90%

カテゴリ:バグ
対象バージョン:正式版リリース準備

説明

ブログのコメント機能で、一度エラーを表示させた(URLにhogehogeと入力してエラーを出力させる)後は
正しい入力を行なってもコメントが送信できずにエラーが表示される。


関係しているリビジョン

リビジョン ac88eba5
Masaharu Takishita が約2年前に追加

fix #13935 ブログのコメントが送信できない (#514)

  • ブログコメントのJS部分のリファクタリング。

何らかの入力エラーがあると、それ以降CSRFエラーで成功しなくなる。このためadminで使われている$.bcTokenを利用。
また、スクリプトがベーステーマ内でインライン展開されているため、各テーマでスクリプトがコピーされている問題がある。
とりあえずスクリプト部分をエレメント化し、さらに純粋なJS部分はJSファイル化。
このコミットでは各テーマの修正は行わない。後のコミットで。

  • ビルトインテーマに新形式のコメント投稿スクリプトを対応
  • fix #13935 ブログコメント投稿で、バリデーション等に失敗した後に投稿できない不具合の修正。管理画面以外でのbcToken対応
  • bcToken.jsのトークン生成時のクエリストリングはURL指定に含めるように仕様変更

履歴

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

  • 対象バージョン正式版リリース準備 にセット

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

  • カテゴリバグ にセット

#3 滝下 真玄約2年前に更新

  • 担当者滝下 真玄 にセット

現象を確認。
ページ内にエラーが出るのとダイアログでエラーがでるのと2種類あるけど、2つあるってなんかきもいですね(^^;

#4 滝下 真玄約2年前に更新

JSでcapthaIdを変更しているのにBlogComment.captcha_idのhiddenに保存していないのが第一の原因。
しかしエラー時の内容そのままのcaptchaでもエラーになるので、別の要因もあるかもしれない。
あと、これ各テーマにハードコピーされているからヤバイですね。elementなり素のJSファイルなりにした方が良いと思います。

【追記】
CSRFエラーでした。しかしこのケースの直し方がちょっと分からない。

#5 滝下 真玄約2年前に更新

  • ステータス担当 から 解決 に変更
  • 進捗率0 から 90 に変更

修正してPRしました

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