Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

play(some_number) calls both saynumber and playback

#53

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

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

State: resolved
Milestone: 0.8.5
Assigned to: Ben Klang Ben Klang

Activity

  1. Jay Phillips
    Jay Phillips
    • Milestone cleared.
    • State changed from new to open

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

    March 24th, 2009 @ 06:43 PM

  2. Ben Klang
    Ben Klang
    • 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...

    August 2nd, 2010 @ 12:08 PM

  3. Ben Klang
    Ben Klang
    • State changed from fixcommitted to resolved

    August 24th, 2010 @ 01:56 PM

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.