プロジェクト

全般

プロフィール

Bug(バグ) #1504

完了

コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない

Masato Nagasawa さんが14年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-08-12
期日:
進捗率:

100%

予定工数:
3.6 で発生するか:
No (いいえ)
[QA]バグ通知済:
いいえ
3.8 で発生するか:
opCommunityTopicPlugin 絞込条件:

説明

Overview (現象)

コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない。

Causes (原因)

aclを使用して権限チェックを行っているが、インスタンスが存在していないため false となっている。

<?php if ($acl->isAllowed($sf_user->getMemberId(), null, 'addComment')): ?>
<?php
$options = array();
$options['title'] = __('Post a new topic comment');
$options['url'] = url_for('communityTopic_comment_create', $communityTopic);
op_include_form('formCommunityTopicComment', $form, $options);
?>
<?php endif; ?>

opCommunityTopicPluginTopicCommentActions::preExecute() で acl のインスタンスを生成をしていない事が原因。

Way to fix (修正内容)

opCommunityTopicPluginTopicCommentActions::preExecute() で opCommunityTopicAclBuilder::buildResource() し、
メンバ変数 $acl に代入する。


関連するチケット 1 (0件未完了1件完了)

関連している opCommunityTopicPlugin - Backport(バックポート) #2813: トピックで本文が空のコメントを投稿してもエラーメッセージが表示されないInvalid(無効)Yuya Watanabe2009-07-28

操作

Rimpei Ogawa さんが約14年前に更新

  • 3.6 で発生するかYes にセット

Masato Nagasawa さんがほぼ14年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Masato Nagasawa にセット
  • [QA]バグ通知済いいえ にセット

Masato Nagasawa さんがほぼ14年前に更新

  • 対象バージョン1.0.2 から 0.9.9 に変更

0.9.8 のみで発生。 1.0.1 では再現せず。

Masato Nagasawa さんがほぼ14年前に更新

以下のリビジョンで修正しました。
https://github.com/tejimaya/opCommunityTopicPlugin/commit/008ad941931a9fd5096a7bc9327fa496f8c5d174

区分 期待結果 結果
PC:空欄の状態でコメントを書きこむ フォームが表示され「必須項目です。」と表示される
PC:「てすと」と入力してコメントを書き射込む 正常に投稿表示される
携帯:空欄の状態でコメントを書きこむ フォームが表示され「必須項目です。」と表示される
携帯:「てすと」と入力してコメントを書き射込む 正常に投稿表示される

Masato Nagasawa さんがほぼ14年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

Kousuke Ebihara さんが13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Yuya Watanabe さんが約13年前に更新

  • 対象バージョン0.9.9 から 0.9.x に変更

Yuya Watanabe さんが約13年前に更新

stable-0.9.x にコミットされていますが 0.9.9 リリースのため一旦Revert対象とし,Revertを行った後にステータスをPending Fixingに変更することを予定しています. 0.9.9 リリース後, 再度 stable-0.9.x へ適用する必要があります.なお,本チケットにおけるRevert対象のコミットは以下のとおりです.

https://github.com/tejimaya/opCommunityTopicPlugin/commit/008ad941931a9fd5096a7bc9327fa496f8c5d174

参考: #2575 「opCommunityTopicPlugin 0.9.9 のリリースとバンドルをおこなう」

Yuya Watanabe さんが約13年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Fixing(修正待ち) に変更

Yuma Sakata さんが12年以上前に更新

  • 3.6 で発生するかYes から Unknown (未調査) に変更
  • 3.4 で発生するかYes (はい) にセット

再現確認できました。

Environment (再現バージョン)

OpenPNE3.4.20

Way to repro (再現手順)

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

Way to fix (修正内容)

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

Yuma Sakata さんが12年以上前に更新

  • 題名コミュニティトピックにて不正なコメントを投稿した場合のページがおかしい から コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない に変更

Yuma Sakata さんが12年以上前に更新

  • 3.6 で発生するかUnknown (未調査) から No (いいえ) に変更

Yuma Sakata さんが12年以上前に更新

  • 対象バージョン0.9.x から 0.9.10 に変更

Yuya Watanabe さんが12年以上前に更新

  • ステータスPending Fixing(修正待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

Kousuke Ebihara さんが12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Kousuke Ebihara さんが12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

このバグは #2813 の変更を適用することで生じなくなっているため、 https://github.com/tejimaya/opCommunityTopicPlugin/commit/05fe7694f0702a4ae6ddedfe82543c1c649a5bdb の変更は無意味なものとなっています。

変更の内容としては https://github.com/tejimaya/opCommunityTopicPlugin/commit/05fe7694f0702a4ae6ddedfe82543c1c649a5bdb の方が適切だと思いますが、 master では #913 の内容の変更が既に取り込まれてしまっているため、その実績を考慮してこのチケットの変更を取り消し、 #2813 の変更を採用するのがよいのではないかと思います(が、 #2813 を取り消すアプローチでも構わないと思います)。

ひとまずこちらのチケットを差し戻します。

Yuya Watanabe さんが12年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#2813 の変更よりもこちらが適切ということで #2813 で取り込んだ https://github.com/tejimaya/opCommunityTopicPlugin/commit/3925e4826126d7313c90ac3ee43883c267b4b46b の修正を revert する方針とします.

#2813 はこのノートを記述している時点でテスト完了となっていますが, ステータスを変更して invalid としてチケットをクローズします.

Yuya Watanabe さんが12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#2813 のコミット内容を下記コミットで revert を行いました.

https://github.com/tejimaya/opCommunityTopicPlugin/commit/1c850ce9d3bc5de0bea77fe47303d470bccae09f

Kousuke Ebihara さんが12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Yuma Sakata さんが12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

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