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

CastleDynamicProxyFactory_HasItem_true_when_array_is_null #683

Conversation

phongphanq
Copy link

The method HasItem used to check the constructorArguments is incorrect when passing null object.

image

@phongphanq phongphanq force-pushed the CastleDynamicProxyFactory_HasItem_true_when_array_is_null branch from e142dd4 to 7ff4994 Compare March 4, 2022 02:28
@rcdailey
Copy link

Can someone merge this please?

@dtchepak dtchepak merged commit 8d00402 into nsubstitute:main Jul 9, 2022
@dtchepak
Copy link
Member

dtchepak commented Jul 9, 2022

Thanks for this! 🙇

@rcdailey
Copy link

rcdailey commented Jul 9, 2022

Thank you @dtchepak. When can I expect this to hit a nuget release?

@dtchepak
Copy link
Member

@rcdailey Just published 4.4.0. Could you please confirm this issue is resolved?

@rcdailey
Copy link

@dtchepak Sorry for the delay. I responded in another issue, but I'm not able to reproduce the issue anymore after upgrading to v4.4.0. Thank you for the fix!!

rcdailey added a commit to recyclarr/recyclarr that referenced this pull request Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants