ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 11 years ago

Last modified 10 years ago

#1819 closed defect (fixed)

管理機能:日記削除確認画面にコメント件数が表示されない

Reported by: kiwa Owned by: shingo
Priority: minor Milestone: OpenPNE2.12.4
Component: pne-admin Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.10 OpenPNE2.13.3 Cc:

Description (last modified by kudo)

■現象

管理画面の日記一覧(page_diary_list)には日記のコメント数が表示されているが、日記削除確認画面(page_delete_diary)にはコメント数が表示されない。

■発生バージョン

  • 2.8.8
  • 2.10.3
  • 2.11.3

■再現手順

  1. 管理画面の日記一覧(page_diary_list)を表示
  2. 日記コメントが1件以上入っている日記の「削除」ボタンを押下
  3. 日記削除確認画面(page_delete_diary)が表示されるが、コメント数が表示されていない

■環境

■原因

コメント件数の表示部が出力されていない。(ソースに未記入)

■修正内容

コメント件数の表示部を出力するように修正。

■関連情報

  • #1745:管理機能:トピック・イベントのコメント削除確認画面でコメント件数が0になってしまう

Change History (18)

comment:1 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added
Milestone: OpenPNE2.10.5

comment:2 Changed 12 years ago by ebihara

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.5

実害はないので今回のリリース項目からははずします。

comment:3 Changed 12 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.12.x & 2.13.x

comment:4 Changed 11 years ago by shingo

Owner: changed from nobody to shingo
Status: newassigned

修正します。

comment:5 Changed 11 years ago by shingo

Description: modified (diff)
Keywords: OpenPNE2.10.10 OpenPNE2.13.3 確認中 added
Milestone: OpenPNE2.12.4

以下のリビジョンで修正しました。ご確認ください。

comment:6 Changed 11 years ago by shingo

Description: modified (diff)

comment:7 Changed 11 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

db_diary_get_c_diary4id() は非常に多くの箇所で使用されているので安易に処理を追加しない方がよいと思います。(コメント件数を取得する分遅くなるため)

また、テンプレート内の hash() の第2引数の「page」は省略可能です。あっても構わないです。

comment:8 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:9 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

今回の修正によって、存在しない c_diary_id をリクエストパラメータに指定して admin_page_delete_diary にアクセスした場合にエラーとならなくなってしまっています。

comment:10 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:11 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

この修正の else は不要ですし、可読性を下げると思います。

if 文の直後に記述するのが好ましいです。

comment:12 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:13 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:14 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

2.10はコメント一覧(admin_page_diary_comment_ilst)の日記ID検索機能が無いので、2.10に限り日記削除確認画面(admin_page_delete_diary)のコメント部分のリンクを外したほうが良いと思います。

comment:15 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:16 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:17 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題ありません。

comment:18 Changed 10 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.