Skip to content

[Php81] Exclude Doctrine ODM MongoDB 2.16 Document and EmbeddedDocument from ReadOnlyPropertyRector. - #7848

Merged
TomasVotruba merged 1 commit into
rectorphp:mainfrom
mickverm:readonlypropertyrector-doctrine-mongodb-odm-2.16
Jan 21, 2026
Merged

[Php81] Exclude Doctrine ODM MongoDB 2.16 Document and EmbeddedDocument from ReadOnlyPropertyRector.#7848
TomasVotruba merged 1 commit into
rectorphp:mainfrom
mickverm:readonlypropertyrector-doctrine-mongodb-odm-2.16

Conversation

@mickverm

@mickverm mickverm commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

The namespace was changed in doctrine/mongodb-odm 2.16, see: doctrine/mongodb-odm#2933

Ref: #6721

Should a rule be added to automatically replace the attribute?

@samsonasik

samsonasik commented Jan 21, 2026

Copy link
Copy Markdown
Member

Could you rebase latest main branch? We currently pin to phpstan 2.1.34 on main branch as phpstan 2.1.35 cause issue.

We don't have ODM set on doctrine, feel free to create rule/upgrade config there and register to config set

https://github.com/rectorphp/rector-doctrine/tree/main/config/sets

Thank you.

@mickverm
mickverm force-pushed the readonlypropertyrector-doctrine-mongodb-odm-2.16 branch from 16085f0 to 64fff92 Compare January 21, 2026 12:13
@mickverm

Copy link
Copy Markdown
Contributor Author

I've rebased it.

I'll see what I can do once I have some spare time.

@TomasVotruba

Copy link
Copy Markdown
Member

Looks good, thank you 👍

@TomasVotruba
TomasVotruba merged commit 581b0cb into rectorphp:main Jan 21, 2026
57 checks passed
@mickverm
mickverm deleted the readonlypropertyrector-doctrine-mongodb-odm-2.16 branch January 21, 2026 13:24
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants