fix: improve performance and robustness of Relation::getMorphAlias()

laravel · Jun 19, 2024 · 797ee4f · 797ee4f
1 parent 9b9e862
commit 797ee4f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Illuminate/Database/Eloquent/Relations/Relation.php b/src/Illuminate/Database/Eloquent/Relations/Relation.php
@@ -509,7 +509,7 @@ public static function getMorphedModel($alias)
      */
     public static function getMorphAlias(string $className)
     {
-        return array_flip(static::$morphMap)[$className] ?? null;
+        return array_search($className, static::$morphMap, strict: true) ?: $className;
     }
 
     /**

diff --git a/tests/Database/DatabaseEloquentRelationTest.php b/tests/Database/DatabaseEloquentRelationTest.php
@@ -239,8 +239,8 @@ public function testGetMorphAlias()
     {
         Relation::morphMap(['user' => 'App\User']);
 
-        $this->assertEquals('user', Relation::getMorphAlias('App\User'));
-        $this->assertNull(Relation::getMorphAlias('Does\Not\Exist'));
+        $this->assertSame('user', Relation::getMorphAlias('App\User'));
+        $this->assertSame('Does\Not\Exist', Relation::getMorphAlias('Does\Not\Exist'));
     }
 
     public function testWithoutRelations()
-Original file line number
+Diff line change
@@ Expand Up / @@ -509,7 +509,7 @@ public static function getMorphedModel($alias) @@
          */
         public static function getMorphAlias(string $className)
         {
-            return array_flip(static::$morphMap)[$className] ?? null;
+            return array_search($className, static::$morphMap, strict: true) ?: $className;
         }
         /**
@@ Expand Down @@