Skip to content

fix bug for parameter reflection #33

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

fix bug for parameter reflection #33

wants to merge 1 commit into from

Conversation

@motia
Copy link

@motia motia commented Aug 21, 2016

parameters defaulted to null did not recieve recieve their value #32
@motia
fix bug for parameter reflection
f439802 
parameters defaulted to null were skipped
@motia motia mentioned this pull request Aug 21, 2016
Closed
@gossi
Copy link
Member

gossi commented Aug 22, 2016

Unfortunately your code is wrong. Because a null default for a parameter should be an expression instead of a value, also see the failing tests. So problem is somewhere else. I don't know where exactly but I'm aware of this. See my post in #32 I'm going to address this.
@gossi gossi closed this Aug 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@motia @gossi