doctrine / orm

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2182Test.php

<?php

declare(strict_types=1);

namespace Doctrine\Tests\ORM\Functional\Ticket;

use Doctrine\ORM\Annotation as ORM;
use Doctrine\Tests\OrmFunctionalTestCase;

class DDC2182Test extends OrmFunctionalTestCase
{
    public function testPassColumnOptionsToJoinColumns()
    {
        if ($this->em->getConnection()->getDatabasePlatform()->getName() != 'mysql') {
            $this->markTestSkipped("This test is useful for all databases, but designed only for mysql.");
        }

        $sql = $this->schemaTool->getCreateSchemaSql(
            [
            $this->em->getClassMetadata(DDC2182OptionParent::class),
            $this->em->getClassMetadata(DDC2182OptionChild::class),
            ]
        );

        self::assertEquals("CREATE TABLE DDC2182OptionParent (id INT UNSIGNED NOT NULL, PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB", $sql[0]);
        self::assertEquals("CREATE TABLE DDC2182OptionChild (id VARCHAR(255) NOT NULL, parent_id INT UNSIGNED DEFAULT NULL, INDEX IDX_B314D4AD727ACA70 (parent_id), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB", $sql[1]);
        self::assertEquals("ALTER TABLE DDC2182OptionChild ADD CONSTRAINT FK_B314D4AD727ACA70 FOREIGN KEY (parent_id) REFERENCES DDC2182OptionParent (id)", $sql[2]);
    }
}

/**
 * @ORM\Entity
 * @ORM\Table
 */
class DDC2182OptionParent
{
    /** @ORM\Id @ORM\Column(type="integer", options={"unsigned": true}) */
    private $id;

The private property $id is not used, and could be removed.

}

/**
 * @ORM\Entity
 * @ORM\Table
 */
class DDC2182OptionChild
{
    /** @ORM\Id @ORM\Column */
    private $id;

    /**
     * @ORM\ManyToOne(targetEntity=DDC2182OptionParent::class)
     * @ORM\JoinColumn(referencedColumnName="id")
     */
    private $parent;

The private property $parent is not used, and could be removed.

}