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:57 | Date Closed: | 2006-10-02 15:36:30 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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. |