Bug（バグ） #1504
完了
コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない
100%
説明
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 に代入する。
Masato Nagasawa さんがほぼ14年前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Masato Nagasawa にセット
- ［QA］バグ通知済 を いいえ にセット
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年前に更新
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 さんがほぼ13年前に更新
- 3.6 で発生するか を Yes から Unknown (未調査) に変更
- 3.4 で発生するか を Yes (はい) にセット
Yuma Sakata さんがほぼ13年前に更新
- 題名 を コミュニティトピックにて不正なコメントを投稿した場合のページがおかしい から コメント投稿時にフォーム内容がエラーであった場合に、フォームが再表示されない に変更
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です。