http://redmine.openpne.jp/
http://redmine.openpne.jp/favicon.ico
2011-12-06T09:35:16Z
OpenPNE Issue Tracking System
opAuthOpenIDPlugin - Bug(バグ) #2599: メンバー登録画面からOpenIDの認証をおこなうと、プロバイダの認証後にプロフィール登録画面ではなくメンバー登録画面に遷移してしまう
http://redmine.openpne.jp/issues/2599?journal_id=12301
2011-12-06T09:35:16Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>担当者</strong> を <i>Yuya Watanabe</i> にセット</li><li><strong>対象バージョン</strong> を <i>1.3.1</i> にセット</li></ul>
opAuthOpenIDPlugin - Bug(バグ) #2599: メンバー登録画面からOpenIDの認証をおこなうと、プロバイダの認証後にプロフィール登録画面ではなくメンバー登録画面に遷移してしまう
http://redmine.openpne.jp/issues/2599?journal_id=12315
2011-12-06T12:41:48Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>New(新規)</i> から <i>Accepted(着手)</i> に変更</li></ul><a name="原因"></a>
<h3 >原因<a href="#原因" class="wiki-anchor">¶</a></h3>
<p>下記コード部で20行目や21行目では登録開始あるいは終了かどうかでリダイレクトを行なっているが,13行目でlogin()処理を行なっている.これは新規登録時だとloginできない(登録されていないため)ので20行目および21行目に到達することができず,プロフィール登録ができない状態となっている.</p>
<p>apps/pc_frontend/modules/OpenID/actions/registerOpenIDConfirmAction.class.php<br /><pre>
11 class RegisterOpenIDConfirmAction extends sfAction
12 {
13 public function execute($request)
14 {
15 $this->forward404Unless($request->hasParameter('authMode'));
16 $this->getUser()->setCurrentAuthMode('OpenID');
17
18 if ($uri = $this->getUser()->login())
19 {
20 $this->redirectIf($this->getUser()->isRegisterBegin(), $this->getUser()->getRegisterInputAction());
21 $this->redirectIf($this->getUser()->isRegisterFinish(), $this->getUser()->getRegisterEndAction());
22 $this->redirectIf($this->getUser()->isMember(), $uri);
23 }
24
25 $this->getUser()->setFlash('error', 'Invalid.');
26 $this->redirect('member/register?token='.$request['token']);
27 }
28 }
</pre></p>
<a name="修正案"></a>
<h3 >修正案<a href="#修正案" class="wiki-anchor">¶</a></h3>
<p>login処理を行う前に新規登録状態かどうかのリダイレクト処理を行う.</p>
<pre>
diff --git a/apps/pc_frontend/modules/OpenID/actions/registerOpenIDConfirmAction.class.php b/apps/pc_frontend/modules/OpenID/actions/registerOpenIDConfirmAction.class.php
index 3f99c6c..4003f86 100644
--- a/apps/pc_frontend/modules/OpenID/actions/registerOpenIDConfirmAction.class.php
+++ b/apps/pc_frontend/modules/OpenID/actions/registerOpenIDConfirmAction.class.php
@@ -15,10 +15,10 @@ class RegisterOpenIDConfirmAction extends sfAction
$this->forward404Unless($request->hasParameter('authMode'));
$this->getUser()->setCurrentAuthMode('OpenID');
+ $this->redirectIf($this->getUser()->isRegisterBegin(), $this->getUser()->getRegisterInputAction());
+ $this->redirectIf($this->getUser()->isRegisterFinish(), $this->getUser()->getRegisterEndAction());
if ($uri = $this->getUser()->login())
{
- $this->redirectIf($this->getUser()->isRegisterBegin(), $this->getUser()->getRegisterInputAction());
- $this->redirectIf($this->getUser()->isRegisterFinish(), $this->getUser()->getRegisterEndAction());
$this->redirectIf($this->getUser()->isMember(), $uri);
}
</pre>
opAuthOpenIDPlugin - Bug(バグ) #2599: メンバー登録画面からOpenIDの認証をおこなうと、プロバイダの認証後にプロフィール登録画面ではなくメンバー登録画面に遷移してしまう
http://redmine.openpne.jp/issues/2599?journal_id=12316
2011-12-06T12:47:55Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>0</i> から <i>50</i> に変更</li></ul><p>下記コミットで適用しました.</p>
<p><a class="external" href="https://github.com/ebihara/opAuthOpenIDPlugin/commit/b05e3b607aaa69d8484ba8400b16b76af6a1e72d">https://github.com/ebihara/opAuthOpenIDPlugin/commit/b05e3b607aaa69d8484ba8400b16b76af6a1e72d</a></p>
opAuthOpenIDPlugin - Bug(バグ) #2599: メンバー登録画面からOpenIDの認証をおこなうと、プロバイダの認証後にプロフィール登録画面ではなくメンバー登録画面に遷移してしまう
http://redmine.openpne.jp/issues/2599?journal_id=12337
2011-12-07T08:25:32Z
Kousuke Ebihara
ebihara@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Pending Testing(テスト待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>70</i> に変更</li></ul>
opAuthOpenIDPlugin - Bug(バグ) #2599: メンバー登録画面からOpenIDの認証をおこなうと、プロバイダの認証後にプロフィール登録画面ではなくメンバー登録画面に遷移してしまう
http://redmine.openpne.jp/issues/2599?journal_id=12342
2011-12-08T01:57:26Z
Yuma Sakata
sakata@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Pending Testing(テスト待ち)</i> から <i>Fixed(完了)</i> に変更</li><li><strong>進捗率</strong> を <i>70</i> から <i>100</i> に変更</li></ul><p>テストOKです。</p>