[Home]

Summary:ASTERISK-07763: [patch] meetme uses inconsistent names for callerid information for manager events
Reporter:Stephen Uhler (suhler)Labels:
Date Opened:2006-09-18 10:38:57Date Closed:2006-10-02 15:36:30
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) meetme-diffs
Description:When generating manager events, meetme uses the names:
 "CallerIDnum" and "CallerIDname"
[note "CallerIDname" with a small "n" for name]
Which is inconsistent with other applications that provide this information
in Manager event.  This makes it harder for applications that use AMI, as they
need to keep a table to map the inconsistent event names.

The fix to meetme is trivial, so no patch is included:
change "CallerIDname" to "CallerIDName"

In addition, "features.c" and "channel.c"
use "CallerID" instead of "CallerIDnum", [but perhaps that should be
a different bug report?]

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

Summary of the commands and the various spellings used to report callerid
information in Manager events:

app_queue.c
app_dial.c
manager.c
 CallerIDNum CallerIDName
features.c:
channel.c:
 CallerID CallerIDName
app_meetme.c
 CallerIDnum CallerIDname
Comments:By: Serge Vecher (serge-v) 2006-09-18 10:51:13

suhler: we will accept a patch here, however trivial, which will earn you karma. Please do file another bug report for other changes. Thanks.

By: Stephen Uhler (suhler) 2006-09-18 11:37:56

I attached a diff to change the callerid names meetme uses in manager events to be consistent with "app_queue", "app_dial", and "manager"

By: Joshua C. Colp (jcolp) 2006-10-02 15:36:29

Deprecated by 7976 fix.