Merge pull request #26 from jozefrebjak/develop

fix: 🐛 is_int -> is_numeric in Visit Entity

Author: MGatner

src/Entities/Visit.php

@@ -52,7 +52,7 @@ public function setIpAddress($ipAddress): void
      */
     public function getIpAddress(): ?string
     {
-        if (is_int($this->attributes['ip_address'])) {
+        if (is_numeric($this->attributes['ip_address'])) {
             return long2ip($this->attributes['ip_address']);
         }
 

tests/EntityTest.php

@@ -37,4 +37,20 @@ public function ipAddressProvider(): array
             [null, null, null],
         ];
     }
+
+    public function testIpAddressHandlesStrings(): void
+    {
+        $visit = new Visit();
+        $visit->setAttributes([
+            'ip_address' => '2130706433',
+        ]);
+
+        $result = $visit->getIpAddress();
+        $this->assertSame('127.0.0.1', $result);
+
+        $visit->setAttributes([
+            'ip_address' => 'foo',
+        ]);
+        $this->assertNull($visit->getIpAddress());
+    }
 }