aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorewinslow <ewinslow@36083f99-b078-4883-b0ff-0f9b5a30f544>2010-11-20 08:36:32 +0000
committerewinslow <ewinslow@36083f99-b078-4883-b0ff-0f9b5a30f544>2010-11-20 08:36:32 +0000
commit863c7044a223d101b29e630d048d0f53580f4096 (patch)
tree80fe3ad2c7487bcaf62bd641118edd632ea5157d
parent9f41e8b1a34044d654ce3c6381efe3b3e2030c35 (diff)
downloadelgg-863c7044a223d101b29e630d048d0f53580f4096.tar.gz
elgg-863c7044a223d101b29e630d048d0f53580f4096.tar.bz2
Fixes #2468: Deprecated call_gatekeeper() and callpath_gatekeeper()
git-svn-id: http://code.elgg.org/elgg/trunk@7369 36083f99-b078-4883-b0ff-0f9b5a30f544
-rw-r--r--engine/lib/elgglib.php10
-rw-r--r--engine/lib/users.php18
2 files changed, 15 insertions, 13 deletions
diff --git a/engine/lib/elgglib.php b/engine/lib/elgglib.php
index cf3d25822..fb7a4578b 100644
--- a/engine/lib/elgglib.php
+++ b/engine/lib/elgglib.php
@@ -1335,9 +1335,11 @@ function elgg_deprecated_notice($msg, $dep_version) {
* @param string $file Optional file that the function must reside in.
*
* @return bool
- * @todo This is neat but is it necessary?
+ *
+ * @deprecated 1.8 A neat but pointless function
*/
function call_gatekeeper($function, $file = "") {
+ elgg_deprecated_notice("call_gatekeeper() is neat but pointless", 1.8);
// Sanity check
if (!$function) {
return false;
@@ -1401,11 +1403,13 @@ function call_gatekeeper($function, $file = "") {
* called by something on $path, if false the whole call stack is
* searched.
*
- * @todo Again, very neat, but is it necessary?
- *
* @return void
+ *
+ * @deprecated 1.8 A neat but pointless function
*/
function callpath_gatekeeper($path, $include_subdirs = true, $strict_mode = false) {
+ elgg_deprecated_notice("callpath_gatekeeper() is neat but pointless", 1.8);
+
global $CONFIG;
$path = sanitise_string($path);
diff --git a/engine/lib/users.php b/engine/lib/users.php
index 62f2bdcc8..9ed3d0863 100644
--- a/engine/lib/users.php
+++ b/engine/lib/users.php
@@ -889,19 +889,17 @@ function send_new_password_request($user_guid) {
function force_user_password_reset($user_guid, $password) {
global $CONFIG;
- if (call_gatekeeper('execute_new_password_request', __FILE__)) {
- $user = get_entity($user_guid);
+ $user = get_entity($user_guid);
- if ($user) {
- $salt = generate_random_cleartext_password(); // Reset the salt
- $user->salt = $salt;
+ if ($user) {
+ $salt = generate_random_cleartext_password(); // Reset the salt
+ $user->salt = $salt;
- $hash = generate_user_password($user, $password);
+ $hash = generate_user_password($user, $password);
- $query = "UPDATE {$CONFIG->dbprefix}users_entity
- set password='$hash', salt='$salt' where guid=$user_guid";
- return update_data($query);
- }
+ $query = "UPDATE {$CONFIG->dbprefix}users_entity
+ set password='$hash', salt='$salt' where guid=$user_guid";
+ return update_data($query);
}
return false;