Remove unnecessary overriding of '_findUser()' in 'DigestAuthenticate'.

This commit is contained in:
ADmad 2013-05-08 04:10:13 +05:30
parent 7d06f7b304
commit 56fa0dccda

View file

@ -116,7 +116,11 @@ class DigestAuthenticate extends BasicAuthenticate {
if (empty($digest)) {
return false;
}
$user = $this->_findUser($digest['username']);
list(, $model) = pluginSplit($this->settings['userModel']);
$user = $this->_findUser(array(
$model . '.' . $this->settings['fields']['username'] => $digest['username']
));
if (empty($user)) {
return false;
}
@ -128,34 +132,6 @@ class DigestAuthenticate extends BasicAuthenticate {
return false;
}
/**
* Find a user record using the standard options.
*
* @param string $username The username/identifier.
* @param string $password Unused password, digest doesn't require passwords.
* @return Mixed Either false on failure, or an array of user data.
*/
protected function _findUser($username, $password = null) {
$userModel = $this->settings['userModel'];
list(, $model) = pluginSplit($userModel);
$fields = $this->settings['fields'];
$conditions = array(
$model . '.' . $fields['username'] => $username,
);
if (!empty($this->settings['scope'])) {
$conditions = array_merge($conditions, $this->settings['scope']);
}
$result = ClassRegistry::init($userModel)->find('first', array(
'conditions' => $conditions,
'recursive' => $this->settings['recursive']
));
if (empty($result) || empty($result[$model])) {
return false;
}
return $result[$model];
}
/**
* Gets the digest headers from the request/environment.
*