BIZ：携帯ユーザにスケジュールお知らせメールが来ない

[imamura623]

■現象

​http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=3365 より転記

2.10.1を使用。

携帯で登録したユーザーが、
予定登録を行っても、当日のお知らせメールが送信されないのですが、
新しいバージョンでは送信されるようになっているでしょうか？

■原因

仕様でPCのみに送信するようになっていた為。

■修正内容

まず、PCアドレスがあればPCにメール送信し、なければ携帯のメールアドレスに送信するようにしました。
またこの修正で管理画面より携帯に送信を行うかどうかの設定も追加しました。
この修正で携帯アドレスのみのユーザにもスケジュールお知らせメール送信するようになりました。

■関連情報

• comment:14

[akamine]

対応します。

[akamine]

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

• 2.12.x - r9077

[akamine]

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

• 2.12.x - r9079

[akamine]

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

• 2.12.x - r9126

[ebihara]

• BIZの関数には biz_ という prefix を付けてください
• この変更では、PCの場合は設定値に関わらず常にメールが送信されてしまいます
• 変数展開等の必要ない文字列はダブルクオートではなくシングルクオートで囲うよう心がけてください。ただし、以下のように意味もなくシングルクオートとダブルクオートが一ヶ所に混在しているのは好ましくありませんので、こういった箇所は確実に修正してください。

  1415             $params = array(
  1416                 'c_member'    => db_member_c_member4c_member_id_LIGHT($key),
  1417                 "c_schedule_list" => $value,
  1418                 'login_url'   => $login_url,
  1419             );
  

• 以下が PEAR 標準コーディング規約に準拠していません
  • webapp_biz/modules/biz/lib/mysql_functions.php

    1387     if ($target_c_member_config['SEND_SCHEDULE_MAIL_PC']){
    

    1391     if ($target_c_member_config['SEND_SCHEDULE_MAIL_KTAI']){
    

  • webapp_biz/modules/ktai_biz/page/h_member_config_schedule_mail.php

    13         $OPTION =util_get_c_member_config($u);
    

    16         $SEND_SCHEDULE_MAIL_KTAI =$OPTION['SEND_SCHEDULE_MAIL_KTAI'];
    

• webapp_biz/modules/biz/lib/mysql_functions.php 内の以下のインデントが適切ではありません。

  1422     return ($result_ktai && $result_pc);
  

• webapp_biz/modules/ktai_biz/page/h_member_config_schedule_mail.php で、 OPTION と変数名を全て大文字で表現していることに対して意味を見いだせません。意図があるようであれば説明をお願いします。

[akamine]

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

• 2.12.x - r9141
• trunk - r9142

[ebihara]

• biz_db_schedule_pcktai_config() という関数名から、スケジュール通知メール設定数を取得できる関数であると理解するのは難しいように思います。関数の役割に従って関数名を変更するか、もしくは関数名に従って関数の役割を変更してください。
• 取得するのが COUNT(*) の結果だけなら、 db_get_all() を使用する必要性はないのではないでしょうか
• biz_db_schedule_pcktai_config() がコメント通り 「スケジュール通知メールの設定があるかどうかを取得」という役割しか担わないのであれば、単に bool 値を返す関数にするべきです
• SEND_SCHEDULE_MAIL_KTAI は大文字のままにしておくのが望ましいと思います
• comment:10 の、「ただし、以下のように意味もなくシングルクオートとダブルクオートが一ヶ所に混在しているのは好ましくありませんので、こういった箇所は確実に修正してください」に関しては comment:11 の修正に反映されているようですが、「変数展開等の必要ない文字列はダブルクオートではなくシングルクオートで囲うよう心がけてください」に関しては反映されていませんが、これにはなにか意図があるのでしょうか（修正までは必要ありませんが。。。）
• 主旨や内容の違う修正は、たとえ同一チケットのものであっても分けてコミットするようにしてください

[akamine]

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

• 2.12.x - r9209,r9207
• trunk - r9210,r9208

[kiwa]

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

• 安定版 - 単純に携帯版へスケジュールお知らせメールを飛ばすようにし、管理画面のメール送信設定・メール文言設定に携帯版スケジュールお知らせメールを追加する
• 開発版 - 別にenhancementチケットを作成して、安定版の修正に加えてPC・携帯のスケジュールお知らせメール受信設定の変更も取り込む

[akamine]

comment:14

の為、安定版の修正をいったん取り消します。

• 2.12.x - r9255

[akamine]

comment:15

で取り消しきれていなかった分を取り消しました。

• 2.12.x - r9259

[akamine]

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

• 2.12.x - r9209

修正内容はcomment:14
を参考にしています。

[akamine]

訂正修正したリビジョンは * 2.12.x - r9263

[ebihara]

• この対処であれば、 2.12.x だけでなく 2.10.x に対してもおこなえるような気がするので、対応を検討してください
• 開発版の機能改善についてはこのチケットでは対応しないので、新規に対応用のチケットを作ってください

[akamine]

以下のリビジョンで2.12の変更箇所を2.10に反映しました。

• 2.10.x - r9288
  

• #3159に新規対応チケットを発行しました。

(comment:19 より)

[ebihara]

テスト待ちにします。

2.13.5 については #3159 で対応する物とし、とりあえず今のところはキーワードから外しておきます。このチケットでおこなった対応については派生チケット側で注意しましょう。

[kiwa]

安定版のみ動作確認を行いました。(開発版の確認は#3159で行います)

1. 携帯メールアドレス向け「スケジュールお知らせメール」の「({$login_url})」の部分にURLが表示されません
2. PC・携帯両方のメールアドレスを登録しているメンバーに対して、PC・携帯両方にスケジュールお知らせメールが送信されるのは意図した仕様でしょうか？
   • 修正として問題はありませんが、#2032は「PC・携帯両方に届く」という仕様を避けるために開発版先行対応になったので、特に理由がなければこちらも「単純にPCアドレスの登録がない場合に携帯版へ」スケジュールお知らせメールを飛ばすという仕様にしてはどうでしょう？
3. PC・携帯ともメッセージお知らせメールが「参加者」ではなく「登録者」にのみ飛ばされるのは、この修正以前からの仕様でしょうか？
   • 以前からこの仕様だった場合は、別にバグチケットを作成します（今回は対応しません）
   • 今回の変更でそのようになった場合は、修正前の送信の仕様に合わせてください
4. Descriptionの修正内容を更新してください

[akamine]

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

• 2.10.x - r9320
• 2.12.x - r9319

*上記コメントに従い、まずpcアドレスがあればに送信、なければ携帯アドレスに送信するように修正しました。
*お知らせメールが登録者のみに飛ばされるのは修正以前からの使用ですのでチケットの作成をお願いします。

[kiwa]

*お知らせメールが登録者のみに飛ばされるのは修正以前からの使用ですのでチケットの作成をお願いします。

#3161 チケット作成しました。

修正された部分の動作は問題ありませんでした。