[asterisk-bugs] [Asterisk-GUI 0012331]: Macro_trunkdial needs a small change

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12331 
====================================================================== 
Reported By:                dmgeurts
Assigned To:                bkruse
====================================================================== 
Project:                    Asterisk-GUI
Issue ID:                   12331
Category:                   Service Providers/Trunks
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-29-2008 16:46 CDT
Last Modified:              03-30-2008 16:27 CDT
====================================================================== 
Summary:                    Macro_trunkdial needs a small change
Description: 
[macro-trunkdial]
exten = s,1,set(CALLERID(all)=${IF($["${LEN(${CALLERID(num)})}" >
"6"]?${CALLERID(all)}:${ARG2})})
exten = s,n,Dial(${ARG1})
exten = s,n,Goto(s-${DIALSTATUS},1)
exten = s-NOANSWER,1,Hangup

first line should be (no "'s):

[macro-trunkdial]
exten = s,1,set(CALLERID(all)=${IF($[${LEN(${CALLERID(num)})} >
6]?${CALLERID(all)}:${ARG2})})
====================================================================== 

---------------------------------------------------------------------- 
 dmgeurts - 03-30-08 16:26  
---------------------------------------------------------------------- 
I started looking at the line as it wasn't working for me. Adding a few
NoOp's showed me that "14" > "6" was regarded as false and 14 > 6 was true.
So now I've removed the quotes all works fine.

Sounds to me that there might be a bug elsewhere?

Tx 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-30-08 16:27  dmgeurts       Note Added: 0084766                          
======================================================================