Project

General

Profile

Backport(バックポート) #919

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

Added by Naoto Inoue over 12 years ago. Updated over 12 years ago.

Status:
Fixed(完了)
Priority:
Urgent(急いで)
Assignee:
Target version:
Start date:
2010-03-30
Due date:
% Done:

100%


Description

バグ?と思われる物を見つけたので報告
バージョン:3.4.2
現象:
1)招待制ではなく、新規登録できる様に設定
2)管理画面のSNS設定、携帯関連設定、メンバー登録時の携帯個体識別番号取得設定をPC・携帯登録時に携帯個体識別番号の登録を必須にするに設定
3)ログイン画面から新規登録。メールを飛ばし、招待状のURLから登録画面を開く
4)フォームに必要事項を入力し登録
5)エラー発生
Catchable fatal error: Argument 1 passed to MemberConfigMobileAddressForm::__construct() must be an instance of Member, boolean given, called in /home/xxx/openpne/apps/pc_frontend/modules/member/actions/actions.class.php on line 153 and defined in /home/xxx/openpne/lib/form/MemberConfigForm/MemberConfigMobileAddressForm.class.php on line 22
Catchable fatal error: Argument 1 passed to MemberConfigForm::__construct() must be an instance of Member, boolean given, called in /home/xxx/openpne/lib/form/MemberConfigForm/MemberConfigMobileAddressForm.class.php on line 24 and defined in /home/xxx/openpne/lib/form/doctrine/MemberConfigForm.class.php on line 26
Fatal error: Call to a member function getIsActive() on a non-object in /home/xxx/openpne/lib/form/doctrine/MemberConfigForm.class.php on line 35

opAuthMailAddressPluginのバグかコアのバグか判断出来なかったのでこちらのチケットを書きました


Related issues

Related to OpenPNE 3 - Bug(バグ) #1059: PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない Fixed(完了) 2010-03-30
Related to OpenPNE 3 - Backport(バックポート) #1067: URL in registering mail to mobile is wrong (携帯への新規登録メール内のURLが間違っている) Fixed(完了) 2010-02-03

Associated revisions

Revision 7c8d0e01 (diff)
Added by Minoru Takai over 12 years ago

(refs #919) fixed registration flow in case of require UID

Revision eb3feaf0 (diff)
Added by Minoru Takai over 12 years ago

(refs #919) fixed registration flow in case of require UID

Revision 6dc8ffbf (diff)
Added by Minoru Takai over 12 years ago

(refs #919, BP from #1059) changed template description in english to adapt i18n

History

#1 Updated by Kiwa Sakai over 12 years ago

  • Status changed from New(新規) to Pending Fixing(修正待ち)
  • Priority changed from Normal(通常) to Urgent(急いで)

OpenPNE3.5.2 にて上の手順による再現が行えました。

#2 Updated by Minoru Takai over 12 years ago

修正内容

  • apps/pc_frontend/modules/member/actions/actions.class.php
      public function executeRegisterMobileToRegisterEnd(sfWebRequest $request)
      {
        opActivateBehavior::disable();
        $this->form = new registerMobileForm($this->getUser()->getMember());
        opActivateBehavior::enable();
        if ($request->isMethod(sfWebRequest::POST))
        {
          $this->form->bind($request->getParameter('member_config'));
          if ($this->form->isValid())
          {
            $this->form->save();
            $this->redirect('member/registerMobileToRegisterEndFinish');
          }
        }
    
        return sfView::SUCCESS;
      }
    
      public function executeRegisterMobileToRegisterEndFinish(sfWebRequest $request)
      {
      }
    
    • 153行目($this->form = new ... の行)の前後に opActivateBehavior::disable() と opActivateBehavior::enable() を記述
    • redirect先を member/registerMobileToRegisterEndFinish に変更
    • executeRegisterMobileToRegisterEndFinish() を追加
  • apps/pc_frontend/modules/member/templates/registerMobileToRegisterEndFinishSuccess.php
    <?php op_include_box('finish', '携帯登録をおこなうことでメンバー登録が完了します。<br />入力された携帯メールアドレス宛てにメールを送信いたしましたので、メール内に記載されたURLから携帯登録をおこなってください。', array('title' => '携帯登録メール送信完了')); ?>
    
    • このテンプレートを新規に作成
  • apps/pc_frontend/modules/member/config/security.yml
    registerMobileToRegisterEndFinish:
      is_secure: false
    
    • security.yml の末に記述

#3 Updated by Shinichi Urabe over 12 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)
  • Assignee set to Minoru Takai
  • Target version set to OpenPNE 3.4.4
  • % Done changed from 0 to 50

#4 Updated by Shinichi Urabe over 12 years ago

  • Tracker changed from Bug(バグ) to Backport(バックポート)

#5 Updated by Rimpei Ogawa over 12 years ago

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

テンプレートは英語で書いてください。

#6 Updated by Minoru Takai over 12 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

#7 Updated by Minoru Takai over 12 years ago

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

#1059 で英語化も含めた修正を行いました。

Revision 6dc8ffbf2dbd80e025157db48c05525cccb9b84b#919 に対する英語化の修正を行ったので確認をお願いします。

#8 Updated by Rimpei Ogawa over 12 years ago

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

#9 Updated by Mutsumi Imamura over 12 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)

#568 (携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。

#10 Updated by Mutsumi Imamura over 12 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)

Imamura Mutsumi は書きました:

#568 (携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。

↑の#568 件とこのチケットの件について混乱していました。
このチケットはPC側の設定変更から携帯を登録した場合ですので、#568 の現象とは別件になります。
ただ、#568 と同じ現象は3.4.xでも再現しますのでバックポートチケットを作成し今後対応します。

#11 Updated by Shinichi Urabe over 12 years ago

Imamura Mutsumi は書きました:

Imamura Mutsumi は書きました:

#568 (携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。

↑の#568 件とこのチケットの件について混乱していました。
このチケットはPC側の設定変更から携帯を登録した場合ですので、#568 の現象とは別件になります。
ただ、#568 と同じ現象は3.4.xでも再現しますのでバックポートチケットを作成し今後対応します。

こちらについては次のバージョンで #1067 で対応しましょう

#12 Updated by Mutsumi Imamura over 12 years ago

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

確認しました。PC・携帯ともに登録完了しログインできることは確認できました。

  • #1069 PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない
  • #1071 PC・携帯登録時に携帯個体識別番号の登録を必須で新規登録すると登録中にdevモードだとWarningが発生する

上記2点の問題がありますが、別チケットで対応する方針にします。

#13 Updated by Shogo Kawahara over 12 years ago

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

Also available in: Atom PDF