<p dir="ltr">Thanks Miguel.</p>
<p dir="ltr">I'll see if the tests are something I can help out with, but I'm not sure what use I'll be...</p>
<p dir="ltr">Thanks,<br>
Martin </p>
<div class="gmail_quote">On 2 May 2014 21:39, "Miguel de Icaza" <<a href="mailto:miguel@xamarin.com">miguel@xamarin.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">I just did a quick review, and added a series of comments.<div><br></div><div>Among other things, it is missing tests :-)</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 24, 2014 at 9:51 AM, Martin Thwaites <span dir="ltr"><<a href="mailto:monoforum@my2cents.co.uk" target="_blank">monoforum@my2cents.co.uk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I've just found that Matthid on Github has done a far better implementation in pull 888 so I won't bother...<br>

<br></div>Who needs to look at it before it's merged?<br></div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On 23 April 2014 19:56, Martin Thwaites <span dir="ltr"><<a href="mailto:monoforum@my2cents.co.uk" target="_blank">monoforum@my2cents.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div dir="ltr"><div><div><div><div><div><div>First, I'm not asking for something to be implemented, I'm asking for help implementing it as I think there will be more and I'm endeavoring to implement them all if possible.<br>



</div><div><br>I'm having a play with trying to get the MVC5 boilerplate working on mono and one of the libraries (Microsoft.Owin) uses the new HttpApplication.RegisterModule method, which is not yet present in the current master branch.<br>



<br></div>Looking at the descriptions on MSDN, there is very little information on what it's supposed to do, but it looks like it just does the same as the method DynamicModuleHelper.RegisterModule... in that it just registers a module at runtime, in memory.<br>



<br></div>So, in light of the above, I have a few questions.<br></div>1. Would a pull be accepted if it just copies the implementation from the Microsoft.Web.Infrastructure class in the current mono codebase?  Given that the new method is in the HttpApplication class, I would say that it's a valid assumption that it's only Web modules that can be added.<br>



</div>2. If I'm just copying the implementation into System.Web, would it be advisable to edit the Microsoft.Web.Infrastructure class to just forward through to System.Web?<br></div>3. What, if any, tests should be created (I'm used to working in an environment with interfaces that could be mocked with Moq).  I could do with some information on how to do this, in terms of possibly some examples that are considered good, and locations in the current codebase.<br>



</div><div>4. Given that this is only available in .NET 4.5, is there some way I should stop it from being accessible on .NET 4.0 (if so, how)? I'm not sure there is an issue with leaving it available on .NET 4.0 other than the mono version has more methods than .NET...<br>



</div><div><br></div>References:<br><br><a href="http://msdn.microsoft.com/en-us/library/microsoft.web.infrastructure.dynamicmodulehelper.dynamicmoduleutility.registermodule%28v=vs.111%29.aspx" target="_blank">http://msdn.microsoft.com/en-us/library/microsoft.web.infrastructure.dynamicmodulehelper.dynamicmoduleutility.registermodule%28v=vs.111%29.aspx</a><br>



<a href="http://msdn.microsoft.com/en-us/library/system.web.httpapplication.registermodule%28v=vs.110%29.aspx" target="_blank">http://msdn.microsoft.com/en-us/library/system.web.httpapplication.registermodule%28v=vs.110%29.aspx</a><br>


</div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br></div>
</blockquote></div>