#53 ✓resolved
ronin-9952 (at lighthouseapp)

play(some_number) calls both saynumber and playback

Reported by ronin-9952 (at lighthouseapp) | January 11th, 2009 @ 10:17 AM | in 0.8.5

Although saynumber executes successfully, adhearsion assumes it failed and calls playback

Code:

play response_time # response_time is an integer

Results in ahn_log:

DEBUG agi: >>> EXEC saynumber 90 DEBUG agi: <<< 200 result=0

DEBUG agi: >>> EXEC playback 90 DEBUG agi: <<< 200 result=0

Results in asterisk console:

-- AGI Script Executing Application: (saynumber) Options: (90)
-- <SIP/excel2-16522910> Playing 'digits/90' (language 'en')
-- AGI Script Executing Application: (playback) Options: (90)

[Jan 11 15:11:19] WARNING[11120]: file.c:602 ast_openstream_full: File 90 does not exist in any format [Jan 11 15:11:19] WARNING[11120]: file.c:912 ast_streamfile: Unable to open 90 (format 0x4 (ulaw)): No such file or directory [Jan 11 15:11:19] WARNING[11120]: app_playback.c:439 playback_exec: ast_streamfile failed on SIP/excel2-16522910 for 90

Comments and changes to this ticket

  • Jay Phillips

    Jay Phillips March 24th, 2009 @ 06:43 PM

    • Milestone cleared.
    • State changed from “new” to “open”

    I need to complete remove the error?() method altogether.

  • Ben Klang

    Ben Klang August 2nd, 2010 @ 12:08 PM

    • State changed from “open” to “fixcommitted”
    • Assigned user changed from “Jay Phillips” to “Ben Klang”
    • Tag set to agi
    • Milestone set to 0.8.5
    • Milestone order changed from “0” to “0”

    I have just pushed a fix for this. Previously, Adhearsion treated AGI results like "200 result=0" as if the command had failed. Based on my reading of the Asterisk sources, I believe that only negative numbers (ie. "200 result=-1") are the actual errors. I have just pushed a change to Github that should fix this issue. Testing is appreciated to make sure I did not break anything else in the process (though I suspect this also fixes other issues).

    http://github.com/adhearsion/adhearsion/commit/53aab23f2e18caae14bd...

  • Ben Klang

    Ben Klang August 24th, 2010 @ 01:56 PM

    • State changed from “fixcommitted” to “resolved”

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Shared Ticket Bins

People watching this ticket

Tags

Referenced by

Pages