Ensure Association::$_className is always set.

cakephp · Jul 24, 2019 · f550426 · f550426
1 parent 12c6a3c
commit f550426
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 3 deletions.
diff --git a/src/ORM/Association.php b/src/ORM/Association.php
@@ -226,7 +226,7 @@ public function __construct(string $alias, array $options = [])
             }
         }
 
-        if (empty($this->_className) && strpos($alias, '.')) {
+        if (empty($this->_className)) {
             $this->_className = $alias;
         }
 
@@ -378,7 +378,7 @@ public function setTarget(Table $table)
     public function getTarget(): Table
     {
         if ($this->_targetTable === null) {
-            if (strpos((string)$this->_className, '.')) {
+            if (strpos($this->_className, '.')) {
                 [$plugin] = pluginSplit($this->_className, true);
                 $registryAlias = $plugin . $this->_name;
             } else {

diff --git a/tests/TestCase/ORM/AssociationTest.php b/tests/TestCase/ORM/AssociationTest.php
@@ -17,6 +17,7 @@
 namespace Cake\Test\TestCase\ORM;
 
 use Cake\Core\Configure;
+use Cake\ORM\Association;
 use Cake\ORM\Table;
 use Cake\TestSuite\TestCase;
 use TestApp\Model\Table\AuthorsTable;
@@ -49,7 +50,7 @@ public function setUp(): void
             'sourceTable' => $this->source,
             'joinType' => 'INNER',
         ];
-        $this->association = $this->getMockBuilder('Cake\ORM\Association')
+        $this->association = $this->getMockBuilder(Association::class)
             ->setMethods([
                 '_options', 'attachTo', '_joinCondition', 'cascadeDelete', 'isOwningSide',
                 'saveAssociated', 'eagerLoader', 'type', 'requiresKeys',
@@ -130,6 +131,22 @@ public function testSetNameToTargetAlias()
         $this->assertEquals($alias, $this->association->getName());
     }
 
+    /**
+     * Test that _className property is set to alias when "className" config
+     * if not explicitly set.
+     *
+     * @return void
+     */
+    public function testSetttingClassNameFromAlias()
+    {
+        $association = $this->getMockBuilder(Association::class)
+            ->setMethods(['type', 'eagerLoader', 'cascadeDelete', 'isOwningSide', 'saveAssociated'])
+            ->setConstructorArgs(['Foo'])
+            ->getMock();
+
+        $this->assertSame('Foo', $association->getClassName());
+    }
+
     /**
      * Tests that setClassName() succeeds before the target table is resolved.
      *