Summary: | ASTERISK-03411: non-numerical calleridnum can't be presented | ||
Reporter: | mcisse (mcisse) | Labels: | |
Date Opened: | 2005-02-01 09:58:36.000-0600 | Date Closed: | 2008-01-15 15:24:31.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Channels/chan_sip/Interoperability |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
Description: | When trying to set calleridnum to a non numerical value, it doesn't work. Normal, you'll say. But I've some stupid phones that will present only NUM from "NAME" <sip:NUM@IP>. NAME is ignored by the phone. So, I'd like to be able to present non-numerical values (names) using NUM. When trying to set a numerical calleridnum, it is replaced by the default callerid. The "problem" is line 3730 of chan_sip.c (CVS HEAD): if (!l || !ast_isphonenumber(l)) l = default_callerid; Personally, I've replaced it by: if ((!l || !ast_isphonenumber(l)) && *default_callerid!=0) l = default_callerid; | ||
Comments: | By: Mark Spencer (markster) 2005-02-01 13:24:29.000-0600 Okies fixed in CVS. By: Russell Bryant (russell) 2005-02-06 21:24:52.000-0600 fixed in 1.0 By: Digium Subversion (svnbot) 2008-01-15 15:23:59.000-0600 Repository: asterisk Revision: 4944 U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r4944 | markster | 2008-01-15 15:23:58 -0600 (Tue, 15 Jan 2008) | 2 lines Only use default callerid if it's specified (bug ASTERISK-3411) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=4944 By: Digium Subversion (svnbot) 2008-01-15 15:24:31.000-0600 Repository: asterisk Revision: 4972 U branches/v1-0/channels/chan_sip.c ------------------------------------------------------------------------ r4972 | russell | 2008-01-15 15:24:30 -0600 (Tue, 15 Jan 2008) | 2 lines only use default cid if specified (bug ASTERISK-3411) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=4972 |