プロジェクト

全般

プロフィール

Backport(バックポート) #919

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

Naoto Inoue約14年前に追加. 13年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Urgent(急いで)
担当者:
対象バージョン:
開始日:
2010-03-30
期日:
進捗率:

100%


説明

バグ?と思われる物を見つけたので報告
バージョン: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のバグかコアのバグか判断出来なかったのでこちらのチケットを書きました


関連するチケット

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

関係しているリビジョン

リビジョン 7c8d0e01 (差分)
Minoru Takaiほぼ14年前に追加

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

リビジョン eb3feaf0 (差分)
Minoru Takaiほぼ14年前に追加

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

リビジョン 6dc8ffbf (差分)
Minoru Takaiほぼ14年前に追加

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

履歴

#1 Kiwa Sakaiほぼ14年前に更新

  • ステータスNew(新規) から Pending Fixing(修正待ち) に変更
  • 優先度Normal(通常) から Urgent(急いで) に変更

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

#2 Minoru Takaiほぼ14年前に更新

修正内容

  • 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 Shinichi Urabeほぼ14年前に更新

  • ステータスPending Fixing(修正待ち) から Pending Review(レビュー待ち) に変更
  • 担当者Minoru Takai にセット
  • 対象バージョンOpenPNE 3.4.4 にセット
  • 進捗率0 から 50 に変更

#4 Shinichi Urabeほぼ14年前に更新

  • トラッカーBug(バグ) から Backport(バックポート) に変更

#5 Rimpei Ogawaほぼ14年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

#6 Minoru Takaiほぼ14年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

#7 Minoru Takaiほぼ14年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

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

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

#8 Rimpei Ogawaほぼ14年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更

#9 Mutsumi Imamuraほぼ14年前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更

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

#10 Mutsumi Imamuraほぼ14年前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更

Imamura Mutsumi は書きました:

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

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

#11 Shinichi Urabeほぼ14年前に更新

Imamura Mutsumi は書きました:

Imamura Mutsumi は書きました:

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

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

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

#12 Mutsumi Imamuraほぼ14年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率50 から 100 に変更

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

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

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

#13 Shogo Kawahara13年以上前に更新

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

他の形式にエクスポート: Atom PDF