We have such a variable called "read_result"<br>possible vals: success, timeout, failure<br><br>"read_terminator_used" contains the terminator<br><br><br><div class="gmail_quote">On Tue, Nov 3, 2009 at 2:20 PM, Peter Volchek <span dir="ltr"><<a href="mailto:pvolchek@voicemobility.com">pvolchek@voicemobility.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hi,<br>
I am having some troubles adopting [session:playAndGetDigits] function<br>
to my application needs.<br>
<br>
There are three possible outcomes that the function should report on:<br>
1. Success. The required number of of digits was collected and they<br>
match the mask.<br>
2. Fail 1.The entered digits do not match the mask<br>
3. Fail2. Timeout failure<br>
<br>
Right now, I cannot distinguish between 2 & 3. In both cases, an empty<br>
string is returned, so I have no idea what was the real failure.<br>
<br>
4. Success upon termination<br>
That is a tricky one, and I believe, there is a bug in this function.<br>
This function accept the terminators, which should stop digits collection.<br>
Consider the following function call:<br>
<br>
digits = session:playAndGetDigits(<br>
0, 5, 1, 3000, "#",<br>
"enter_mailbox_number_or_hit_pound_to_login.wav",<br>
"error.wav",<br>
"\\d*"<br>
);<br>
<br>
<br>
Here, I want to collect up to 5 digits, that represent a mailbox number.<br>
Note that I set the minimum value to be 0, as I want "#" itself to be<br>
the sign, that I want to login to my system.<br>
However, when I execute this code and hit "#" immediately, the system<br>
play me an error message. Oops! Well, I changed my digits mask to be<br>
"#", and it did not help either.<br>
BTW, about the last case. If I my digits mask contains the symbol used<br>
in the terminators, then it has to be collected and returned. The<br>
current implementation just uses terminates the collection and does not<br>
include the terminator to the returned string.<br>
<br>
Possible solutions to determine the function's outcome:<br>
1. Return multiple values (state, digits). That may break tons of the<br>
existing code<br>
2. Set a channel (or session) variable. Similar to C's errno and define<br>
some variables defining the outcome. Below is a summary table that<br>
covers all the cases:<br>
<br>
------------------------------------------------------<br>
ERRNO | Description<br>
------------------------------------------------------<br>
RET_OK | Maximum number of digits was collected or<br>
| terminated when in range [min_digits..max_digits-1]<br>
| or timeout triggered after collecting enough digits<br>
[min_digits..max_digits-1].<br>
| The collected digits match digits_regex<br>
| Returns the collected digits<br>
<br>
RET_TERM | Terminating before collecting enough digits or when<br>
| the collected digits do not match digits_regex<br>
| Return whatever was collected (for logging)<br>
<br>
<br>
RET_TIMEOUT | Did not collect enough digits for the time specified or when<br>
| the collected digits do not match digits_regex<br>
| Return whatever was collected (for logging)<br>
<br>
<br>
Does it all make sense?<br>
Cheers,<br>
PV<br>
<br>
<br>
<br>
_______________________________________________<br>
FreeSWITCH-dev mailing list<br>
<a href="mailto:FreeSWITCH-dev@lists.freeswitch.org">FreeSWITCH-dev@lists.freeswitch.org</a><br>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev</a><br>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-dev</a><br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Anthony Minessale II<br><br>FreeSWITCH <a href="http://www.freeswitch.org/">http://www.freeswitch.org/</a><br>ClueCon <a href="http://www.cluecon.com/">http://www.cluecon.com/</a><br>
Twitter: <a href="http://twitter.com/FreeSWITCH_wire">http://twitter.com/FreeSWITCH_wire</a><br><br>AIM: anthm<br><a href="mailto:MSN%3Aanthony_minessale@hotmail.com">MSN:anthony_minessale@hotmail.com</a><br>GTALK/JABBER/<a href="mailto:PAYPAL%3Aanthony.minessale@gmail.com">PAYPAL:anthony.minessale@gmail.com</a><br>
IRC: <a href="http://irc.freenode.net">irc.freenode.net</a> #freeswitch<br><br>FreeSWITCH Developer Conference<br><a href="mailto:sip%3A888@conference.freeswitch.org">sip:888@conference.freeswitch.org</a><br><a href="http://iax:guest@conference.freeswitch.org/888">iax:guest@conference.freeswitch.org/888</a><br>
<a href="mailto:googletalk%3Aconf%2B888@conference.freeswitch.org">googletalk:conf+888@conference.freeswitch.org</a><br>pstn:213-799-1400<br>