Project

General

Profile

Backport(バックポート) #2813

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

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

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

0%

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

Related to opCommunityTopicPlugin - Bug(バグ) #193: トピックで本文が空のコメントを投稿してもエラーメッセージが表示されない Fixed(完了) 2009-07-28
Related to opCommunityTopicPlugin - Bug(バグ) #1504: コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない Fixed(完了) 2010-08-12

History

#1 Updated by Yuma Sakata almost 9 years ago

再現確認できました。

Environment (再現バージョン)

OpenPNE3.4.20

Way to repro (再現手順)

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

Way to fix (修正内容)

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

備考

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

#2 Updated by Yuma Sakata almost 9 years ago

  • Target version changed from 0.9.x to 0.9.10

#3 Updated by Yuya Watanabe almost 9 years ago

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

#4 Updated by Kousuke Ebihara almost 9 years ago

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

#5 Updated by Yuma Sakata almost 9 years ago

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

テストOKです。

#6 Updated by Yuya Watanabe almost 9 years ago

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

#7 Updated by Yuya Watanabe almost 9 years ago

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

Also available in: Atom PDF