asterisk users - Jun 2005 - * 1.0.8: no more reacting to callerid?

It's not just you.  Same thing happens here. I went back to 1.0.7.

Stefan Gofferje wrote:
> Hi folks,
> 
> I used to have some constructions like
> 
> exten => number/callerid,1,Goto(somewhere)
> 
> After updating to 1.0.8 those does not work any more.
> Any hints?
> 
> Regards,
> Stefan
>

Daryl Jones wrote:
> It's not just you.  Same thing happens here. I went back to 1.0.7.
There is definitely breakage in 1.0.8 in this area; please test the 
patch below and report back the results here so we can get a new release 
made.

diff -u -r1.45.2.2 pbx_config.c
--- pbx/pbx_config.c    19 May 2005 02:51:00 -0000      1.45.2.2
+++ pbx/pbx_config.c    25 Jun 2005 17:32:47 -0000
@@ -1687,15 +1687,10 @@
                                                                 else
 
  data = "";
                                                         }
- 
pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext)-1);
-                                                       cidmatch = 
strchr(ext, '/');
-                                                       if (cidmatch) {
- 
*cidmatch = '\0';
-                                                               cidmatch++;
-                                                       }
-                                                       stringp=ext;
-                                                       strsep(&stringp, 
"/");
-
+ 
pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext)-1);
+                                                       stringp = realext;
+                                                       ext = 
strsep(&stringp, "/");
+                                                       cidmatch = stringp;
                                                         if (!data)
                                                                
data="";
                                                         while(*appl && 
(*appl < 33)) appl++;

Stefan Gofferje wrote:
> The patch is rejected here.
My email client ate the tabs; the patch should apply with '-l' to ignore
whitespace changes, or you can manually do it (it's only four lines).

Stefan Gofferje wrote:
> Tested:
> 
> [incoming_ISDN]
> exten => msn/number,1,Zapateller
> exten => msn/mymobileno,1,DISA(no-password|client_int_unrestricted)
> (...)
> 
> [internal_ISDN_clients]
> exten => s/isdnphoneno,1,DISA(no-password|client_int_unrestricted)
> (...)
> 
> Looks fine!
Great, we'll get it into CVS and a new release made ASAP!