[Asterisk-code-review] bridge basic: don't play an attended transfer fail sound aft... (asterisk[master])

[Joshua Colp]

Joshua Colp has submitted this change and it was merged.

Change subject: bridge_basic: don't play an attended transfer fail sound after target hangs up
......................................................................


bridge_basic: don't play an attended transfer fail sound after target hangs up

If the attended transfer destination answers (picks call up or goes to
voicemail) and then hangs up on the transferer then transferer hears the
fail sound.

This patch makes it so the fail sound is not played when the transfer
destination/target hangs up after answering.

ASTERISK-25697 #close

Change-Id: I97f142fe4fc2805d1a24b7c16143069dc03d9ded
---
M main/bridge_basic.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Mark Michelson: Looks good to me, but someone else must approve
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/main/bridge_basic.c b/main/bridge_basic.c
index cd19915..47d838f 100644
--- a/main/bridge_basic.c
+++ b/main/bridge_basic.c
@@ -2178,6 +2178,7 @@
 		bridge_unhold(props->transferee_bridge);
 		return TRANSFER_COMPLETE;
 	case STIMULUS_TRANSFER_TARGET_HANGUP:
+		return TRANSFER_REBRIDGE;
 	case STIMULUS_DTMF_ATXFER_ABORT:
 		play_sound(props->transferer, props->failsound);
 		return TRANSFER_REBRIDGE;

-- 
To view, visit https://gerrit.asterisk.org/2018
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I97f142fe4fc2805d1a24b7c16143069dc03d9ded
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>