<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
It seems I was not very happy on my example given that the channel
won't be present and/or usefull on "config_load_func" and
"realtime_update" methods.<br>
<br>
Anyway, what I would like to know is how open is Asterisk's code base
decision board to suggestions like this?<br>
I mean, not exactly like this because I agree is there some wrong
points on my suggestion.<br>
<br>
The point is: Is it possible to get some patch sent to you guys and see
it applied if it's somehow replicable (even when it doen't aid to
current implementations) ?<br>
For example: a patch to add the channel to methods of realtime
responsible for retrive the dialplan information can be usefull for
other people that decides to make their own proprietary drivers like
us, while this change don't demand any kind of test and/or validation
on drivers already created that will only disrespect this new parameter.<br>
<br>
I'm asking this question because we use to contribute with other
open-source projects here and we try to always validate the possibility
to get some patch applied when we see that it can be usefull to other
people, on a effort to keep our codes as similar as possible to the
original ones to facilitate new patch creation and submission.<br>
<br>
Thanks again for everything and best regards,<br>
Mauro.<br>
<br>
<br>
<br>
<br>
Tilghman Lesher escreveu:
<blockquote
cite="mid:200908171039.28359.tilghman@mail.jeffandtilghman.com"
type="cite">
<pre wrap="">On Monday 17 August 2009 08:33:36 am Mauro Sergio Ferreira Brasil wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Tilghman Lesher escreveu:
</pre>
<blockquote type="cite">
<pre wrap="">Well, most of the information you're looking for is actually passed back
to the realtime driver already, as parameters to the query. That's true
for both context and extension. The only part that isn't passed back is
callerid number. That could be added without too much trouble to the
pbx_realtime module, although it would not be backwards compatible with
most
implementations. In addition, I'd suggest using something like the
res_config_curl driver to make your extra evaluation on the backend
easy.
</pre>
</blockquote>
<pre wrap="">I'm not completely sure whether your suggestion is to add the additional
CALLERID information to the list of parameters that are already being
passed to realtime procedures, because on that case it will get
automatically added to SQL query statement, won't it ?
</pre>
</blockquote>
<pre wrap=""><!---->
Only if your backend goes directly to an SQL query statement. As I detailed
above, it's certainly possible to substitute a middleware piece in between
the realtime backend and your database, and as I've indicated, there's already
a driver that lets you do exactly that.
</pre>
<blockquote type="cite">
<pre wrap="">Anyway, I would like to place a question that will be more profitable:
It would be acceptable to asterisk staff a change on realtime so the
channel parameter gets forwarded to all realtime configuration routines
("config_load_func", "realtime_var_get", "realtime_multi_get" and
"realtime_update") declared at "main/config.h" ? (I know that this will
demand changes on some methods from "pbx/pbx_realtime.c" so the channel
can indeed be forwarded, and on all implemented drivers as well. But a
controlled change, I think.)
</pre>
</blockquote>
<pre wrap=""><!---->
No, we're not going to do that. The callerid is the only extra information
that you need, the channel is not necessary, and in some cases, the channel
does not even exist when realtime is called. It is therefore inappropriate to
demand that a channel parameter be passed.
</pre>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<meta http-equiv="Content-Language" content="pt-br">
<meta name="GENERATOR" content="Microsoft FrontPage 5.0">
<meta name="ProgId" content="FrontPage.Editor.Document">
<meta http-equiv="Content-Type" content="text/html; ">
<title>TQI - Technology and Quality on Information</title>
<table style="border-width: 0pt; border-collapse: collapse;"
id="AutoNumber1" border="1" bordercolor="#111111" cellpadding="0"
cellspacing="0" height="151" width="46%">
<tbody>
<tr>
<td style="border-style: none; border-width: medium;" height="17"
width="96%"> <sub><u>At.,
</u></sub></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="32"
width="16%"> <img src="cid:part1.01000606.00080301@tqi.com.br"
border="0" height="23" width="150"> </td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="12"
width="100%"><b><font face="Verdana" size="1">Technology and Quality
on Information</font></b></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="13"
width="100%"><font face="Verdana" size="1">Mauro Sérgio Ferreira Brasil</font></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="14"
width="100%"> <font face="Verdana" size="1">Coordenador de Projetos e
Analista de Sistemas</font></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="14"
width="100%"><font color="#0000ff" face="Wingdings" size="2">+</font><font
face="Courier New" size="2"> </font> <font style="font-size: 9pt;"
face="Courier New"> <a href="mailto:@tqi.com.br">mauro.brasil@tqi.com.br</a></font></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="14"
width="100%"><font color="#0000ff" face="Wingdings" size="2">:</font><font
face="Courier New" size="2"> </font>
<font style="font-size: 9pt;" face="Courier New"> <a
href="http://www.tqi.com.br">www.tqi.com.br</a></font></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="14"
width="100%"><font color="#0000ff"><font face="Wingdings" size="2">(</font><font
face="Courier New" size="2"> + 55 </font>
<font style="font-size: 9pt;" face="Courier New">(34)3291-1700</font></font></td>
</tr>
<tr>
<td style="border-style: none; border-width: medium;" height="14"
width="100%"><font color="#0000ff"><font face="Wingdings" size="2">(</font><font
face="Courier New" size="2"> + 55 </font>
<font style="font-size: 9pt;" face="Courier New">(34)9971-2572</font></font></td>
</tr>
</tbody>
</table>
</div>
</body>
</html>