Refactor

src/classes/user.php

@@ -12,6 +12,17 @@ namespace wishthis;
 
 class User
 {
+    /**
+     * Static
+     */
+    public static function generatePassword(string $plainPassword): string
+    {
+        return sha1($plainPassword);
+    }
+
+    /**
+     * Non-Static
+     */
     public function __construct(int $id = -1)
     {
         if (-1 === $id) {

src/pages/login.php

@@ -6,7 +6,7 @@
  * @author Jay Trees <github.jay@grandel.anonaddy.me>
  */
 
-use wishthis\{Page, Email};
+use wishthis\{Page, Email, User};
 
 $page = new Page(__FILE__, __('Login'));
 
@@ -15,7 +15,7 @@ $page = new Page(__FILE__, __('Login'));
  */
 if (isset($_POST['login'], $_POST['email'], $_POST['password'])) {
     $email    = $_POST['email'];
-    $password = sha1($_POST['password']);
+    $password = User::generatePassword($_POST['password']);
 
     $database->query('UPDATE `users`
                          SET `last_login` = NOW()

src/pages/register.php

@@ -6,7 +6,7 @@
  * @author Jay Trees <github.jay@grandel.anonaddy.me>
  */
 
-use wishthis\Page;
+use wishthis\{Page, User};
 
 $passwordReset = isset($_GET['password-reset'], $_GET['token']);
 
@@ -70,7 +70,7 @@ if (isset($_POST['email'], $_POST['password']) && !empty($_POST['planet'])) {
                 if (time() > $user['password_reset_valid_until']) {
                     $database
                     ->query('UPDATE `users`
-                                SET `password`                   = "' . sha1($_POST['password']) . '",
+                                SET `password`                   = "' . User::generatePassword($_POST['password']) . '",
                                     `password_reset_token`       = NULL,
                                     `password_reset_valid_until` = NULL
                               WHERE `id`                         = ' . $user['id'] . ';');
@@ -97,7 +97,7 @@ if (isset($_POST['email'], $_POST['password']) && !empty($_POST['planet'])) {
                                     `power`
                                 ) VALUES (
                                     "' . $_POST['email'] . '",
-                                    "' . sha1($_POST['password']) . '",
+                                    "' . User::generatePassword($_POST['password']) . '",
                                     100
                                 )
                 ;');
@@ -115,7 +115,7 @@ if (isset($_POST['email'], $_POST['password']) && !empty($_POST['planet'])) {
                                         `password`
                                     ) VALUES (
                                         "' . $_POST['email'] . '",
-                                        "' . sha1($_POST['password']) . '"
+                                        "' . User::generatePassword($_POST['password']) . '"
                                     )
                     ;');
                     $userRegistered = true;