[PR #7163] Add constructor types to SqlWalker and TreeWalker #10346

Closed
opened 2026-01-22 16:07:15 +01:00 by admin · 0 comments
Owner

Original Pull Request: https://github.com/doctrine/orm/pull/7163

State: closed
Merged: Yes


I was trying to use the SqlWalker but had no idea what objects were required for passing in. It had an inherit doc annotation, so I found that in the interface.

Then I noticed that although the docblock said what each param was in the interface, it allows you to pass in any old rubbish, so I thought I'd fix that for you :-)

**Original Pull Request:** https://github.com/doctrine/orm/pull/7163 **State:** closed **Merged:** Yes --- I was trying to use the SqlWalker but had no idea what objects were required for passing in. It had an inherit doc annotation, so I found that in the interface. Then I noticed that although the docblock said what each param was in the interface, it allows you to pass in any old rubbish, so I thought I'd fix that for you :-)
admin added the pull-request label 2026-01-22 16:07:15 +01:00
admin closed this issue 2026-01-22 16:07:15 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#10346