Project

General

Profile

Enhancement(機能追加・改善) #3829

Updated by Youichi Kimura almost 5 years ago

h3. Overview (概要)

現状、member_config の pc_address_pre は下記の 2 つの目的で使用されている。

* メンバーの新規登録または招待時に入力されたメールアドレスの保持
** メンバー登録が完了するまで有効
** 登録が完了すると、name 完了すると name が pc_address_pre から pc_address に変更される。
** 参照: "@opAuthRegisterFormMailAddress::doSave()@":https://github.com/openpne/OpenPNE3/blob/OpenPNE-3.8.16/plugins/opAuthMailAddressPlugin/lib/form/opAuthRegisterFormMailAddress.class.php#L52-L60
* メンバーのメールアドレス変更時に、確認手続きが完了するまでの変更後のメールアドレスの保持
** 変更後のメールアドレスの確認手続きが完了するまで有効 変更後のメールアドレスの確認手続きが完了されるまで有効
** 確認手続きが完了すると、name 変更後のメールアドレスに送信された確認 URL にアクセスし確認手続きが完了すると、name が pc_address_pre から pc_address に変更される。
** 参照: "@opMemberAction::executeConfigComplete()@":https://github.com/openpne/OpenPNE3/blob/OpenPNE-3.8.16/lib/action/opMemberAction.class.php#L301-L317

このように pc_address_pre が複数の意味を持っている現状では、新規登録の実装に手を加える際にはメールアドレス変更の影響を受けないか、実装やレビュー時に余分に考慮しなければならない。(メールアドレス変更の実装に変更を加える場合も同様)
新規登録はユーザーの権限に影響を及ぼすため変更に慎重を要することや、新規登録のフローは #3416 のコメントに記載されているように複雑な手順を踏むことから、考慮しなければならない事項をなるべく減らすためにも pc_address_pre を複数の目的で使用することは避けるべきである。

h3. Spec (仕様)

新規登録かメールアドレス変更のいずれかで pc_address_pre 以外の別の名前を使用するように変更する。
ただし、いずれの場合も既存の pc_address_pre を仕様変更後にどのように扱うか考慮する必要がある。(マイグレーション時に pc_address_pre の用途を判別して適切な名前に変更するなど)

Back