diff options
| author | Marc Alexander <admin@m-a-styles.de> | 2014-02-02 14:09:09 +0100 |
|---|---|---|
| committer | Marc Alexander <admin@m-a-styles.de> | 2014-02-02 14:09:09 +0100 |
| commit | 292961a2771c9237197b9770b23a2a14a981c329 (patch) | |
| tree | 5c9eee5425d2dd1b7c85648d08098d1ec729bdd5 /phpBB | |
| parent | 85f1e8afa55356f620a8715ba9fcaaec491dff96 (diff) | |
| download | forums-292961a2771c9237197b9770b23a2a14a981c329.tar forums-292961a2771c9237197b9770b23a2a14a981c329.tar.gz forums-292961a2771c9237197b9770b23a2a14a981c329.tar.bz2 forums-292961a2771c9237197b9770b23a2a14a981c329.tar.xz forums-292961a2771c9237197b9770b23a2a14a981c329.zip | |
[feature/passwords] Get rid of set_name/get_name methods for passwords drivers
PHPBB3-11610
Diffstat (limited to 'phpBB')
| -rw-r--r-- | phpBB/config/passwords.yml | 8 | ||||
| -rw-r--r-- | phpBB/phpbb/passwords/driver/base.php | 18 | ||||
| -rw-r--r-- | phpBB/phpbb/passwords/driver/driver_interface.php | 7 | ||||
| -rw-r--r-- | phpBB/phpbb/passwords/manager.php | 20 |
4 files changed, 8 insertions, 45 deletions
diff --git a/phpBB/config/passwords.yml b/phpBB/config/passwords.yml index 4eeca4e34f..9e249a2c12 100644 --- a/phpBB/config/passwords.yml +++ b/phpBB/config/passwords.yml @@ -11,8 +11,6 @@ services: arguments: - @config - @passwords.driver_helper - calls: - - [set_name, [passwords.driver.bcrypt]] tags: - { name: passwords.driver } @@ -21,8 +19,6 @@ services: arguments: - @config - @passwords.driver_helper - calls: - - [set_name, [passwords.driver.bcrypt_2y]] tags: - { name: passwords.driver } @@ -31,8 +27,6 @@ services: arguments: - @config - @passwords.driver_helper - calls: - - [set_name, [passwords.driver.salted_md5]] tags: - { name: passwords.driver } @@ -41,8 +35,6 @@ services: arguments: - @config - @passwords.driver_helper - calls: - - [set_name, [passwords.driver.phpass]] tags: - { name: passwords.driver } diff --git a/phpBB/phpbb/passwords/driver/base.php b/phpBB/phpbb/passwords/driver/base.php index 91c07a813b..8256fd721c 100644 --- a/phpBB/phpbb/passwords/driver/base.php +++ b/phpBB/phpbb/passwords/driver/base.php @@ -42,22 +42,4 @@ abstract class base implements driver_interface { return true; } - - /** - * @inheritdoc - */ - public function get_name() - { - return $this->name; - } - - /** - * Set driver name - * - * @param string $name Driver name - */ - public function set_name($name) - { - $this->name = $name; - } } diff --git a/phpBB/phpbb/passwords/driver/driver_interface.php b/phpBB/phpbb/passwords/driver/driver_interface.php index 53065dfb02..ebaf0626af 100644 --- a/phpBB/phpbb/passwords/driver/driver_interface.php +++ b/phpBB/phpbb/passwords/driver/driver_interface.php @@ -57,11 +57,4 @@ interface driver_interface * @return string String containing the hash settings */ public function get_settings_only($hash, $full = false); - - /** - * Get the driver name - * - * @return string Driver name - */ - public function get_name(); } diff --git a/phpBB/phpbb/passwords/manager.php b/phpBB/phpbb/passwords/manager.php index 539fdca954..7aeb496eb5 100644 --- a/phpBB/phpbb/passwords/manager.php +++ b/phpBB/phpbb/passwords/manager.php @@ -79,7 +79,7 @@ class manager { if ($this->algorithms[$type]->is_supported()) { - $this->type = $type; + $this->type = $this->algorithms[$type]->get_prefix(); break; } } @@ -94,16 +94,12 @@ class manager { foreach ($hashing_algorithms as $algorithm) { - if (!isset($this->algorithms[$algorithm->get_name()])) - { - $this->algorithms[$algorithm->get_name()] = $algorithm; - } - if (!isset($this->type_map[$algorithm->get_prefix()])) { - $this->type_map[$algorithm->get_prefix()] = $algorithm->get_name(); + $this->type_map[$algorithm->get_prefix()] = $algorithm; } } + $this->algorithms = $hashing_algorithms; } /** @@ -130,9 +126,9 @@ class manager */ protected function get_algorithm($prefix) { - if (isset($this->type_map[$prefix]) && isset($this->algorithms[$this->type_map[$prefix]])) + if (isset($this->type_map[$prefix])) { - return $this->algorithms[$this->type_map[$prefix]]; + return $this->type_map[$prefix]; } else { @@ -216,9 +212,9 @@ class manager return $this->helper->combined_hash_password($password, $type); } - if (isset($this->algorithms[$type])) + if (isset($this->type_map[$type])) { - $hashing_algorithm = $this->algorithms[$type]; + $hashing_algorithm = $this->type_map[$type]; } else { @@ -260,7 +256,7 @@ class manager return $correct; } - if ($stored_hash_type->get_name() !== $this->type) + if ($stored_hash_type->get_prefix() !== $this->type) { $this->convert_flag = true; } |