Implements x00's proposed fix for OpenID using Session->Stash. closes v…

…anilla#907
unlight · Sep 16, 2011 · ff6510b · ff6510b
1 parent 200768f
commit ff6510b
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/plugins/OpenID/class.openid.plugin.php b/plugins/OpenID/class.openid.plugin.php
@@ -117,8 +117,14 @@ public function Base_ConnectData_Handler($Sender, $Args) {
          return; // this will error out
 
       $this->EventArguments = $Args;
-      $OpenID = $this->GetOpenID();
-      if ($OpenID->validate()) {
+
+      // Check session before retrieving
+      $Session = Gdn::Session();
+      $OpenID = $Session->Stash('OpenID', '', FALSE);
+      if (!$OpenID)
+         $OpenID = $this->GetOpenID();
+
+      if ($Session->Stash('OpenID', '', FALSE) || $OpenID->validate()) {
          $Attr = $OpenID->getAttributes();
 
          $Form = $Sender->Form; //new Gdn_Form();
@@ -129,6 +135,7 @@ public function Base_ConnectData_Handler($Sender, $Args) {
          $Form->SetFormValue('FullName', GetValue('namePerson/first', $Attr).' '.GetValue('namePerson/last', $Attr));
          $Form->SetFormValue('Email', GetValue('contact/email', $Attr));
          $Sender->SetData('Verified', TRUE);
+         $Session->Stash('OpenID', $OpenID);
       }
    }