Asterisk
  1. Asterisk
  2. ASTERISK-14794

[patch] Posibility to send two channels in different direcitons

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Labels:
      None
    • Mantis ID:
      15853
    • Regression:
      No

      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?

      1. manager.c.patch
        1 kB
        Håkon Nessjøen
      2. trunk-manager.c.patch
        2 kB
        Håkon Nessjøen

        Issue Links

          Activity

          Hide
          Leif Madsen added a comment -

          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!

          Show
          Leif Madsen added a comment - 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!
          Hide
          Leif Madsen added a comment -

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

          Show
          Leif Madsen added a comment - Status to "Needs License" until the license has been approved.
          Hide
          Håkon Nessjøen added a comment -

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

          Show
          Håkon Nessjøen added a comment - Uploaded patch for current HEAD revision 216993. See trunk-manager.c.patch
          Hide
          Håkon Nessjøen added a comment -

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

          Show
          Håkon Nessjøen added a comment - Is it normal that the ticket stays this long in "Needs license" status?
          Hide
          Leif Madsen added a comment -

          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.

          Show
          Leif Madsen added a comment - 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.
          Hide
          Håkon Nessjøen added a comment -

          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?

          Show
          Håkon Nessjøen added a comment - 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?
          Hide
          Jeff Peeler (Inactive) added a comment -

          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.

          Show
          Jeff Peeler (Inactive) added a comment - 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.
          Hide
          Leif Madsen added a comment -

          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.

          Show
          Leif Madsen added a comment - 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.
          Hide
          Digium Subversion added a comment -

          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

          Show
          Digium Subversion added a comment - 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

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development