[Home]

Summary:ASTERISK-15561: [patch] Initial pause not implemented, but documented as available.
Reporter:Alec Davis (alecdavis)Labels:
Date Opened:2010-02-02 02:53:07.000-0600Date Closed:2010-03-11 09:47:14.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_directory
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) directory_pause_1.6.1_diff.txt
( 1) directory_pause.trunk.diff.txt
Description:the Intial pause after entering the first 3 letters of the directory name although documented, isn't implemented, from 1.6.1 to trunk.

Command is accepted, just not acted on.


****** ADDITIONAL INFORMATION ******

core show application DIRECTORY shows:

...
 p(n): Pause for n milliseconds after the digits are typed.  This
   is helpful for people with cellphones, who are not holding the receiver
   to their ear while entering DTMF.
...

dialplan example:
exten => 1,n,Directory(default,ivr2-day-transfer,p(1000))
Comments:By: Alec Davis (alecdavis) 2010-02-02 03:01:13.000-0600

uploaded directory_pause_1.6.1_diff.txt and directory_pause.trunk.diff.txt

Tested in 1.6.1 production server, with options;

'p(6000)' where initial pause is shortened to 3000ms, and individual names are presented, requiring '*' to move on to next.
'mp(1000)' where names are present as item list, and initial pause of 1000ms is honoured.
'p' where no pause is is actioned.


By: Alec Davis (alecdavis) 2010-02-02 03:54:52.000-0600

Up for review: https://reviewboard.asterisk.org/r/481/
By: Leif Madsen (lmadsen) 2010-02-02 08:23:33.000-0600

Thanks for the contribution! Love when things get documented but not implemented :)
By: Digium Subversion (svnbot) 2010-03-11 01:03:55.000-0600

Repository: asterisk
Revision: 251779

U   trunk/apps/app_directory.c

------------------------------------------------------------------------
r251779 | alecdavis | 2010-03-11 01:03:53 -0600 (Thu, 11 Mar 2010) | 14 lines

Add supporting code for app-directory pause option.

Since 1.6.1 CLI help reports that option p(n) 'initial pause' is available.
Supporting code was never implemented.

(closes issue ASTERISK-15561)
Reported by: alecdavis
Patches:
     directory_pause.trunk.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis

Review: https://reviewboard.asterisk.org/r/481/


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=251779
By: Digium Subversion (svnbot) 2010-03-11 01:37:49.000-0600

Repository: asterisk
Revision: 251819

U   branches/1.6.1/apps/app_directory.c

------------------------------------------------------------------------
r251819 | alecdavis | 2010-03-11 01:37:49 -0600 (Thu, 11 Mar 2010) | 14 lines

Add supporting code for app-directory pause option.

Since 1.6.1 CLI help reports that option p(n) 'initial pause' is available.
Supporting code was never implemented.

(closes issue ASTERISK-15561)
Reported by: alecdavis
Patches:
     based on directory_pause.trunk.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis

Review: https://reviewboard.asterisk.org/r/481/


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=251819
By: Digium Subversion (svnbot) 2010-03-11 01:48:15.000-0600

Repository: asterisk
Revision: 251820

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_directory.c

------------------------------------------------------------------------
r251820 | alecdavis | 2010-03-11 01:48:15 -0600 (Thu, 11 Mar 2010) | 20 lines

Merged revisions 251779 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r251779 | alecdavis | 2010-03-11 20:03:51 +1300 (Thu, 11 Mar 2010) | 14 lines
 
 Add supporting code for app-directory pause option.
 
 Since 1.6.1 CLI help reports that option p(n) 'initial pause' is available.
 Supporting code was never implemented.
 
 (closes issue ASTERISK-15561)
 Reported by: alecdavis
 Patches:
       directory_pause.trunk.diff.txt uploaded by alecdavis (license 585)
 Tested by: alecdavis
 
 Review: https://reviewboard.asterisk.org/r/481/
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=251820