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

False positive for ramsey/uuid-doctrine and beberlei/doctrineextensions #74

Open
VincentLanglet opened this issue Jan 24, 2020 · 2 comments
Labels
duplicate This issue or pull request already exists

Comments

@VincentLanglet
Copy link
Contributor

Describe the bug

The https://github.com/ramsey/uuid-doctrine and https://github.com/beberlei/doctrineextensions is reported as unused in my symfony project.

But I do used them. This way

doctrine:
    dbal:
        types:
            uuid: Ramsey\Uuid\Doctrine\UuidType
    orm:
        dql:
            datetime_functions:
                DATEDIFF: DoctrineExtensions\Query\Mysql\DateDiff
                DATE_FORMAT: DoctrineExtensions\Query\Mysql\DateFormat
                MONTH: DoctrineExtensions\Query\Mysql\Month
                NOW: DoctrineExtensions\Query\Mysql\Now
                SEC_TO_TIME: DoctrineExtensions\Query\Mysql\SecToTime
                TIMEDIFF: DoctrineExtensions\Query\Mysql\TimeDiff
                TIME_TO_SEC: DoctrineExtensions\Query\Mysql\TimeToSec
                WEEK: DoctrineExtensions\Query\Mysql\Week
                YEAR: DoctrineExtensions\Query\Mysql\Year
                YEARWEEK: DoctrineExtensions\Query\Mysql\YearWeek
            numeric_functions:
                FLOOR: DoctrineExtensions\Query\Mysql\Floor
                NULLIF: DoctrineExtensions\Query\Mysql\NullIf
            string_functions:
                SOUNDEX: DoctrineExtensions\Query\Mysql\Soundex

Can this project handle yaml config ?

@icanhazstring
Copy link
Member

Hi @VincentLanglet thanks for reaching out.

Sadly at the moment, no.
I am working on this with the next major release where I want to add some custom usage scanner (see #69). So I am currently working on this.

I will mark this as duplicate to #43 since this all should be solved with above mentioned implementation.

@icanhazstring icanhazstring added the duplicate This issue or pull request already exists label Jan 24, 2020
@icanhazstring
Copy link
Member

Also mentioned with #73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants