public/index.php

@@ -12,11 +12,6 @@ ini_set(option: 'display_startup_errors', value: 1);
 // no one sane should ignore deprecations
 error_reporting(error_level: E_ALL);
 
-// just during dev
-opcache_reset();
-
-
-
 session_start();
 
 require dirname(path: __DIR__) . '/src/bootstrap.php';

src/Controller/SecurityController.php

@@ -32,7 +32,6 @@ class SecurityController
             $password = $_POST['password'] ?? '';
 
             if ($nick && $password) {
-                $nick = strtolower(string: $nick);
                 if ($user = $this->userRepository->findbyNick(nick: $nick)) {
                     if (password_verify(password: $password, hash: $user->getPassword())) {
                         $_SESSION['user_id'] = $user->getId();

src/Repository/AddressRepository.php

@@ -105,6 +105,7 @@ class AddressRepository
     public function update(AddressBookEntry $address): bool|int
     {
         $id = $address->getId();
+        $owner = $address->getOwner();
         $first = $address->getFirst();
         $last = $address->getLast();
         $street = $address->getStreet();

src/Repository/UserRepository.php

@@ -71,8 +71,6 @@ class UserRepository
 
     public function findByNick(string $nick): ?User
     {
-        $nick = strtolower(string: $nick);
-
         $sql = "
             SELECT id, nick, password, first, last, is_admin
             FROM " . DatabaseConnection::TABLE_USERS . "

src/Service/Config.php

@@ -9,6 +9,8 @@
 
 namespace App\Service;
 
+use Exception;
+
 /**
  *
  */

src/Service/Router.php

@@ -84,7 +84,7 @@ class Router
 
         foreach ($this->dynamicRoutes as $route) {
             // PHPStorm doesn't know that $parameters are always available,
-            // (as it this are dynamic routes) so I init the array just to make PHPStorm happy.
+            // (as it is a dynamic route) so I init the array just to make PHPStorm happy.
             $parameters = [];
             if (preg_match(pattern: $route->getRegex(), subject: $requestUri, matches: $matches)) {
                 foreach ($route->getParameters() as $id => $parameter) {

src/Service/Template.php

@@ -13,6 +13,8 @@ namespace App\Service;
  * using PHP as a templating engine.
  */
 
+use JetBrains\PhpStorm\NoReturn;
+
 class Template
 {
     /*
@@ -26,7 +28,8 @@ class Template
     /*
      * Add variables to template and throw it out
      */
-    public function render(string $templateName, array $vars = []): never
+    #[NoReturn]
+    public function render(string $templateName, array $vars = []): void
     {
         // assign template vars
         foreach ($vars as $name => $value) {