[asterisk-bugs] [Asterisk 0015439]: [patch] URIENCODE() throws a warning when passed an empty string
Asterisk Bug Tracker
noreply at bugs.digium.com
Tue Jul 21 17:38:58 CDT 2009
A NOTE has been added to this issue.
======================================================================
https://issues.asterisk.org/view.php?id=15439
======================================================================
Reported By: pkempgen
Assigned To: tilghman
======================================================================
Project: Asterisk
Issue ID: 15439
Category: Functions/func_uri
Reproducibility: always
Severity: trivial
Priority: normal
Status: ready for testing
Asterisk Version: SVN
Regression: No
SVN Branch (only for SVN checkouts, not tarball releases): 1.4
SVN Revision (number only!): 176029
Request Review:
======================================================================
Date Submitted: 2009-07-02 00:54 CDT
Last Modified: 2009-07-21 17:38 CDT
======================================================================
Summary: [patch] URIENCODE() throws a warning when passed an
empty string
Description:
URIENCODE(${empty}) throws a warning:
WARNING[23535]: func_uri.c:51 uriencode: Syntax: URIENCODE(<data>) -
missing argument!
other functions like
QUOTE(${empty})
don't complain.
======================================================================
----------------------------------------------------------------------
(0108057) svnbot (reporter) - 2009-07-21 17:38
https://issues.asterisk.org/view.php?id=15439#c108057
----------------------------------------------------------------------
Repository: asterisk
Revision: 207945
U branches/1.4/funcs/func_strings.c
------------------------------------------------------------------------
r207945 | tilghman | 2009-07-21 17:38:57 -0500 (Tue, 21 Jul 2009) | 8
lines
Force an error if a blank is passed to QUOTE (because the documentation
states the argument is not optional).
This change makes URIENCODE and QUOTE behave similarly, since the
documentation
states that the argument is not optional, for both.
(closes issue https://issues.asterisk.org/view.php?id=15439)
Reported by: pkempgen
Patches:
20090706__issue15439.diff.txt uploaded by tilghman (license 14)
------------------------------------------------------------------------
http://svn.digium.com/view/asterisk?view=rev&revision=207945
Issue History
Date Modified Username Field Change
======================================================================
2009-07-21 17:38 svnbot Checkin
2009-07-21 17:38 svnbot Note Added: 0108057
======================================================================
More information about the asterisk-bugs
mailing list