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

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#2032 closed enhancement (fixed)

ランクアップメール受信設定機能

Reported by: imamura623 Owned by: shingo
Priority: minor Milestone: OpenPNE2.13.5
Component: pne-ktai Version: 2.14.x
Keywords: Cc:

Description (last modified by kudo)

■現象

携帯にランクアップメールが来ない

■原因

comment:4に記載(仕様)

■修正内容

PC、携帯でランクアップメールを受け取るか設定できるように修正。

又、設定を行ってない場合は従来通りPCアドレスがあればPCへ、なければ携帯アドレスへ送信するようにしました。

この修正で利用者がランクアップメールの受信設定を自身で行うことができるようになりました。

■関連情報

  • このチケットは2.10系ではDB変更が発生する為、対応できません。

元のDescription

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

携帯にランクアップメールが来ない…

Attachments (1)

20081127_pc_page_h_config.png (6.8 KB) - added by kiwa 9 years ago.

Download all attachments as: .zip

Change History (45)

comment:1 Changed 9 years ago by kiwa

Priority: minormajor

comment:2 Changed 9 years ago by kiwa

Keywords: 再現待ち removed
Priority: majorminor

再現できました

■再現バージョン

  • 2.8.x
  • 2.10.x
  • 2.11.x

■再現手順

  1. ランクアップするアクションを行う
  2. PCにランクアップメールが届く
  3. 携帯にはランクアップメールが届かない

comment:3 Changed 9 years ago by kiwa

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

comment:4 Changed 9 years ago by ebihara

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

これは仕様です。

PC版と携帯版の両方を利用しているメンバーがランクアップした場合は、PCメールアドレスのみにランクアップメールを送信します。

本来であればほかのメールとあわせて両方送信するようにするべきではありますが、現状はメンバーがランクアップメールの受信設定をおこなうことができないため、ただ単に両方送るようにするという対策を施すのは不適切です。

  • メンバーがランクアップメールの受信設定をおこなえるようにする
  • 受信設定を見てランクアップメールの送信をおこなうようにする

という変更が必要かと思われます。

今回のリリースで対処をおこなうのは難しいので、リリース項目からは外します。

comment:5 Changed 9 years ago by ogawa

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

comment:6 Changed 9 years ago by ebihara

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

comment:7 Changed 9 years ago by kiwa

Keywords: OpenPNE2.13.5 added; OpenPNE2.13.4 removed

comment:8 Changed 9 years ago by akamine

対応します。

comment:9 Changed 9 years ago by akamine

Owner: changed from nobody to akamine
Status: newassigned

comment:10 Changed 9 years ago by akamine

Description: modified (diff)

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

comment:11 Changed 9 years ago by akamine

以下のリビジョンを更に修正しました。

comment:12 Changed 9 years ago by akamine

Description: modified (diff)
Keywords: 確認中 added; OpenPNE2.10.12 removed

comment:13 Changed 9 years ago by akamine

Keywords: 確認中 removed

comment:14 Changed 9 years ago by akamine

Keywords: 確認中 added

comment:15 Changed 9 years ago by akamine

以下のリビジョンを更に修正しました。

comment:16 Changed 9 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • 現在の Description に以下の記述がありますが、
    今回の対応は2.10系では困難な為、対応を外しております。 
    

という理由で対応しないとするのはどうかと思います。

  • この変更では、PCの場合は設定値に関わらず常にメールが送信されてしまいます
  • 以下が PEAR 標準コーディング規約に準拠していません
    • webapp/lib/util/mail_send.php
      829     if ($target_c_member_config['SEND_RANK_UP_MAIL_PC']){
      
      832     if ($target_c_member_config['SEND_RANK_UP_MAIL_KTAI']){
      
      836     if ($is_pc){
      
      848     if ($is_ktai){
      
  • webapp/lib/util/mail_send.php 内の以下のインデントが適切ではありません。
    824         $is_ktai = true;
    
  • webapp_biz/modules/biz/lib/mysql_functions.php 内の以下のインデントが適切ではありません。
    1422     return ($result_ktai && $result_pc);
    

comment:17 Changed 9 years ago by akamine

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

以下のリビジョンを更に修正しました。 (webapp_biz/modules/biz/lib/mysql_functions.php 内の以下のインデントが適切ではありません。

に関しては#2151で編集したのでそちらで修正します)

comment:18 Changed 9 years ago by kiwa

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

ランクアップメールの受信設定の追加は、便利ですけど修正の範囲を超えて機能追加になっている気がします。これを安定版に取り込むのはちょっと不安です。入れるとしたら、以下のような対応にできないでしょうか。

  • 安定版 - 単純にPCアドレスの登録がない場合に携帯版へランクアップメールを飛ばすようにし、管理画面のメール送信設定・メール文言設定に携帯版ランクアップメールを追加する
  • 開発版 - 別にenhancementチケットを作成して、安定版の修正に加えてPC・携帯のランクアップメール受信設定の変更も取り込む

comment:19 Changed 9 years ago by kiwa

開発版2.13.5で先行対応して、問題なければ安定版にも入れましょう。

akamineさん

2.10.x, 2.12.xの変更を取り消してください

変更後のコードチェックを担当する方へ

安定版の変更取り消し後のコードを確認した後、問題がないようであれば以下の変更をお願いします。

  • typeを「enhancement」にする
  • Milestoneを「OpenPNE2.13.5」にする
  • keywordsに入っているバージョンを取り消し、「確認中」や「テスト待ち」のみにする
    • 開発版のコードチェックを行ったか行ってないかで適当なものに変えてください

comment:20 Changed 9 years ago by akamine

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

comment:19

の為開発版の修正を取り消しました。

comment:21 Changed 9 years ago by akamine

comment:20

のコメントは間違いです。正しくは安定版の修正を取り消しました。

comment:22 Changed 9 years ago by akamine

訂正です。 * 2.12.x - r9256

comment:23 Changed 9 years ago by shingo

Keywords: OpenPNE2.13.5 removed
Milestone: OpenPNE2.12.6OpenPNE2.13.5
Type: defectenhancement
Version: 2.10.x & 2.12.x & 2.13.x2.13.x
  • 2.12.x の変更取り消しを確認しました
  • comment:19 に挙げられた事項を行いました。

comment:24 Changed 9 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • 以下の関数について
    webapp/lib/db/point.php:db_rank_up_pcktai_config()
    
    • 変数名がわかりづらい
    • SQL文を適切な表現に修正してください
    • 返り値がint一つなので「db_get_all()」を使う必要が無い

comment:25 Changed 9 years ago by shingo

  • 」で始めるコメントアウトは「」の後に空白を入れるようです。

comment:26 Changed 9 years ago by akamine

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

以下のリビジョンで関数名db_rank_up_pcktai_configを内容、関数名共修正し、プレフィックスも修正しておきました。

comment:27 in reply to:  26 Changed 9 years ago by shingo

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

comment:26 の修正について

  • 関数db_point_rank_up_pcktai_mail_setting()の関数名について
    • db_point_is_set_rank_up_pcktai_mail_config()とかどうでしょう?


  • SQL文について、以下の点を修正してください
    • 文字列連結演算子の位置
    • where句を以下のようにすればわかりやすいのでは?
      $sql = 'SELECT COUNT(*) FROM c_member_config'
           . ' WHERE c_member_id = ?'
           . ' AND (name = "SEND_RANK_UP_MAIL_KTAI" OR name = "SEND_RANK_UP_MAIL_PCi")';
      

comment:28 Changed 9 years ago by shingo

  • 以下の関数について
    webapp/lib/util/mail_send.php:send_mail_pcktai_rank_up()
    
    • 「$send_mail_config」という変数名がわかりづらい
    • 変数「$target_c_member_config」:変数名になぜ「target~」をつけたのかがわからない
    • 「$is_pc」・「$is_ktai」という変数名がわかりづらい
    • 838行目の空行はなぜあるのかわからない

comment:29 Changed 9 years ago by akamine

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

以下のリビジョンでpage_h_configのUIを修正しました。

又以下のリビジョンでdb_point_rank_up_pcktai_mail_settingの関数名と記述、send_mail_pcktai_rank_upの記述を修正しました。

comment:30 Changed 9 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • 以下の注意書きを「携帯使用不可」時には表示しないように修正してください。(ついでに「メール受信設定」の項目も修正願います)
    ※携帯アドレスへのランクアップメールの送信は携帯の設定変更より行えます。
    
  • 「PCアドレス」という表現は他では使われていないようです
  • タイトル「ランクアップメールの設定」を統一してください(PC・携帯版も考慮)
  • webapp/modules/pc/templates/h_config.tpl
    • 160行:「checked=」の前に空白が無い・({/if})の前の空白はifの後ろにくるべき(bizも同様)
  • modules/ktai/templates/h_home.tpl
    • 表示位置をメールに関するものはまとめて表示するようにしたほうが良い(bizも同様)
  • webapp/modules/ktai/templates/h_member_config_rank_up_mail.tpl
    • PC版には「※携帯アドレスへのランクアップメールの送信は携帯の設定変更より行えます。」という注意書きがありましたがけ、携帯版ではいらないのでしょうか?

comment:31 Changed 9 years ago by kiwa

Summary: 携帯にランクアップメールが来ないランクアップメール受信設定機能

Summaryを変更しました

comment:32 Changed 9 years ago by akamine

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

以下のリビジョンで修正を行いました

  • r9401 do_h_config_3の記述の修正と関数名db_point_rank_up_pcktai_mail_configを再度修正しました
  • r9402 h_config.tplのUIを修正しました
  • r9403 ランクアップメールの設定のタイトルをPC,携帯とも「ランクアップメールの設定」で統一しました
  • r9404 携帯版のランクアップメール設定画面にPCアドレスへの受信についての注意書きを追加しました
  • r9405 BIZモードにランクアップメールの修正を反映させました

comment:33 Changed 9 years ago by akamine

以下のリビジョンで修正を行いました

  • r9407 do_h_config_3の記述を変更し、そのvalidateファイルも修正しましたdb_point_rank_up_pcktai_mail_configを再度修正しました
  • r9408 PCのh_configのランクアップメールのラジオボタンのチェック方法を修正しました
  • r9409 ランクアップメールのタイトルを「ランクアップメール受信設定」に統一し、携帯のh_config.tplのUIを携帯のh_config_mail_receiveに沿うように修正しました
  • r9410 BIZモードに上記の修正を反映させました

comment:34 Changed 9 years ago by shingo

Owner: changed from akamine to shingo
Status: assignednew

引き継ぎます。

comment:35 Changed 9 years ago by shingo

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

Changed 9 years ago by kiwa

comment:36 Changed 9 years ago by kiwa

PC版ランクアップメール受信設定について

  1. 「ポイント・ランク使用設定」「管理者ポイント加算・表示設定」の設定を考慮してください
  2. メンバー画面のメール受信設定の選択項目は「受け取る」「受け取らない」にしてください
  3. 項目名「ランクアップメール受信設定」の「設定」は不要ではないでしょうか?「メール受信設定」以外、設定項目に「設定」と入りません
  4. PC版「メール受信設定」や携帯版「ランクアップメール受信設定」のように、PCの項目にも説明をいれてはどうでしょう

上記を踏まえてこのようなUIを考えました

携帯版ランクアップメール受信設定について

  1. ktai_page_h_homeの設定変更の「ランクアップメール受信設定」は「日記コメントメール受信設定」の下に入れてください

comment:37 Changed 9 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
  • ktai_do_h_member_config_update_rank_up_mail 不要なコメント・空行を削除: trunk - r9411
  • ktai_do_h_member_config_update_rank_up_mailにvalidateを編集: trunk - r9412
  • 「ランクアップメール受信設定」に統一の修正漏れを修正: trunk - r9413
  • 表示位置をメールに関するものはまとめて表示するように変更・背景色修正: trunk - r9415
  • 「PCメールアドレス」に統一: trunk - r9416
  • PC版ランクアップメール受信設定の表示について変更:trunk - r9418

comment:38 Changed 9 years ago by ogawa

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

webapp/modules/pc/templates/h_config.tpl

<input ... value="1" ({if $C_MEMBER_CONFIG.SEND_RANK_UP_MAIL_PC}) checked="checked"({/if}) />
<input ... value="0"({if !$C_MEMBER_CONFIG.SEND_RANK_UP_MAIL_PC}) checked="checked"({/if}) />

value="[01]" の後にスペースがあるかないかが違っています。value="0" の方に合わせるのがよいと思います。

comment:39 Changed 9 years ago by shingo

  • 不要な空白を削除:trunk - r9419

comment:40 Changed 9 years ago by shingo

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

comment:41 Changed 9 years ago by ogawa

以下のリビジョンでtrunkに修正を加えています。

  • r9421, r9422 - ランクアップメールの送信部分を修正
  • r9423 - 携帯版ランクアップメール受信設定画面のデフォルト値を修正
  • r9424 - ランクアップメール受信設定画面の文言を修正

これ以前の修正の確認はまだすべて終わっていないので確認作業を続けます。

comment:42 Changed 9 years ago by ogawa

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

確認しました。テスト待ちにします。

comment:43 Changed 9 years ago by kiwa

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

comment:44 Changed 8 years ago by kudo

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