Project

General

Profile

Backport(バックポート) #1250

PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない

Added by Mutsumi Imamura about 9 years ago. Updated about 9 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-05-13
Due date:
2010-07-12
% Done:

100%


Description

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 (修正内容)


Related issues

Related to OpenPNE 3 - Bug(バグ) #1069: PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない Fixed(完了) 2010-05-13

Associated revisions

Revision a77701aa (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) fixed bug: mobile address is not registered when UID is required

Revision c435d0db (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) added validator to check mobile_address is dublicated or not

Revision 5267660e (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) i18n of error message

Revision 9d693afb (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) revised Japanese error message

Revision e43a477d (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) deleted 2 bytes space in translation file

Revision 92febf51 (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) deleted dublicated "address"

Revision 81e8c223 (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) removed unnecessary spaces at the end of lines

Revision 47c8c261 (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1250, BP from #1069) check if $pre is null

History

#1 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to tu nguyen ngoc

#2 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 a77701aa7b819c27ee3fed25e728da8abf353667 で適用されました。

#3 Updated by Mutsumi Imamura about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

動作確認しました。
すでに登録済みの携帯アドレスを登録することが出来きてしまいます。(携帯アドレスが重複してしまう)
opAuthMailAddressPluginの問題かもしれませが、ご確認をお願いします。

#4 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 c435d0db73c0eb3214050d0a3f84e2bc3147d08f で適用されました。

#5 Updated by Mutsumi Imamura about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

確認しました。
登録済みの携帯アドレスで再度登録を試みると、エラーにはなりますがエラーメッセージが日本語で表示されません。ご確認をお願いします。

#6 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 5267660e9d82b2dd47435b94a5e926b4994cca89 で適用されました。

#7 Updated by tu nguyen ngoc about 9 years ago

更新履歴 9d693afbb4463ce85f4551ed829795315cec8070 で適用されました。

#8 Updated by tu nguyen ngoc about 9 years ago

更新履歴 e43a477d0d912b42933e3bd880b46af560fe2089 で適用されました。

#9 Updated by Mutsumi Imamura about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

英語時のエラーメッセージが「Invalid mobile_address address.」となっており、addressが重複するのに違和感を感じます。「invalid e-mail address」などではいかがでしょうか?

#10 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 92febf518421ef2784e79d54dff2f9546a8fd5fc で適用されました。

#11 Updated by Mutsumi Imamura about 9 years ago

  • Due date set to 2010-07-12

確認しました。テストOKです。

#12 Updated by Rimpei Ogawa about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

以下の変更箇所の行末に不要なスペースが残っています。

  • apps/pc_frontend/modules/member/lib/registerMobileForm.class.php
    • 27行目
    • 35行目
  • i18n/messages.ja.xml
    • 224行目

#13 Updated by Rimpei Ogawa about 9 years ago

ソースコードアップデート前に既に携帯登録メールが送られている場合の考慮がなされていません。
memberActions::executeRegisterMobileToRegisterEnd() の $pre->getValue() としている箇所でエラーになります。

#14 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 81e8c22315daa31dc770fa64f978a328e9049fba で適用されました。

#15 Updated by Rimpei Ogawa about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

スペースに関しては確認OKです。アップデート前にメールが送信されていた場合の対応をお願いします。

#16 Updated by Mutsumi Imamura about 9 years ago

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 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 47c8c261b9a60549b188cdf701b9a21bce193416 で適用されました。

#18 Updated by Rimpei Ogawa about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

OKです。

#19 Updated by Mutsumi Imamura about 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

アップデート前にメールが送信されていた場合の動作もOKです。
アップデート後の正常登録動作も問題ありません。

Also available in: Atom PDF