管理画面の「最新コミュニティ書き込み」がコメントの書き込み日時ではなくトピックの作成日時になっている

[imamura623]

■現象

管理画面の最新コミュニティの書き込みの日時がトピックスが作成された日時になっている。

■原因

■修正内容

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=17432 より転記

2.12.0を使用しています。
管理画面の最新コミュニティの書き込みと言う欄は
コミュニティにトピックスが作成された日なんですね？
書き込みときくとトピックス内のコメントが書かれた日かと思ってしまうので・・・

[nakasone]

以下のような確認を行ったところ再現できました。

■確認ブラウザ

• FireFox2

■確認バージョン

• 2.10.x
• 2.12.x
• trunk

■手順

1. 管理画面→最新コミュニティ書き込みを確認
2. 日時がトピック作成日時になっている
3. トピックに書き込みをする→手順１
4. 日時がコメントの書き込み日時にならないのを確認

[asma]

コメントがある場合はコメントの最新書き込み日時を見るように修正する。

[asma]

• 2.10.x r7715
• 2.12.x r7712
• trunk r7711

対応しました。

[ebihara]

以下の内容を確認してください。

1. r7711 および r7712 の変更により、 monitor_topic_list() をコールしている admin_page_topic_list において問題が発生してしまっています（トピック名が表示できなくなっているなど）
2. 以下のカンマは行末にある方がわかりやすいと思うのですがどうでしょう（OpenPNE のコードに含まれるSQL文の大半は、カンマの後に改行し、文頭に半角スペースを入れています）

       $select = 'SELECT c.name AS commu_name, c.image_filename AS commu_image'
               . ' ,ct.*, ct.u_datetime AS max_datetime';
   

3. r7715 の 2.10.x への対応に対してのみ monitor_new_topic_list() という新しい関数を作成した理由はなんでしょうか。 t_assign_sns_new_topic に特化したものにしては、 $keywords という引数を設けているなど不要な記述が多すぎる気がしますし、このような対処をするのであれば 2.12.x や trunk に対しても同様におこなうべきです。
4. r7715 の、 commu_imai というエイリアスの意味が分かりませんでした。

   $select = 'SELECT c.name AS commu_name, c.image_filename AS commu_imai'
   

5. 2.10.x の webapp/modules/admin/lib/db_admin.php の 2376 行目には半角スペースしかありません。空行にするのが望ましいです。
6. 2371 行目の $params の定義は不要ではないでしょうか。

admin_page_topic_list に関しては、わざわざトピックコメントの書き込み時間を見てソートする理由がないので、従来通り r_datetime でソートするのがよいでしょう。

ですので、 2.12.x や trunk についても、 2.10.x の対応と同じように専用の関数を定義するという方法を採るのが望ましい気がします。

[asma]

• 2.10.x r7731
• 2.12.x r7730
• trunk r7729

対応しました。

[ebihara]

以下、確認よろしくお願いします。

1. monitor_new_topic_list() 内で "ct.r_datetime AS max_datetime" でソートしようとしていますが、これではこのチケットの要件を満たさなくなってしまうと思います。
2. monitor_new_topic_list() に無駄な $where の定義があります。
3. monitor_new_topic_list() に comment:7 で指摘した無駄な $params の定義があります。
4. 2.12.x, trunk の monitor_new_topic_list() で、おそらく不要と思われる以下のコードがありますが、$wheres は未定義であるため Notice エラーが発生してしまいます。

   if ($wheres) {
   

5. monitor_new_topic_list() において $list[$key][ 'original_filename' ] の代入などは必要でしょうか。
6. monitor_new_topic_list() では c.image_filename の取得は不要ではないでしょうか。
7. monitor_new_topic_list() の以下のコードですが、カンマ後のスペースなどが PEAR 標準コーディング規約に沿っていません。

   db_get_all_limit($sql,($page-1)*$page_size,$page_size,$params);
   

8. ところで monitor_new_topic_list() ってページャを考慮する必要ってあるんでしょうか。現状ページャは必要でないはずなので、コールする側で無理に array_shift() して最初の要素のみ取得して使っているのはどうかなという気がしています。

[ebihara]

修正を引き受けます。

[ebihara]

以下のリビジョンで、コメントの書き込み時刻ではなくトピックの作成日時を見るようになってしまっていたのを修正しました。

• 2.12.x - r7750
• trunk - r7751

以下のリビジョンで、冗長な記述などの削除などをおこないました。

• 2.10.x - r7754
• 2.12.x - r7752
• trunk - r7753

ご確認ください。

[ogawa]

2.10.x だけmonitor_new_topic_listの第2引数が残っているようです。

function monitor_new_topic_list($limit, $page = 1)

[ebihara]

以下で削除しました。

• 2.10.x - r7755

[ebihara]

以下のリビジョンで半角スペースを削除しました。r7756

[kiwa]

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