ASTERISK-09120: [patch] app_queue should use membername for COMPLETECALLER instead of channel ID

[Home]

Summary: ASTERISK-09120: [patch] app_queue should use membername for COMPLETECALLER instead of channel ID

Reporter: Bradley Watkins (marquis) Labels:

Date Opened: 2007-03-27 16:35:37 Date Closed: 2007-04-09 14:07:24

Priority: Minor Regression? No

Status: Closed/Complete Components: Applications/app_queue

Versions: Frequency of
Occurrence

Related
Issues:

Environment: Attachments: ( 0) app_queue_COMPLETECALLER_membername.diff

Description: app_queue should use membername rather than peer channel name for COMPLETECALLER queue log entries.

Comments: By: jmls (jmls) 2007-04-04 01:14:31

is this a bug, or a feature ? Bugs can go into 1.4, features into trunk ...
By: Bradley Watkins (marquis) 2007-04-04 04:17:12

In my opinion it's a bug. All other instances of ast_queue_log use the membername where appropriate (i.e., COMPLETEAGENT, TRANSFER, etc.; not for ENTERQUEUE and so forth). COMPLETECALLER is the only one that does not, which I believe is merely an oversight.
By: Jason Parker (jparker) 2007-04-09 14:07:23

Fixed in svn 1.4 and trunk in revisions 61022 and 61023.

1.2 has similar code, but was not fixed on purpose. All of 1.2 still uses peer->name.