Skip to content

Fix dynamic method exception when method is defined but not visible#334

Merged
mvorisek merged 5 commits intodevelopfrom
fix_dyn_method_ex
Nov 10, 2021
Merged

Fix dynamic method exception when method is defined but not visible#334
mvorisek merged 5 commits intodevelopfrom
fix_dyn_method_ex

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Nov 8, 2021

match PHP behaviour, see https://3v4l.org/EKcDHc

@mvorisek mvorisek requested a review from DarkSide666 November 8, 2021 13:23
Copy link
Copy Markdown
Member

@DarkSide666 DarkSide666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addHasMany / HasOne changes shouldn't be in this PR (see docs/factory.rst)

@mvorisek mvorisek merged commit 39df847 into develop Nov 10, 2021
@mvorisek mvorisek deleted the fix_dyn_method_ex branch November 10, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants