Use everytime lowercase identifier
authorCyperghost <olaf_schmitz_1@t-online.de>
Tue, 27 Feb 2024 12:04:01 +0000 (13:04 +0100)
committerCyperghost <olaf_schmitz_1@t-online.de>
Tue, 27 Feb 2024 12:04:01 +0000 (13:04 +0100)
wcfsetup/install/files/lib/system/spider/Spider.class.php
wcfsetup/install/files/lib/system/spider/event/SpiderCollecting.class.php

index 3e20693c79764cf93da108ec351fcb63f3aca9e7..6a86469dd88e94fe52bcb9165ae1018192d15a08 100644 (file)
@@ -9,10 +9,13 @@ namespace wcf\system\spider;
  */
 final readonly class Spider
 {
+    public string $identifier;
+
     public function __construct(
-        public string $identifier,
+        string $identifier,
         public string $name,
         public ?string $url = null,
     ) {
+        $this->identifier = \mb_strtolower($identifier);
     }
 }
index d808d8d6348578a0940b3e740eae14e092d0024f..3da81d3fe8ff2e23944bc4827c2d2e98c2c17b58 100644 (file)
@@ -643,11 +643,10 @@ final class SpiderCollecting implements IEvent
      */
     public function register(Spider $spider): void
     {
-        $identifier = \mb_strtolower($spider->identifier);
-        if (\array_key_exists($identifier, $this->spiders)) {
-            throw new \InvalidArgumentException('Spider with identifier ' . $identifier . ' already exists');
+        if (\array_key_exists($spider->identifier, $this->spiders)) {
+            throw new \InvalidArgumentException('Spider with identifier ' . $spider->identifier . ' already exists');
         }
-        $this->spiders[$identifier] = $spider;
+        $this->spiders[$spider->identifier] = $spider;
     }
 
     /**