Project

General

Profile

Actions

Backport(バックポート) #2813

closed

トピックで本文が空のコメントを投稿してもエラーメッセージが表示されない

Added by Yuya Watanabe almost 13 years ago. Updated almost 13 years ago.

Status:
Invalid(無効)
Priority:
Low(低め)
Assignee:
Target version:
Start date:
2009-07-28
Due date:
% Done:

0%

Estimated time:
opCommunityTopicPlugin 絞込条件:

Description

以前のチケットは http://trac.openpne.jp/ticket/4181 です

=== ■ バグ概要 ===
トピック詳細画面からコメントを投稿する際、本文に何も入力せずに「送信」を押してもエラーなどが表示されず、コメントフォームなしのトピック詳細画面が表示される。

=== ■ 原因 ===
フォームの内容が不正だった際の処理が何もないため。

{{{
// lib/action/opCommunityTopicPluginTopicCommentActions.class.php

56    if ($this->form->isValid())
57    {
58      $this->form->save();
59      $this->redirect('@communityTopic_show?id='.$this->communityTopic->getId());
60    }
61
62    $this->setTemplate('../../communityTopic/templates/show');
63
64    return sfView::SUCCESS;
}}}

$this->form->isValid()がfalseだった場合そのまま関数の末尾まで処理され、sfView::SUCCESSによりトピック詳細が表示されるようになっています。

=== ■ 修正内容 ===

=== ■ 関連項目 ===


Related issues 2 (0 open2 closed)

Related to opCommunityTopicPlugin - Bug(バグ) #193: トピックで本文が空のコメントを投稿してもエラーメッセージが表示されないFixed(完了)洋一郎 櫻井2009-07-28

Actions
Related to opCommunityTopicPlugin - Bug(バグ) #1504: コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されないFixed(完了)Masato Nagasawa2010-08-12

Actions
Actions #1

Updated by Yuma Sakata almost 13 years ago

再現確認できました。

Environment (再現バージョン)

OpenPNE3.4.20

Way to repro (再現手順)

0. コミュニティ作成しておく
1. 手順0 で作成したコミュニティにトピック作成する
2. 手順1 で作成したコミュニティトピックに未入力でコメント投稿する
3. 「必須項目です。」などのエラーメッセージが表示されず、フォームが再表示されない

Way to fix (修正内容)

コメント投稿時にフォーム内容がエラーであった場合に、「必須項目です。」などのエラーメッセージが表示させて、フォームが再表示されるように修正お願いします。

備考

#1504 と重複ではないでしょうか?

Actions #2

Updated by Yuma Sakata almost 13 years ago

  • Target version changed from 0.9.x to 0.9.10
Actions #3

Updated by Yuya Watanabe almost 13 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to Yuya Watanabe
  • % Done changed from 0 to 50
Actions #4

Updated by Kousuke Ebihara almost 13 years ago

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

Updated by Yuma Sakata almost 13 years ago

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

テストOKです。

Actions #6

Updated by Yuya Watanabe almost 13 years ago

レビュー,テストともに完了していますが, note-1 の記述通り #1504 の内容と重複していて, #1504 のほうが修正として適切であるということで,本チケットでの修正を revert し,本チケットを invalid としてクローズします.

Actions #7

Updated by Yuya Watanabe almost 13 years ago

  • Status changed from Fixed(完了) to Invalid(無効)
  • % Done changed from 100 to 0
Actions

Also available in: Atom PDF