adrenalinkin / doctrine-naming-strategy Goto Github PK

Currently we have

    CREATE TABLE UserReadComments (
        UserId VARCHAR(255) NOT NULL,
        CommentId VARCHAR(255) NOT NULL,
        PRIMARY KEY(UserId, CommentId)
);

Should be UserId -> userId

    CREATE TABLE UserReadComments (
        userId VARCHAR(255) NOT NULL,
        commentId VARCHAR(255) NOT NULL,
        PRIMARY KEY(userId, commentId)
);

Add back compatibility and deprecate old format. Remove in 2.0.0

Doctrine/orm 3 released on february 3 (https://github.com/doctrine/orm/releases/tag/3.0.0). Is it possible to update the constraint in composer.json to allow for this new version?

Problem

Symfony profiler shows the following deprecation message:

Method "Doctrine\ORM\Mapping\NamingStrategy::joinKeyColumnName()" might add "string" as a native return type declaration in the future.
Do the same in implementation "Linkin\Component\DoctrineNamingStrategy\ORM\Mapping\CamelCaseNamingStrategy" now to avoid errors or add an explicit @return annotation to suppress this message.

Solution

Add string return type.

Versions
Symfony: 6.1.x
adrenalinkin/doctrine-naming-strategy: v1.1.0

Use this image in the documentation - https://user-images.githubusercontent.com/4967813/183507072-c05d17ae-4742-4602-958d-fc8676b6e9d6.png