Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BetterPhpDocParser] Add support for parsing Doctrine annotations #1902

Merged
merged 3 commits into from
Aug 26, 2019

Conversation

TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Aug 26, 2019

1) Rector\DoctrinePhpDocParser\Ast\PhpDoc\EntityTagValueNode

<?php

use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity(readOnly=true, repositoryClass="Rector\DoctrinePhpDocParser\Tests\PhpDocParser\OrmTagParser\Class_\Source\ExistingRepositoryClass")
 */
final class SomeEntity
{

}

2) Rector\DoctrinePhpDocParser\Ast\PhpDoc\ColumnTagValueNode

<?php

use Doctrine\ORM\Mapping as ORM;

final class FromOfficialDocs
{
    /**
     * @ORM\Column(type="integer", name="login_count", nullable=false, options={"unsigned":true, "default":0}, columnDefinition="CHAR(2) NOT NULL")
     */
    private $loginCount;
}

@TomasVotruba TomasVotruba force-pushed the parse-and-change-doctrine-annotations branch 2 times, most recently from 4f06f5e to 166ae2e Compare August 26, 2019 16:53
@TomasVotruba TomasVotruba force-pushed the parse-and-change-doctrine-annotations branch from 6d5d735 to 3fff6b1 Compare August 26, 2019 19:52
@TomasVotruba TomasVotruba force-pushed the parse-and-change-doctrine-annotations branch from 3fff6b1 to faaf605 Compare August 26, 2019 19:58
@TomasVotruba TomasVotruba merged commit 073e7cc into master Aug 26, 2019
@TomasVotruba TomasVotruba deleted the parse-and-change-doctrine-annotations branch August 26, 2019 20:38
TomasVotruba added a commit that referenced this pull request Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant