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

Opened 13 years ago

Closed 13 years ago

#685 closed defect (fixed)

CSVインポートで登録したユーザの生年月日が「1700/1/1」になる

Reported by: kiwa Owned by: ebihara
Priority: minor Milestone: OpenPNE2.6.7
Component: owner【オーナー向け機能】 Version:
Keywords: Cc:

Description

「メンバー管理」において「CSVインポート」を行った際に、
CSVファイルから登録されたメンバーの生年月日が「1700/1/1」と表示されます。

・「マイページ」の「プロフィール設定」の「生まれた年」
上記項目では1900年以前の表示または入力が行えない為、仕様を統一するか
CSVファイルのインポートを行った際には生年月日は未記入の状態の表示にする方が
よろしいかと思われます。

Change History (22)

comment:1 Changed 13 years ago by ogawa

Owner: changed from ogawa to nobody

comment:2 Changed 13 years ago by ebihara

Owner: changed from nobody to ebihara

やります。

comment:3 Changed 13 years ago by ebihara

Status: newassigned

comment:4 Changed 13 years ago by ebihara

ちなみに、SNS初期設定直後の1番ユーザ(OpenPNE君)の生年月日は、0/1/1となっています。ということで、CSVインポートで登録したユーザの生年月日もこれにあわせて、0/1/1とするように修正します。

comment:5 Changed 13 years ago by ebihara

以下のリビジョンで個人用ブランチにコミットしました。マージしてみて問題がなければ、リリースブランチにコミットします。

comment:6 Changed 13 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンでリリースブランチにコミットしました。ご確認ください。

comment:7 Changed 13 years ago by ogawa

Status: newassigned

確認します。

comment:8 Changed 13 years ago by ogawa

Owner: changed from ogawa to ebihara
Status: assignednew

初期メンバー(OpenPNE君)、CSVインポートで登録されたメンバーともに「0/0/0」とはできないでしょうか。

  • カレンダーの誕生日表示
  • 誕生日お知らせメール

等で誤動作がないことを確認して欲しいです。

また、管理画面の「世代別メンバー数表示」の対象外にしたいので、こちらの調査もお願いします。

comment:9 Changed 13 years ago by ebihara

Status: newassigned

了解です。調査、対応します。

comment:10 Changed 13 years ago by ebihara

ユーザの誕生日を0/0/0にした際の調査結果です。

  • bin/tool_send_birthday_mail.php(誕生日お知らせメール)
    • 誕生日お知らせメールが送られてくることはない(db_member_c_member_list4birthday_mail()の対象とならないため)
  • admin_page_list_c_member(ユーザリスト)
    • 「生年月日」が年・月・日ともすべて空欄
    • 「生年月日」によるソートの対象になる
    • 「生年月日」による絞り込みの対象になる(「選択してください」~「1907」)
  • admin_page_user_analysis_generation(世代別メンバー数表示)
    • 統計の対象になる(80~に含まれている)
  • pc_page_f_home
  • pc_page_h_prof
    • プロフィール項目の「誕生日」が「0月0日」と表示される
    • 週間カレンダーの誕生日表示は正常(BIZでも正常)
  • pc_page_h_config_prof
    • 「生まれた年」が「0」
    • 「誕生日」が「『--』月『--』日」
  • ktai_page_f_home
    • プロフィール項目で「誕生日:0月0日」と表示される
  • ktai_page_h_config_prof
    • 「生まれた年」が「0」
    • 「誕生日」が「『1』『1』」(PC版のように「--」がない)

comment:11 Changed 13 years ago by ebihara

ユーザの誕生日を0/0/0にした際の調査結果(追加)

  • ktai_biz_page_fh_calendar_week
  • pc_page_h_calendar
  • biz_page_fh_biz_schedule_calendar
    • いずれも正常表示

comment:12 Changed 13 years ago by ebihara

admin_page_user_analysis_generation(世代別メンバー数表示)の統計の対象にならないようにするための手段を以下に示します。

get_analysis_generation()で各世代の人数の計算をしています(ここで、誕生年が0のユーザは2007歳と判定されています)。この関数のなかの以下のSQL文を、誕生年が0のユーザが引っかからないように書き換えることで対処できます。

    $sql = "select ((year(curdate()) - birth_year)- (RIGHT(CURDATE(),5)<concat(birth_month,'-',birth_day))) as age from c_member;";

comment:13 Changed 13 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

以上の調査結果を見ていただいて、どの部分をどう対処すればよいか、チェック漏れがないかなど、ご判断いただければと思います。

comment:14 Changed 13 years ago by ebihara

Owner: changed from ogawa to ebihara
Status: newassigned

いったん ebihara 担当に戻します。

個人用ブランチに、以下の修正を加え、コードをコミットします。リリースブランチにどのコードをリビジョンするかどうかをご判断いただければと思います。

  • admin_page_list_c_member
    • 生年月日によるソートの対象にしない
    • 生年月日による絞り込みの対象にしない
  • admin_page_user_analysis_generation
    • 統計の対象にしない
  • ktai_page_h_config_prof
    • 「誕生日」に「--」を追加

comment:15 Changed 13 years ago by ebihara

  • admin_page_list_c_member
    • 生年月日によるソートの対象にしない

これはやっぱりナシ。

comment:16 Changed 13 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

個人用ブランチに、初期ユーザ・CSVインポートで登録したユーザの生年月日を0/0/0にし、それに伴う修正を加えたものを、r2760:2763にコミットしました。

どのコードを取り込むかなどの回答をよろしくお願いします。

comment:17 Changed 13 years ago by ebihara

Owner: changed from ogawa to ebihara
Status: newassigned

誕生日が「0月0日」などとなってしまう場合には非表示にするように修正します。

comment:18 Changed 13 years ago by ebihara

  • r2782に、誕生日が「0月0日」になるなど不自然な表示を改善しました
  • 上で加えたr2724以外の変更はすべて2.8beta7のコードに加えたものですので、r2782の変更も含めて、2.6.7向けの変更をr2789:2791にコミットしました

以上、個人用ブランチにコミットしたすべての変更を、マージしてみて問題がないようでしたら、リリースブランチにコミットします。

comment:19 Changed 13 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンでリリースブランチにコミットしました。ご確認ください。

comment:20 Changed 13 years ago by ogawa

Status: newassigned

確認します。

comment:21 Changed 13 years ago by ogawa

Owner: changed from ogawa to ebihara
Status: assignednew

comment:22 Changed 13 years ago by ogawa

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.