誕生日の設定されていないメンバーのフレンドホームに「もうすぐお誕生日です！」のメッセージが表示される

[kiwa]

■現象

1. 管理画面のメンバー管理、csvインポートでメンバーを作成する。ニックネームとメールアドレスとパスワードしかないので誕生日は０月０日として登録される。
2. 携帯からCSVインポートで作成したメンバーのフレンドホームを表示させる
3. 「☆もうすぐお誕生日です！☆」のメッセージが表示されている

■原因

誕生日等のある日付までの日数を計算している関数で０月０日に対応していなかった。推測ですが０月は前年の１２月として解釈されているのではないか？

■修正内容

今日と次の誕生日までの日数を計算するところで、誕生日が０月０日ならば、エラーとして扱い。表示用のテンプレートではエラーのときに何も表示させない処理を入れる

修正リビジョン

• Trunk
  • r5069
• 2.8.x
  • r5068
• 2.10.x
  • r5070 r5071

■関連情報

webapp/lib/utilのgetCountdownDays

[kiwa]

書き込みしたとき（2007/12/14）には全バージョンでPC・携帯ともに表示されていましたが、

本日（2008/1/17）確認したら表示されなくなっていました。

０月０日の誕生日判定がどこかにあるものと思われます。

[yamaji]

CSVアップロード機能で作成されたメンバーは誕生日が０月０日であることが分かったので、今日と次の誕生日までの日数を計算するところで、誕生日が０月０日ならば、次の誕生日までは4999年などの大きな値を返すようにすれば、問題は解消するはずです。

[kunitada]

CSVインポートでメンバー追加すると、
項目の順序「ニックネーム」「登録メールアドレス」「パスワード」
しか登録されないのか。。。

選択肢としては、
1) 誕生日がDB登録されて無い状態でも、OpenPNEを動くようにする
2) csvインポート時は、固定で誕生日を1970 1/1 とかにしてしまう。
3) OpenPNEのシステムとしての必須項目なので、初回ログイン時に入力してもらう(だが入力する前もc_memberはあるので、お誕生日メールとか挙動確かめる必要有り)。

ぐらいかな？
僕の感覚だと「3)」だけど、どうかな？また、他に選択肢はないかな。
[14:34:18] おがわおがわの発言: OpenPNE君も0月0日ですよ
[14:34:27] おがわおがわの発言: 1でしょう
[14:35:13] おがわおがわの発言: 誕生日まであと何日かを計算する関数が
[14:35:14] おがわおがわの発言: どこかにあるので
[14:35:28] おがわおがわの発言: その辺を何とかするとうまくいくと思います
[14:35:43] 山路 正樹の発言: getCountdownDays　です
[14:40:00] kunitada@shinjukuの発言: なるほど、では1で。

getCountdownDaysが0/0だと-1返すとかがいいんじゃないかな？
そんでそれ受ける処理でエラー処理とか。

[kunitada]

[14:52:50] おがわおがわの発言: んーこれはちょっと難しいですね
[14:53:12] おがわおがわの発言: 一概にどちらがいいとは言えないですが
[14:55:02] おがわおがわの発言: テンプレート内の条件式が

({if $days_birthday == 0})({* 誕生日当日 *})
({elseif $days_birthday <= 3})({* 誕生日3日以内 *})
[14:55:08] おがわおがわの発言: のようになっているので
[14:55:51] おがわおがわの発言: このあたりを変更しないということであれば、山路さん案が有力になります
[14:58:36] おがわおがわの発言: 条件式を変更するんだということになれば國忠さん案の方が素直かなあという気もします
[15:01:18] kunitada@shinjukuの発言: 僕的には國忠案でいきたいなぁ。tpl書き換えても良いと思うので。。。
その方がきれいだと思う。後でソース見た人もわかりやすいだろうし。。。
(なぜ4999年！？とかなくなるとおもう)
[15:19:08] 山路 正樹の発言: 多少手間がかかりますが、-1をエラーコードとして誕生日が来てから1秒経過にならないようにエラー処理を行うことにしましょう
[15:31:10] kunitada@shinjukuの発言: では、それで。携帯・PCのテンプレートがあるので注意って感じでお願いします
[15:32:39] 山路 正樹の発言: 了解

[yamaji]

修正リビジョン
Trunk
r5069
2.8.x
r5068
2.10.x
r5070

[yamaji]

2.10.x修正リビジョン r5071

[ogawa]

getCountdownDays() に細かい修正入れました。

• trunk - r5118, r5119, r5121
• 2.10 - r5120, r5122
• 2.8 - r5123