Changeset 21584 for extensions/oAuth/include/public_events.inc.php

Timestamp:

Mar 15, 2013, 6:43:07 PM (11 years ago)

Author:

mistic100

Message:

don't delete keys when the provider is removed, hide fieldset when no provider active, use jQuery instead of $

File:

• extensions/oAuth/include/public_events.inc.php (modified) (6 diffs)

extensions/oAuth/include/public_events.inc.php

@@ -61 +61 @@
       $adapter = $hybridauth->authenticate($provider);
       $remote_user = $adapter->getUserProfile();
+      
+      // security, check remote identifier
+      if ($remote_user->identifier != $user_identifier)
+      {
+        pwg_unset_session_var('oauth_new_user');
+        throw new Exception('Hacking attempt!');
+      }
     
       $template->assign(array(
@@ -73 +80 @@
       
       $oauth_id = $provider.'---'.$remote_user->identifier;
-      
-      // security, check remote identifier
-      if ($remote_user->identifier != $user_identifier)
-      {
-        pwg_unset_session_var('oauth_new_user');
-        throw new Exception('Hacking attempt!');
-      }
       
       // form submited
@@ -96 +96 @@
           $user_id = get_userid($_POST['login']);
           
-          // udpdate oauth field
+          // update oauth field
           $query = '
 UPDATE '.USERS_TABLE.'
@@ -235 +235 @@
   $template->assign('REDIRECT_TO', get_gallery_home_url());
   
-  $template->set_prefilter('menubar', 'oauth_add_menubar_buttons');
+  $template->set_prefilter('menubar', 'oauth_add_menubar_buttons_prefilter');
 }
 
@@ -254 +254 @@
   $add = 'disabled="disabled" ';
   $script = '
-{footer_script}{literal}
+{footer_script}
 jQuery("input[type=\'password\'], input[name=\'send_password_by_mail\']").parent("li").css("display", "none");
-{/literal}{/footer_script}';
+{/footer_script}';
 
   $content = str_replace($search, $search.$add, $content);
@@ -269 +269 @@
 }
 
-function oauth_add_menubar_buttons($content)
+function oauth_add_menubar_buttons_prefilter($content)
 {
   $search = '{include file=$block->template|@get_extent:$id }';