<div dir="ltr">><span style="font-family:arial,sans-serif;font-size:13px">Hrm, not sure if I like having the default values for the parameters inside of the routing definition.</span><div style><font face="arial, sans-serif">Would you prefer not having support for default parameters values, or would you like default parameter values but in a different form?</font></div>
<div style><font face="arial, sans-serif"><br></font></div><div style>In the example above we are passing query parameters and might not make sense to have default values in this case, but these parameters could also be header, form, or cookie parameters. Not sure if this makes a different but I'm thinking that there might be situations where one might consider a header parameter or a cookie parameter as optional and would like a default value for it. Without defaults, we would need to have overloaded target methods as the current implementation needs an exact match for arguments -> parameters (but I'm sure we could look into different solutions).</div>
<div style><br></div><div style><br></div><div style>/Dan</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 January 2013 11:35, Matthias Wessendorf <span dir="ltr"><<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div class="im">On Thu, Jan 3, 2013 at 11:24 AM, Daniel Bevenius <span dir="ltr"><<a href="mailto:daniel.bevenius@gmail.com" target="_blank">daniel.bevenius@gmail.com</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"><span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif">We've pushed a version of AeroGear Controller with support for request parameters</span><span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif"> (</span><a href="https://issues.jboss.org/browse/AEROGEAR-671" style="font-family:Helvetica,arial,freesans,clean,sans-serif;font-size:14px;line-height:22px;color:rgb(65,131,196);text-decoration:initial" target="_blank">AEROGEAR-671</a>)<span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif"> </span><span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif">to</span><span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif"> </span><a href="http://controllerdemo-danbev.rhcloud.com/aerogear-controller-demo/" style="font-family:Helvetica,arial,freesans,clean,sans-serif;font-size:14px;line-height:22px;color:rgb(65,131,196);text-decoration:initial" target="_blank">openshift</a><span style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif">.</span><div>
<div><p style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif;margin:15px 0px">This demo has the following new route added to it:</p><div style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif;padding:0px;border:none">
<pre style="font-size:13px;line-height:19px;font-family:Consolas,'Liberation Mono',Courier,monospace;word-wrap:break-word;margin-top:15px;margin-bottom:15px;background-color:rgb(248,248,248);border:1px solid rgb(204,204,204);overflow:auto;padding:6px 10px;border-top-left-radius:3px;border-top-right-radius:3px;border-bottom-right-radius:3px;border-bottom-left-radius:3px">
<span>route</span><span style="font-weight:bold">()</span>
<span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">from</span><span style="font-weight:bold">(</span><span style="color:rgb(221,17,68)">"/cars"</span><span style="font-weight:bold">)</span>
<span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">on</span><span style="font-weight:bold">(</span><span>RequestMethod</span><span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">GET</span><span style="font-weight:bold">)</span>
<span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">produces</span><span style="font-weight:bold">(</span><span>MediaType</span><span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">JSON</span><span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">toString</span><span style="font-weight:bold">(),</span> <span style="color:rgb(221,17,68)">"application/custom"</span><span style="font-weight:bold">)</span>
<span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">to</span><span style="font-weight:bold">(</span><span>Home</span><span style="font-weight:bold">.</span><span style="color:rgb(0,128,128)">class</span><span style="font-weight:bold">).</span><span style="color:rgb(0,128,128)">get</span><span style="font-weight:bold">(</span><span>param</span><span style="font-weight:bold">(</span><span style="color:rgb(221,17,68)">"color"</span><span style="font-weight:bold">,</span> <span style="color:rgb(221,17,68)">"pink"</span><span style="font-weight:bold">),</span> <span>param</span><span style="font-weight:bold">(</span><span style="color:rgb(221,17,68)">"brand"</span><span style="font-weight:bold">,</span> <span style="color:rgb(221,17,68)">"mini"</span><span style="font-weight:bold">));</span>
</pre></div><p style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif;margin:15px 0px"></p></div></div></div></blockquote><div><br></div><div><br></div></div><div>Hrm, not sure if I like having the default values for the parameters inside of the routing definition.</div>
<span class="HOEnZb"><font color="#888888">
<div><br></div><div>-Matthias</div><div><br></div><div><br></div></font></span></div><span class="HOEnZb"><font color="#888888">-- <br>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</font></span><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
<br></blockquote></div><br></div>