``` INFO: <script>: SET_DST trying to get dialplan from dpid:101000 - ci'1cf77cbb5baebdbd0e8635e1116aa713@PBX-2797.xpbx.foehn.co.uk' DEBUG: dialplan [dialplan.c:232]: dp_get_ivalue(): searching 15 DEBUG: dialplan [dialplan.c:240]: dp_get_ivalue(): dpid is 101000 from pv argument DEBUG: dialplan [dialplan.c:249]: dp_get_svalue(): searching 7 DEBUG: dialplan [dialplan.c:345]: dp_translate_f(): input is 0034654205681 DEBUG: dialplan [dp_repl.c:606]: dp_translate_helper(): regex operator testing over [0034654205681] DEBUG: dialplan [dp_repl.c:681]: dp_translate_helper(): found a matching rule 0x7f065c2818d0: pr 1, match_exp ^00([1-9][0-35-9]|[1-35-9][0-9])[0-9]+$ DEBUG: dialplan [dp_repl.c:688]: dp_translate_helper(): the rule's attrs are cli=+34654205682;action=0 DEBUG: dialplan [dp_repl.c:699]: dp_translate_helper(): the copied attributes are: cli=+34654205682;action=0 ERROR: dialplan [dp_repl.c:439]: rule_translate(): the string 0034654205681 matched the match_exp ^00([1-9][0-35-9]|[1-35-9][0-9])[0-9]+$ but not the subst_exp ^([1-9][0-9]{8})$! ERROR: dialplan [dp_repl.c:730]: dp_translate_helper(): could not build the output DEBUG: dialplan [dialplan.c:350]: dp_translate_f(): could not translate 0034654205681 with dpid 101000 INFO: <script>: SET_DST no valid subst_exp for '0034654205681' trying to match - ci='1cf77cbb5baebdbd0e8635e1116aa713@PBX-2797.xpbx.foehn.co.uk' DEBUG: dialplan [dp_repl.c:606]: dp_translate_helper(): regex operator testing over [0034654205681] DEBUG: dialplan [dp_repl.c:681]: dp_translate_helper(): found a matching rule 0x7f065c2818d0: pr 1, match_exp ^00([1-9][0-35-9]|[1-35-9][0-9])[0-9]+$ DEBUG: dialplan [dp_repl.c:688]: dp_translate_helper(): the rule's attrs are cli=+34654205682;action=0 DEBUG: dialplan [dp_repl.c:699]: dp_translate_helper(): the copied attributes are: cli=+34654205682;action=0 DEBUG: <script>: SET_DST $var(dp_attrs):cli=+34654205682;action=0 action:0 - ci='1cf77cbb5baebdbd0e8635e1116aa713@PBX-2797.xpbx.foehn.co.uk' ``` #### Pre-Submission Checklist <!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply --> <!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above--> <!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list --> - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist: <!-- Go over all points below, and after creating the PR, tick the checkboxes that apply --> - [x] PR should be backported to stable branches - [x] Tested changes locally - [x] Related to issue #1952
#### Description
changing ``dp_replace_helper()`` to pass a NULL value indicating no output is needed ( no need to do subst ) to ``dp_translate_helper()``
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1953
-- Commit Summary --
* dialplan: don't try to translate on dp_match()
-- File Changes --
M src/modules/dialplan/dialplan.c (14) M src/modules/dialplan/dp_repl.c (3)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1953.patch https://github.com/kamailio/kamailio/pull/1953.diff
is that the correct behavior for ``dp_match()``? That is what I understand from the description ``` 6.2. dp_match(dpid, inval)
The function matches the input value 'inval' using the rules with dialplan id 'dpid'. If the rule that was applied has attributes, they are stored in the variable provided via the module parameter 'attrs_pvar'. ```
Merged #1953 into master.