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

Opened 12 years ago

Closed 11 years ago

Last modified 10 years ago

#1474 closed defect (fixed)

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

Reported by: cake Owned by: akamine
Priority: trivial Milestone: OpenPNE2.12.6
Component: その他 Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.13.5 Cc:

Description (last modified by kudo)

■現象

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

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

■原因

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

■修正内容

comment:4

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

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

■関連情報

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

comment:4

Attachments (2)

c_home.jpg (31.4 KB) - added by cake 12 years ago.
h_com_comment_list.jpg (70.5 KB) - added by cake 12 years ago.

Download all attachments as: .zip

Change History (51)

Changed 12 years ago by cake

Attachment: c_home.jpg added

Changed 12 years ago by cake

Attachment: h_com_comment_list.jpg added

comment:1 Changed 12 years ago by kiwa

Component: pne-websiteその他
Version: 2.10.x2.8.x & 2.10.x & 2.11.x

#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の小さい順とは限りません。
  以前、私自身が別件で嵌った罠です・・・・

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

comment:2 Changed 12 years ago by kiwa

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

comment:3 Changed 12 years ago by ebihara

Priority: minorcritical

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

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

comment:4 Changed 12 years ago by kiwa

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

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

comment:5 Changed 12 years ago by kiwa

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

2.10.5では見送ります

comment:6 Changed 11 years ago by ogawa

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

comment:7 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.7 added
Milestone: OpenPNE2.12RC1

comment:8 Changed 11 years ago by ebihara

Priority: minortrivial

comment:9 Changed 11 years ago by ogawa

Keywords: OpenPNE2.12RC1 added; OpenPNE2.10.7 removed
Milestone: OpenPNE2.12RC1OpenPNE2.10.7

comment:10 Changed 11 years ago by ebihara

Keywords: OpenPNE2.12RC2 added; OpenPNE2.12RC1 removed

comment:11 Changed 11 years ago by ogawa

Keywords: OpenPNE2.10.7 added; OpenPNE2.12RC2 removed
Milestone: OpenPNE2.10.7OpenPNE2.12.1

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

comment:12 Changed 11 years ago by ogawa

Keywords: OpenPNE2.13.2 added

comment:13 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.8 added; OpenPNE2.10.7 removed
Milestone: OpenPNE2.12.1OpenPNE2.12.2

comment:14 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.9 added; OpenPNE2.10.8 removed
Milestone: OpenPNE2.12.2OpenPNE2.12.3

comment:15 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.10 added; OpenPNE2.10.9 removed
Milestone: OpenPNE2.12.3OpenPNE2.12.4

comment:16 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.3 added; OpenPNE2.13.2 removed

comment:17 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.10 OpenPNE2.13.3 removed
Milestone: OpenPNE2.12.4

comment:18 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.12 OpenPNE2.13.5 added
Milestone: OpenPNE2.12.6

comment:19 Changed 11 years ago by akamine

対応します

comment:20 Changed 11 years ago by akamine

Owner: changed from nobody to akamine
Status: newassigned

comment:21 Changed 11 years ago by akamine

Description: modified (diff)
Keywords: 確認中 added

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

comment:22 Changed 11 years ago by akamine

Description: modified (diff)

comment:23 Changed 11 years ago by akamine

Description: modified (diff)

comment:24 Changed 11 years ago by akamine

Description: modified (diff)

comment:25 Changed 11 years ago by akamine

Description: modified (diff)

comment:26 Changed 11 years ago by shingo

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

【全般】

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


【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'; 

comment:27 Changed 11 years ago by 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>
    

comment:28 Changed 11 years ago by akamine

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

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

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

comment:29 Changed 11 years ago by akamine

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

comment:30 Changed 11 years ago by akamine

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

comment:31 Changed 11 years ago by shingo

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

comment:32 Changed 11 years ago by shingo

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

comment:33 Changed 11 years ago by kiwa

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

親記事に画像が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)

comment:34 Changed 11 years ago by ebihara

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

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

comment:35 Changed 11 years ago by kiwa

Summary: トピック一覧のカメラアイコンの表示が異なるトピック一覧のカメラアイコンの表示条件が異なる

comment:36 Changed 11 years ago by kiwa

Summary: トピック一覧のカメラアイコンの表示条件が異なるトピック一覧のカメラアイコンの表示条件が統一されていない

comment:37 Changed 11 years ago by akamine

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

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

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

comment:38 Changed 11 years ago by ebihara

Owner: changed from akamine to ebihara
Status: assignednew

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

comment:39 Changed 11 years ago by ebihara

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

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

ご確認ください。

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

comment:40 Changed 11 years ago by ebihara

Owner: changed from ebihara to akamine

comment:41 Changed 11 years ago by kiwa

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

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

確認バージョン

内容

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の修正内容を更新してください。

comment:42 Changed 11 years ago by kiwa

Summary: トピック一覧のカメラアイコンの表示条件が統一されていないカメラアイコンの表示条件が統一されていない

comment:43 Changed 11 years ago by akamine

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

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

comment:44 Changed 11 years ago by ebihara

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

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

comment:45 Changed 11 years ago by akamine

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

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

comment:46 Changed 11 years ago by ebihara

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

comment:47 in reply to:  39 Changed 11 years ago by ogawa

Keywords: OpenPNE2.10.12 removed
Version: 2.10.x & 2.12.x & 2.13.x2.12.x & 2.13.x

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

comment:48 Changed 11 years ago by kiwa

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

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

comment:49 Changed 10 years ago by kudo

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