[Home]

Summary:ASTERISK-14794: [patch] Posibility to send two channels in different direcitons
Reporter:Håkon Nessjøen (haakon)Labels:
Date Opened:2009-09-08 11:14:41Date Closed:2009-12-15 18:31:56.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface/NewFeature
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) manager.c.patch
( 1) trunk-manager.c.patch
Description:This patch gives the user the posibility to send two connected channels in two different directions with the AMI Redirect command.

Example:
 * User1 calls user2.
 * "Web interface" wants to put User1 in a parking extension, and User2 in a dial extension.

With this patch, this is feasible with the following AMI request:

Action: Redirect
Channel: SIP/user1-chan
Context: parkinglot
Extension: 123
Priority: 1
ExtraChannel: SIP/user2-chan
ExtraContext: dialout
ExtraExtension: 12345678
ExtraPriority: 1

In before the patch, you had ExtraChannel, but it transfered both channels to the same context/extension. Sometimes this is not what you want.

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

I've also made a patch for this, with current HEAD revision (rev 216993), if this is better to post?
Comments:By: Leif Madsen (lmadsen) 2009-09-08 11:15:57

All patches for new features *must* be against trunk, and not a released branch. No new features go into existing branches other than trunk. Thanks!

By: Leif Madsen (lmadsen) 2009-09-08 11:16:24

Status to "Needs License" until the license has been approved.

By: Håkon Nessjøen (haakon) 2009-09-08 11:26:17

Uploaded patch for current HEAD revision 216993. See trunk-manager.c.patch

By: Håkon Nessjøen (haakon) 2009-09-13 13:40:37

Is it normal that the ticket stays this long in "Needs license" status?

By: Leif Madsen (lmadsen) 2009-09-16 09:10:54

It is a manual process to change the statuses. My apologies for now seeing it sooner.

Thanks snuffy for changing status back.

I will change this to the new appropriate status of Ready for Testing. If you can drum up some support on the mailing list for people to test this issue and report back here, that would be great!

Thanks!
Leif.

By: Håkon Nessjøen (haakon) 2009-12-04 09:22:19.000-0600

Maybe some nice karma is due? :)

I'm unable to get people on the -dev list interested. But I can see you have assigned it to someone, with a target version, so I guess something is happening? Or should I also register and try on the -user list?

By: Jeff Peeler (jpeeler) 2009-12-04 13:47:39.000-0600

We don't really use the karma system, but I can give you some if you really want it once this gets merged. I'll get around to it probably in the new few weeks.

By: Leif Madsen (lmadsen) 2009-12-07 12:22:39.000-0600

I just gave the karma now so it doesn't get missed, but jpeeler is right that we're not really using the karma system anymore.

By: Digium Subversion (svnbot) 2009-12-15 18:31:54.000-0600

Repository: asterisk
Revision: 235265

U   trunk/CHANGES
U   trunk/main/manager.c

------------------------------------------------------------------------
r235265 | jpeeler | 2009-12-15 18:31:54 -0600 (Tue, 15 Dec 2009) | 12 lines

Enhance AMI redirect to allow channels to be redirected to different places.

New parameters ExtraContext, ExtraExtension, and ExtraPriority have been added
to redirect the second channel to a different location. Previously, it was only
possible to redirect both channels to the same place.

(closes issue ASTERISK-14794)
Reported by: haakon
Patches:
     trunk-manager.c.patch uploaded by haakon (license 880)
Tested by: jpeeler

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

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