[Home]

Summary:ASTERISK-16707: [patch] func_strings would like a STRREPLACE function
Reporter:Walter Doekes (wdoekes)Labels:
Date Opened:2010-09-21 10:10:53Date Closed:2011-05-20 11:27:15
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Functions/func_strings
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) issue18023_func_strings_strreplace-1.6.2.13.patch
Description:Hi,

I needed a function to replace substrings in a variable:

STRREPLACE(<varname>,<needle>,<replacement>[,<occurrences>])

I hope it gets included, saves me a manual patch ;)

Regards,
Walter Doekes
OSSO B.V.

Comments:By: Walter Doekes (wdoekes) 2010-09-23 08:20:22

Oh, I missed a </para> in the documentation.

-If <replaceable>occurrences</replaceable> is not specified, it defaults to all.
+If <replaceable>occurrences</replaceable> is not specified, it defaults to all.</para>
By: dovid (dovid) 2011-02-16 05:57:01.000-0600

Hi,

I tried on 1.8.X and it failed. Can anyone update this for 1.8.X so I can test ? Here is what I get:

patching file funcs/func_strings.c
Hunk #1 succeeded at 164 with fuzz 1 (offset 73 lines).
Hunk #2 succeeded at 911 with fuzz 1 (offset 387 lines).
Hunk #3 FAILED at 1521.
Hunk #4 FAILED at 1547.
2 out of 4 hunks FAILED -- saving rejects to file funcs/func_strings.c.rej

Dovid
By: Paul Belanger (pabelanger) 2011-02-16 08:06:21.000-0600

New features should be made against trunk.  Also, there are some formating issues with the patch.
By: Antonio Berrios (aberrios) 2011-02-23 07:03:57.000-0600

Would like to see this patch in 1.8 branch.
By: Digium Subversion (svnbot) 2011-05-20 11:27:14

Repository: asterisk
Revision: 320040

U   trunk/CHANGES
U   trunk/funcs/func_strings.c

------------------------------------------------------------------------
r320040 | jrose | 2011-05-20 11:27:14 -0500 (Fri, 20 May 2011) | 10 lines

Adds STRREPLACE function

Adds a new STRREPLACe function to func_strings.c that allows users to search and replace
against a variable in the dialplan.

(closes issue ASTERISK-16707)
Reported by: wdoekes

Review: https://reviewboard.asterisk.org/r/1219/

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=320040