カメラアイコンの表示条件が統一されていない

[cake]

■現象

c_homeとh_com_comment_list(マイホームのトピック一覧）で、カメラアイコンの表示が異なる場合があります。

画像有無を判断しているコメントが違うようです。

■原因

各ページ毎の呼び出す関数によって *最新トピックに画像が添付されているか？ or *親トピックに画像が添付されているか？ のように判定方法が異なっていたため

■修正内容

comment:4

にあるように/webapp/lib/db/commu.php内の関数を統一条件が親トピックに画像が添付されているか否かに修正

この修正でカメラアイコンの表示が親トピックに画像が添付されている場合に統一されました。

■関連情報

#1482:カメラアイコン表示の基準が統一されていない

comment:4

[kiwa]

#1482 より

---

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

カメラアイコンの表示が異なります。

SQLを見ると、

c_home→(基本）親トピックに画像がついている場合
h_com_comment_list→最新のコメントに画像がついている場合
　のようです。

　どちらかに統一した方が良いように思います。

＊　c_homeで「基本」とした理由は、DBの状態によっては、コメントの画像を取得する場合があるからです。
$sql = "SELECT cct.c_commu_topic_id , cct.name, MAX(cctc.r_datetime) as r_datetime , cct.c_commu_id " .
" , cctc.image_filename1, cctc.image_filename2, cctc.image_filename3 " .
" FROM c_commu_topic_comment as cctc , c_commu_topic as cct" .
" WHERE cctc.c_commu_topic_id = cct.c_commu_topic_id " .
" AND cct.event_flag = ?".
" AND cct.c_commu_id = ?".
" group by cct.c_commu_topic_id " .
" order by r_datetime desc ";

GROUP BY　したテーブルから集計結果以外のデータを取得する場合、ORDERの指定なしで出力される最初のデータの値が入ってきます。
　それは、プライマリIDの小さい順とは限りません。
　　以前、私自身が別件で嵌った罠です・・・・

記事に画像が添付されています。

[ebihara]

修正方法を明確にしていただいているので、是非対応したいです。

プライオリティをあげます。

[kiwa]

日記・トピックなどカメラアイコンが表示される全ての場所（PC・携帯ともに）で表示条件を統一させましょう

統一条件：親記事に画像がついている場合

[kiwa]

2.10.5では見送ります

[ogawa]

安定版マイナーバージョンアップリリースに回します。

[akamine]

対応します

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9219
• 2.12.x - r9217
• trunk - r9218

[shingo]

【全般】

• イベントの修正がなされていない
  

【2.12・2.13】
以下の関数について

• SQL文について･･･カラム「number」を利用すれば確実にトピックの書き込みを特定できます
• コメントが適切ではない（わかりづらい）

  webapp/lib/db/commu.php
  
  3364 /*
  3365  * 最初の書き込みコメントの情報取得(親トピック)
  3366  */
  3367 function db_commu_get_start_c_topic_comment($c_commu_topic_id)
  3368 {
  3369     $sql = 'SELECT * FROM c_commu_topic_comment WHERE c_commu_topic_id = ? ORDER BY r_datetime ASC';
  3370     $params = array(intval($c_commu_topic_id));
  3371     return db_get_row($sql,$params);
  3372 }
  

【2.10】

• WHERE句のところで改行したほうがみやすい

  webapp/lib/db/commu.php
  
  1191     } else {
  1192         $sql = 'SELECT cct.c_commu_topic_id, cct.c_commu_id, MAX(cctc.r_datetime) as r_datetime, cct.c_member_id' .
  1193                 ' , cctc.image_filename1, cctc.image_filename2, cctc.image_filename3 '.
  1194                 ' FROM c_commu_topic as cct, c_commu_topic_comment as cctc'. $hint . ' WHERE cct.c_commu_id IN (' . $ids . ') AND cctc.c_commu_topic_id = cct.c_commu_topic_id'.
  1195                 ' GROUP BY cctc.c_commu_topic_id'.
  1196                 ' ORDER BY r_datetime DESC';
  1197     }
  

• 以下のSQL文(上)を3行に分けて記述する意味を感じない（WHERE句とそれ以外で分ければどうでしょう？）
• 各SQL文の記述で「.(連結修飾子)」は各行の'　'の前にあった方がいいようです。(絶対ではありません)

  webapp/lib/db/commu.php
  
  1204         $sql = 'SELECT number' .
  1205                ' FROM c_commu_topic_comment' .
  1206                ' WHERE c_commu_topic_id = ? AND r_datetime = ?';
  

webapp/lib/db/commu.php

1192	        $sql = 'SELECT cct.c_commu_topic_id, cct.c_commu_id, MAX(cctc.r_datetime) as r_datetime, cct.c_member_id' . 
1193	                ' , cctc.image_filename1, cctc.image_filename2, cctc.image_filename3 '. 
1194	                ' FROM c_commu_topic as cct, c_commu_topic_comment as cctc'. $hint . ' WHERE cct.c_commu_id IN (' . $ids . ') AND cctc.c_commu_topic_id = cct.c_commu_topic_id'. 
1195	                ' GROUP BY cctc.c_commu_topic_id'. 
1196	                ' ORDER BY r_datetime DESC'; 

[shingo]

• 現状、トピック親記事に画像アップロード後、削除してもアイコンは残ってしまいます。
• 以下の部分のカメラアイコン表示の処理が無効なようです。（一応確認してください）チケットの内容とは異なりますが関連事項として処理してください。

  webapp/modules/pc/templates/h_album_list_friend.tpl:21行目
  
  16-({$smarty.capture.pager|smarty:nodefaults})
  17-
  18-({foreach from=$h_album_list_friend item=item})
  19-<dl>
  20-<dt>({$item.r_datetime|date_format:"%Y年%m月%d日 %H:%M"})</dt>
  21:<dd><a href="({t_url m=pc a=page_fh_album})&amp;target_c_album_id=({$item.c_album_id})">({$item.subject})</a> (({$item.c_member.nickname}))({if $item.image_filename_1 || $item.image_filename_2 || $item.image_filename_3})<img src="({t_img_url_skin filename=icon_camera})" alt="" />({/if})</dd>
  22-</dl>
  23-({/foreach})
  24-
  25-({$smarty.capture.pager|smarty:nodefaults})
  26-</div></div>
  

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9247
• 2.12.x - r9252,r9246
• trunk - r9253,r9248

• comment:27のカメラアイコンの処理が無効なようです、に関しても$h_album_list_friendに$item.image_filename・・・の値自体を取得してなかったようでしたので削除しました。

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9267

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9273

[shingo]

以下のリビジョンを再度修正しました。

• 2.10.x - r9276

[kiwa]

親記事に画像が1件以上存在してもカメラアイコンが表示されないページがあります。

pc

• マイホーム(pc_page_h_home)
  • 日記コメント記入履歴
  • 全体のコミュニティ最新書き込み(2.10にはありません)
• 日記コメント記入履歴(ktai_page_h_diary_comment_list)
• トピック検索(全体)(pc_page_h_com_topic_find_all)
• トピック検索(コミュニティ内)(pc_page_c_com_topic_find)

ktai

• マイホーム(ktai_page_h_home)
  • 日記コメント記入履歴
  • 全体のコミュニティ最新書き込み(2.10にはありません)
• 日記コメント記入履歴(ktai_page_h_diary_comment_list)
• トピック検索(全体)(ktai_page_h_com_topic_find_all)
• トピック検索(コミュニティ内)(ktai_page_c_com_topic_find)

[ebihara]

追加で指摘させてください。

• db_commu_get_last_c_topic_comment() を消しているようですが、これは残しておいてください。

[akamine]

以下のリビジョンを再度修正しました。

• 2.12.x - r9329,r9328,r9327,r9326,r9325
• trunk - r9334,r9332

上記に関してはカメラアイコンの表示を統一しております。
基本的に親記事の画像有無で表示させ、例外はブログ、レビュー、アルバムとh_com_find_allの検索結果となります。
上記は指摘された修正をカバーしているため今後修正の必要はありません。

• 2.10.xは現在全く修正しておりません。

[ebihara]

2.10.x の修正については海老原が引き受けます。

[ebihara]

2.10.x に関しては 2.12.x や trunk と違い複雑なSQL文になりうるため、対応をおこないません。そのため、r9335 でこのチケットに関する2.10.xへの変更を取り消しました。

ご確認ください。

※2.12.x と trunk の変更についてはすでに海老原確認済みです。

[kiwa]

コードチェック前ですが差し戻させてもらいます。以下の現象の確認をお願いします。

確認バージョン

• 2.12.x(r9338 時点)
• trunk(r9338 時点)

内容

pc

• マイホーム(pc_page_h_home)
  • 日記コメント記入履歴：カメラの表示条件に問題わありませんが、ニックネーム表示部分の閉じ括弧が消えています
• 日記コメント記入履歴(pc_page_h_diary_comment_list)
  • 条件に一致する日記にカメラアイコンが表示されません
• コミュニティ内トピック検索(pc_page_c_topic_find_all)
  • 条件に一致するトピックにカメラアイコンが表示されません

ktai

• 日記コメント記入履歴(ktai_page_h_diary_comment_list)
  • 条件に一致する日記にカメラアイコンが表示されません

---

Descriptionの修正内容を更新してください。

[akamine]

以下の修正でカメラアイコンが条件に合った場合でも表示されないページを修正しました。

• 2.12.x - r9345,r9343
• trunk - r9346,r9344

[ebihara]

h_homeですが、BIZモードに対しても修正の必要があるのではないでしょうか

[akamine]

以下の修正でbizモードでもh_homeでニックネーム表示部分の閉じ括弧が消えていたのを修正しました。

• 2.12.x - r9347
• trunk - r9348

[ogawa]

comment:39 の2.10への変更取り消し確認しました。Keywordsから2.10.12を外します。

[kiwa]

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