Backport（バックポート） #1250
PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない
100%
説明
Overview (現象)¶
PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない。
管理画面のメンバー一覧やメンバーページの携帯メールアドレス設定に携帯アドレスが表示されない。
管理画面メンバー一覧では個体識別番号は登録されている。
PC・携帯ともにログインすることは可能です。
再現方法¶
1）招待制ではなく、新規登録できる様に設定
2）管理画面のSNS設定、携帯関連設定、メンバー登録時の携帯個体識別番号取得設定をPC・携帯登録時に携帯個体識別番号の登録を必須にするに設定
3）PCログイン画面から新規登録。メールを飛ばし、招待状のURLから登録画面を開く
4）フォームに必要事項を入力し登録
5) 携帯に送られた登録メールから携帯の登録を実施する
6) 管理画面のメンバー一覧または設定変更ページの携帯アドレス登録画面を確認する
再現バージョン¶
- OpenPNE3.0.9-dev
- OpenPNE3.2.7-dev
- OpenPNE3.4.7-dev
- OpenPNE3.6beta1-dev
Causes (原因)¶
Way to fix (修正内容)¶
関連するチケット
関係しているリビジョン
履歴
#1
tu nguyen ngoc がほぼ14年前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を tu nguyen ngoc にセット
#2
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 a77701aa7b819c27ee3fed25e728da8abf353667 で適用されました。
#3
Mutsumi Imamura がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
動作確認しました。
すでに登録済みの携帯アドレスを登録することが出来きてしまいます。（携帯アドレスが重複してしまう）
opAuthMailAddressPluginの問題かもしれませが、ご確認をお願いします。
#4
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 c435d0db73c0eb3214050d0a3f84e2bc3147d08f で適用されました。
#5
Mutsumi Imamura がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
確認しました。
登録済みの携帯アドレスで再度登録を試みると、エラーにはなりますがエラーメッセージが日本語で表示されません。ご確認をお願いします。
#6
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 5267660e9d82b2dd47435b94a5e926b4994cca89 で適用されました。
#7
tu nguyen ngoc がほぼ14年前に更新
更新履歴 9d693afbb4463ce85f4551ed829795315cec8070 で適用されました。
#8
tu nguyen ngoc がほぼ14年前に更新
更新履歴 e43a477d0d912b42933e3bd880b46af560fe2089 で適用されました。
#9
Mutsumi Imamura がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
英語時のエラーメッセージが「Invalid mobile_address address.」となっており、addressが重複するのに違和感を感じます。「invalid e-mail address」などではいかがでしょうか？
#10
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 92febf518421ef2784e79d54dff2f9546a8fd5fc で適用されました。
#12
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
以下の変更箇所の行末に不要なスペースが残っています。
- apps/pc_frontend/modules/member/lib/registerMobileForm.class.php
- 27行目
- 35行目
- i18n/messages.ja.xml
- 224行目
#13
Rimpei Ogawa がほぼ14年前に更新
ソースコードアップデート前に既に携帯登録メールが送られている場合の考慮がなされていません。
memberActions::executeRegisterMobileToRegisterEnd() の $pre->getValue() としている箇所でエラーになります。
#14
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 81e8c22315daa31dc770fa64f978a328e9049fba で適用されました。
#15
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
スペースに関しては確認OKです。アップデート前にメールが送信されていた場合の対応をお願いします。
#16
Mutsumi Imamura がほぼ14年前に更新
Rimpei Ogawa は書きました:
ソースコードアップデート前に既に携帯登録メールが送られている場合の考慮がなされていません。
memberActions::executeRegisterMobileToRegisterEnd() の $pre->getValue() としている箇所でエラーになります。
指摘通り再現しました。エラーメッセージを追記しておきます。
( ! ) Fatal error: Call to a member function getValue() on a non-object in /home/tetete/sns/sns.example.com/apps/mobile_frontend/modules/member/actions/actions.class.php on line 164
Call Stack
# Time Memory Function Location
1 0.0003 52128 {main}( ) ../mobile_frontend_dev.php:0
2 2.9258 21720848 sfContext->dispatch( ) ../mobile_frontend_dev.php:14
3 2.9258 21720848 sfFrontWebController->dispatch( ) ../sfContext.class.php:170
4 2.9268 21739728 sfController->forward( ) ../sfFrontWebController.class.php:48
5 2.9665 22548448 sfFilterChain->execute( ) ../sfController.class.php:229
6 2.9675 22564400 sfRenderingFilter->execute( ) ../sfFilterChain.class.php:53
7 2.9675 22564400 sfFilterChain->execute( ) ../sfRenderingFilter.class.php:33
8 2.9686 22573068 opCacheControlFilter->execute( ) ../sfFilterChain.class.php:53
9 2.9686 22573068 sfFilterChain->execute( ) ../opCacheControlFilter.class.php:27
10 2.9697 22584400 opCheckEnabledApplicationFilter->execute( ) ../sfFilterChain.class.php:53
11 2.9708 22601924 sfFilterChain->execute( ) ../opCheckEnabledApplicationFilter.class.php:39
12 2.9718 22617604 sfDenyFromNonMobileFilter->execute( ) ../sfFilterChain.class.php:53
13 2.9728 22618220 sfFilterChain->execute( ) ../sfDenyFromNonMobileFilter.class.php:39
14 2.9739 22635212 sfMobileIOFilter->execute( ) ../sfFilterChain.class.php:53
15 2.9820 22635784 sfFilterChain->execute( ) ../sfMobileIOFilter.class.php:38
16 2.9831 22656044 opEmojiFilter->execute( ) ../sfFilterChain.class.php:53
17 2.9831 22656044 sfFilterChain->execute( ) ../opEmojiFilter.class.php:29
18 2.9843 22677632 sfExecutionFilter->execute( ) ../sfFilterChain.class.php:53
19 2.9845 22677896 sfOpenPNEExecutionFilter->handleAction( ) ../sfExecutionFilter.class.php:42
20 2.9853 22682016 sfExecutionFilter->handleAction( ) ../sfOpenPNEExecutionFilter.class.php:73
21 2.9854 22682016 sfExecutionFilter->executeAction( ) ../sfExecutionFilter.class.php:78
22 3.0295 23068584 sfActions->execute( ) ../sfExecutionFilter.class.php:92
23 3.0306 23099100 memberActions->executeRegisterMobileToRegisterEnd( ) ../sfActions.class.php:60
#17
tu nguyen ngoc がほぼ14年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 47c8c261b9a60549b188cdf701b9a21bce193416 で適用されました。
#18
Rimpei Ogawa がほぼ14年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
OKです。
#19
Mutsumi Imamura がほぼ14年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
アップデート前にメールが送信されていた場合の動作もOKです。
アップデート後の正常登録動作も問題ありません。