Return message values as string if there's a leading plus sign

chan-sccp / PAMI

Fork of PHP Asterisk Manager Interface ( AMI ) supports synchronous command ( action )/ responses and asynchronous events using the pattern observer-listener. Supports commands with responses with multiple events. Very suitable for development of operator consoles and / or asterisk / channels / peers monitoring through SOA, etc

http://marcelog.github.com/PAMI

Apache License 2.0

27 stars 21 forks source link

Return message values as string if there's a leading plus sign #12

Closed clemens321 closed 4 years ago

clemens321 commented 4 years ago

Avoids losing information e.g. when having numbers in international e.123 format which have a leading plus sign.

dkgroot commented 4 years ago

I agree with your PR, so will merge it right now. Thanks !!

In retrospect: Making the sanitization during input parsing the default, might not have been such a good idea. It might have been better to have getKey($name, $sanitze=true) as part of the API. However that would require re-parsing the key(s).