<div dir="ltr">Hi,<div>I'd go for the second option, cleaner.</div><div>Once we have a list of core functions to be migrated we could split the work among the developers.</div><div>And of course write a nice migration doc 5.2 => 5.3.</div><div><br></div><div>Cheers,</div><div><br></div><div>Federico</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 19, 2018 at 8:50 AM Daniel-Constantin Mierla <<a href="mailto:miconda@gmail.com" target="_blank">miconda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
it was brought into discussions several times in the past about core<br>
functions not accepting variables in the parameters. I think it is time<br>
to update them during the 5.3 release development. For few of them, I<br>
added in the past some alternative function in the corex module (e.g.,<br>
force_send_socket() in core and set_send_socket() in corex module).<br>
<br>
So, I see two options:<br>
<br>
1) add a function with similar name in corex module and same behaviour<br>
like the one from core<br>
<br>
2) remove the function export from the core and export one with the same<br>
name from the corex module<br>
<br>
First one will ensure that configs using the functions right now keep<br>
working without any update.<br>
<br>
The second one will be better in long term from the point of<br>
documentation (no duplicated docs), but there might be few cases that<br>
would require updates in the config -- iirc, there are some functions<br>
that can get special tokens in the parameters (like forward(uri:host,<br>
uri:port)), they will get an equivalent with variables, but old config<br>
will not be compatible.<br>
<br>
Obviously the reason for this email is to ask the developers and users<br>
what would be the preferred way from own point of view.<br>
<br>
Cheers,<br>
Daniel<br>
<br>
-- <br>
Daniel-Constantin Mierla -- <a href="http://www.asipto.com" rel="noreferrer" target="_blank">www.asipto.com</a><br>
<a href="http://www.twitter.com/miconda" rel="noreferrer" target="_blank">www.twitter.com/miconda</a> -- <a href="http://www.linkedin.com/in/miconda" rel="noreferrer" target="_blank">www.linkedin.com/in/miconda</a><br>
Kamailio World Conference - May 6-8, 2019 -- <a href="http://www.kamailioworld.com" rel="noreferrer" target="_blank">www.kamailioworld.com</a><br>
Kamailio Advanced Training - Mar 4-6, 2019 in Berlin; Mar 25-27, 2019, in Washington, DC, USA -- <a href="http://www.asipto.com" rel="noreferrer" target="_blank">www.asipto.com</a><br>
<br>
<br>
_______________________________________________<br>
Kamailio (SER) - Development Mailing List<br>
<a href="mailto:sr-dev@lists.kamailio.org" target="_blank">sr-dev@lists.kamailio.org</a><br>
<a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer" target="_blank">https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
</blockquote></div>