fix issue 35 #36

wants to merge 2 commits into


None yet

2 participants

jonasro commented Oct 29, 2013

This solves my problem, but is not a perfect fix since it does not preserve default values that are null.
Ideally I would probably instead revert the changes made in to make it more consistent:


I think it would be good to add another test that actually tries to call the method on the proxy, without specifying the parameter's value, to see if that works

@kkozmic kkozmic added a commit that closed this pull request Dec 1, 2013
@kkozmic kkozmic updated changes.txt
fixed #35
closes #36
@kkozmic kkozmic closed this in 9b06bb4 Dec 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment