[asterisk-dev] [Code Review] 4167: Allow mis-dialed DTMF-initiated transfers to be re-attempted.

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4167/
-----------------------------------------------------------

(Updated Nov. 12, 2014, 6:32 p.m.)


Review request for Asterisk Developers.


Changes
-------

It was conveyed off-list that if I'm messing with this area, the sound files to play should be configurable as well. It was easy enough to implement, so I added the "transferretrysound" and "transferinvalidsound" options to be played. I'm open to different names for any of the options introduced in this changeset.


Repository: Asterisk


Description
-------

There have been complaints about the fact that when a DTMF transfer is initiated, and an incorrect number is dialled, the prompt says "Please try again" but then doesn't give the transferer a chance to do so without re-initiating the transfer.

This patch adds a features.conf option called "transferdialattempts" (yes, the name is gross, but it fits in with the convention of other options in features.conf). This option controls how many attempts a transferer has to dial a valid extension before being booted back out to the call with the transferee. The option defaults to 3. As with other options from features.conf, this can also be set using the FEATURE() dialplan function so that it may be changed per-call if desired.


Diffs (updated)
-----

  /trunk/main/features_config.c 427676 
  /trunk/main/bridge_basic.c 427676 
  /trunk/include/asterisk/features_config.h 427676 

Diff: https://reviewboard.asterisk.org/r/4167/diff/


Testing
-------

See /r/4168


Thanks,

Mark Michelson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141112/1985174e/attachment.html>