Optional Parameters from IronPython

Any idea as to why when I call a RestEase generated method from IronPython it doesn’t respect my optional query parameters?

        Task<MyThing> MyMethod(
            [Query] bool? param1 = null,
            [Query] string param2 = null,
            [Query] string param3 = null,
            [Query] string param4 = null
            );

It works fine from a standard C# client, but from my embedded IronPython client it tells me MyMethod() takes exactly 4 arguments (0 given)

IronPython generally respects optional parameters and if I pass null values in it works fine.

Is there some ‘magic’ happening in the code generation that means the parameters are not actually optional in the resulting method?

[canton7]

Mind trying this build? https://ci.appveyor.com/project/canton7/restease/build/1.4.5.309/artifacts

[canton7]

Ah, right. I might be able to do something about that: I’ll have a look.

On 28 June 2018 15:02:08 BST, Adrian Cockburn [e-mail] wrote:

That’ll explain it… Python doesn’t care about types or interfaces, as it’s fully dynamic or ‘duck typed’:

If it walks like a duck and it quacks like a duck, then it must be a duck

   public interface IMyThing
   {
       string DoStuffWithOrWithoutAParam(string param = null);
   }

   public class MyThing : IMyThing
   {
       public string DoStuffWithOrWithoutAParam(string param = null)
       {
return string.Format("Hello {0}", string.IsNullOrWhiteSpace(param) ?
"World" : param);
       }
   }

   public class MyBrokenThing : IMyThing
   {
       public string DoStuffWithOrWithoutAParam(string param)
       {
return string.Format("Hello {0}", string.IsNullOrWhiteSpace(param) ?
"World" : param);
       }
   }

So, the first implementation works from IronPython but the second one doesn’t. Strangely, I’ve been working with IronPyhton for about 6 years now and have never come across this before now.

– You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/canton7/RestEase/issues/98#issuecomment-401045543