#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 )
■現象
c_homeとh_com_comment_list(マイホームのトピック一覧)で、カメラアイコンの表示が異なる場合があります。
画像有無を判断しているコメントが違うようです。
■原因
各ページ毎の呼び出す関数によって *最新トピックに画像が添付されているか? or *親トピックに画像が添付されているか? のように判定方法が異なっていたため
■修正内容
にあるように/webapp/lib/db/commu.php内の関数を統一条件が親トピックに画像が添付されているか否かに修正
この修正でカメラアイコンの表示が親トピックに画像が添付されている場合に統一されました。
■関連情報
#1482:カメラアイコン表示の基準が統一されていない
Attachments (2)
Change History (51)
Changed 15 years ago by
Attachment: | c_home.jpg added |
---|
Changed 15 years ago by
Attachment: | h_com_comment_list.jpg added |
---|
comment:1 Changed 15 years ago by
Component: | pne-website → その他 |
---|---|
Version: | 2.10.x → 2.8.x & 2.10.x & 2.11.x |
comment:2 Changed 15 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 added |
---|---|
Milestone: | → OpenPNE2.10.5 |
comment:3 Changed 15 years ago by
Priority: | minor → critical |
---|
修正方法を明確にしていただいているので、是非対応したいです。
プライオリティをあげます。
comment:4 Changed 15 years ago by
日記・トピックなどカメラアイコンが表示される全ての場所(PC・携帯ともに)で表示条件を統一させましょう
統一条件:親記事に画像がついている場合
comment:5 Changed 15 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 removed |
---|---|
Milestone: | OpenPNE2.10.5 |
Priority: | critical → minor |
2.10.5では見送ります
comment:6 Changed 15 years ago by
Version: | 2.8.x & 2.10.x & 2.11.x → 2.10.x & 2.12.x & 2.13.x |
---|
comment:7 Changed 15 years ago by
Keywords: | OpenPNE2.10.7 added |
---|---|
Milestone: | → OpenPNE2.12RC1 |
comment:8 Changed 15 years ago by
Priority: | minor → trivial |
---|
comment:9 Changed 15 years ago by
Keywords: | OpenPNE2.12RC1 added; OpenPNE2.10.7 removed |
---|---|
Milestone: | OpenPNE2.12RC1 → OpenPNE2.10.7 |
comment:10 Changed 15 years ago by
Keywords: | OpenPNE2.12RC2 added; OpenPNE2.12RC1 removed |
---|
comment:11 Changed 15 years ago by
Keywords: | OpenPNE2.10.7 added; OpenPNE2.12RC2 removed |
---|---|
Milestone: | OpenPNE2.10.7 → OpenPNE2.12.1 |
安定版マイナーバージョンアップリリースに回します。
comment:12 Changed 15 years ago by
Keywords: | OpenPNE2.13.2 added |
---|
comment:13 Changed 15 years ago by
Keywords: | OpenPNE2.10.8 added; OpenPNE2.10.7 removed |
---|---|
Milestone: | OpenPNE2.12.1 → OpenPNE2.12.2 |
comment:14 Changed 15 years ago by
Keywords: | OpenPNE2.10.9 added; OpenPNE2.10.8 removed |
---|---|
Milestone: | OpenPNE2.12.2 → OpenPNE2.12.3 |
comment:15 Changed 15 years ago by
Keywords: | OpenPNE2.10.10 added; OpenPNE2.10.9 removed |
---|---|
Milestone: | OpenPNE2.12.3 → OpenPNE2.12.4 |
comment:16 Changed 14 years ago by
Keywords: | OpenPNE2.13.3 added; OpenPNE2.13.2 removed |
---|
comment:17 Changed 14 years ago by
Keywords: | OpenPNE2.10.10 OpenPNE2.13.3 removed |
---|---|
Milestone: | OpenPNE2.12.4 |
comment:18 Changed 14 years ago by
Keywords: | OpenPNE2.10.12 OpenPNE2.13.5 added |
---|---|
Milestone: | → OpenPNE2.12.6 |
comment:20 Changed 14 years ago by
Owner: | changed from nobody to akamine |
---|---|
Status: | new → assigned |
comment:21 Changed 14 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added |
comment:22 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:23 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:24 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:25 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:26 Changed 14 years ago by
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 14 years ago by
- 現状、トピック親記事に画像アップロード後、削除してもアイコンは残ってしまいます。
- 以下の部分のカメラアイコン表示の処理が無効なようです。(一応確認してください)チケットの内容とは異なりますが関連事項として処理してください。
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})&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 14 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
comment:32 Changed 14 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:33 Changed 14 years ago by
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 14 years ago by
追加で指摘させてください。
- db_commu_get_last_c_topic_comment() を消しているようですが、これは残しておいてください。
comment:35 Changed 14 years ago by
Summary: | トピック一覧のカメラアイコンの表示が異なる → トピック一覧のカメラアイコンの表示条件が異なる |
---|
comment:36 Changed 14 years ago by
Summary: | トピック一覧のカメラアイコンの表示条件が異なる → トピック一覧のカメラアイコンの表示条件が統一されていない |
---|
comment:37 Changed 14 years ago by
comment:38 Changed 14 years ago by
Owner: | changed from akamine to ebihara |
---|---|
Status: | assigned → new |
2.10.x の修正については海老原が引き受けます。
comment:39 follow-up: 47 Changed 14 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
2.10.x に関しては 2.12.x や trunk と違い複雑なSQL文になりうるため、対応をおこないません。そのため、r9335 でこのチケットに関する2.10.xへの変更を取り消しました。
ご確認ください。
※2.12.x と trunk の変更についてはすでに海老原確認済みです。
comment:40 Changed 14 years ago by
Owner: | changed from ebihara to akamine |
---|
comment:41 Changed 14 years ago by
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 14 years ago by
Summary: | トピック一覧のカメラアイコンの表示条件が統一されていない → カメラアイコンの表示条件が統一されていない |
---|
comment:43 Changed 14 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added; 差し戻し removed |
comment:44 Changed 14 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
h_homeですが、BIZモードに対しても修正の必要があるのではないでしょうか
comment:45 Changed 14 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
comment:46 Changed 14 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:47 Changed 14 years ago by
Keywords: | OpenPNE2.10.12 removed |
---|---|
Version: | 2.10.x & 2.12.x & 2.13.x → 2.12.x & 2.13.x |
comment:39 の2.10への変更取り消し確認しました。Keywordsから2.10.12を外します。
comment:48 Changed 14 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | new → closed |
確認しました。問題ありません。
comment:49 Changed 14 years ago by
Description: | modified (diff) |
---|
#1482 より
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=13750 より転記
記事に画像が添付されています。