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

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1372 closed enhancement (fixed)

登録時処理の共通化をする

Reported by: ebihara Owned by: ebihara
Priority: critical Milestone: OpenPNE2.10beta2
Component: その他 Version: 2.10.x
Keywords: Cc:

Description (last modified by ogawa)

■機能概要

■仕様

登録処理共通化用のutil関数を作成して、登録処理をおこなう場面でコールする。

■関連情報

#1129で指摘された以下の問題点から、登録処理の共通化の必要が出てきたので、対応する。

  • db_member_update_c_member_pre_secure() を2度呼んでいるのが冗長
  • pc_do_o_regist_ktai_address の携帯アドレスチェックの順番が違う(is_mail_address→is_ktai_mail_address→is_sns_joinとすべき)
  • pc_do_o_regist_profとktai_do_o_regist_ktaiの共通化

現状存在する登録処理は、

  • PC新規登録(通常)
  • 携帯新規登録(通常)
  • PC新規登録(携帯個体識別番号必須性)
  • SlavePNE 等による新規登録(PNEジェイルなど)

等多岐に渡っており、これらに共通して必要な処理も以下のように複数あるが、これらは個別のアクションでばらばらに実行されている。

  • メールアドレス、パスワード等アカウント情報登録
  • プロフィール情報登録
  • 初期コミュニティへの参加
  • PNEPOINT付与

この状態は思わぬバグを引き起こす危険があるなど、あまりいい状態とは言えないので、登録処理を共通化して対処する必要がある。

関連チケット:#1129

Change History (12)

comment:1 Changed 12 years ago by ogawa

Milestone: OpenPNE2.10beta1OpenPNE2.10beta2

comment:2 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:3 Changed 12 years ago by ebihara

r4316で、登録処理共通化用のutil関数を作成しました。pc_do_o_regist_profでのみ、その関数で新規登録処理を行うようにしてあります。

comment:4 Changed 12 years ago by ebihara

Owner: changed from ebihara to nobody
Status: assignednew

手放します

comment:5 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

引き続き取り組みます。

comment:6 Changed 12 years ago by ebihara

r4341で、db_member_insert_c_member()を使用している箇所はすべてutil_regist_c_member()を使うように書き換えました。

次は db_member_insert_c_member() 以外の関数でメンバー登録処理を行っている箇所の修正です。

comment:7 Changed 12 years ago by ebihara

Keywords: 確認中 added
Owner: changed from ebihara to ogawa
Status: assignednew

r4347でdb_member_ktai_insert_c_member()を使用している箇所をutil_regist_c_member()を使うように書き換えました。

残りはスレーブPNE部分ですが、通常の登録処理とは勝手が異なるので、とりあえずここでは対応をしないことにします。

ご確認ください。

comment:8 Changed 12 years ago by ogawa

Keywords: 確認中 removed
Owner: changed from ogawa to ebihara

comment:9 Changed 12 years ago by ogawa

Resolution: fixed
Status: newclosed

comment:10 Changed 12 years ago by ebihara

Description: modified (diff)

comment:11 Changed 12 years ago by ebihara

Description: modified (diff)

comment:12 Changed 12 years ago by ogawa

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