[asterisk-commits] kmoore: branch 1.8 r337061 - in /branches/1.8: main/pbx.c tests/test_pbx.c
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Tue Sep 20 16:04:17 CDT 2011
Author: kmoore
Date: Tue Sep 20 16:04:11 2011
New Revision: 337061
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=337061
Log:
Make CANMATCH with the new pattern match engine behave more like the old one
When checking an extension for E_CANMATCH using the new extension matching
algorithm, an exact match was not returned as a possible match resulting in the
queue failing to allow a caller to exit on DTMF. This removes the requirement
that an extension be longer than acquired digits for an E_CANMATCH operation
to succeed.
(closes issue ASTERISK-18044)
Review: https://reviewboard.asterisk.org/r/1367/
Modified:
branches/1.8/main/pbx.c
branches/1.8/tests/test_pbx.c
Modified: branches/1.8/main/pbx.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/main/pbx.c?view=diff&rev=337061&r1=337060&r2=337061
==============================================================================
--- branches/1.8/main/pbx.c (original)
+++ branches/1.8/main/pbx.c Tue Sep 20 16:04:11 2011
@@ -1752,7 +1752,7 @@
return; /* the first match is all we need */ \
} \
} \
- } else if (p->next_char && !*(str + 1)) { \
+ } else if ((p->next_char || action == E_CANMATCH) && !*(str + 1)) { \
score->canmatch = 1; \
score->canmatch_exten = get_canmatch_exten(p); \
if (action == E_CANMATCH || action == E_MATCHMORE) { \
Modified: branches/1.8/tests/test_pbx.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/tests/test_pbx.c?view=diff&rev=337061&r1=337060&r2=337061
==============================================================================
--- branches/1.8/tests/test_pbx.c (original)
+++ branches/1.8/tests/test_pbx.c Tue Sep 20 16:04:11 2011
@@ -153,29 +153,35 @@
const struct exten_info *exten;
};
-static int test_exten(const struct pbx_test_pattern *test_pattern, struct ast_test *test)
+static int test_exten(const struct pbx_test_pattern *test_pattern, struct ast_test *test, int new_engine)
{
struct pbx_find_info pfi = { { 0 }, };
struct ast_exten *exten;
if (!(exten = pbx_find_extension(NULL, NULL, &pfi, test_pattern->context,
test_pattern->test_exten, test_pattern->priority, NULL,
test_pattern->test_cid, E_MATCH))) {
- ast_test_status_update(test, "Cannot find extension %s in context %s."
- "Test failed.\n", test_pattern->test_exten, test_pattern->context);
+ ast_test_status_update(test, "Cannot find extension %s in context %s with the %s pattern match engine. "
+ "Test failed.\n", test_pattern->test_exten, test_pattern->context, (new_engine ? "new" : "old"));
return -1;
}
if (strcmp(ast_get_extension_name(exten), test_pattern->exten->exten)) {
- ast_test_status_update(test, "Expected extension %s but got extension %s instead."
- "Test failed.\n", test_pattern->exten->exten, ast_get_extension_name(exten));
+ ast_test_status_update(test, "Expected extension %s but got extension %s instead with the %s pattern match engine. "
+ "Test failed.\n", test_pattern->exten->exten, ast_get_extension_name(exten), (new_engine ? "new" : "old"));
return -1;
}
if (test_pattern->test_cid && strcmp(ast_get_extension_cidmatch(exten), test_pattern->test_cid)) {
- ast_test_status_update(test, "Expected CID match %s but got CID match %s instead."
- "Test failed.\n", test_pattern->exten->cid, ast_get_extension_cidmatch(exten));
+ ast_test_status_update(test, "Expected CID match %s but got CID match %s instead with the %s pattern match engine. "
+ "Test failed.\n", test_pattern->exten->cid, ast_get_extension_cidmatch(exten), (new_engine ? "new" : "old"));
return -1;
}
- ast_test_status_update(test, "Successfully matched %s to exten %s in context %s\n",
- test_pattern->test_exten, test_pattern->exten->exten, test_pattern->context);
+ if (!ast_canmatch_extension(NULL, test_pattern->context, test_pattern->test_exten,
+ test_pattern->priority, test_pattern->test_cid)) {
+ ast_test_status_update(test, "Partial match failed for extension %s in context %s with the %s pattern match engine. "
+ "Test failed.\n", test_pattern->test_exten, test_pattern->context, (new_engine ? "new" : "old"));
+ return -1;
+ }
+ ast_test_status_update(test, "Successfully matched %s to exten %s in context %s with the %s pattern match engine\n",
+ test_pattern->test_exten, test_pattern->exten->exten, test_pattern->context, (new_engine ? "new" : "old"));
return 0;
}
@@ -185,7 +191,7 @@
enum ast_test_result_state res = AST_TEST_PASS;
static const char TEST_PATTERN[] = "test_pattern";
static const char TEST_PATTERN_INCLUDE[] = "test_pattern_include";
- int i;
+ int i, j;
/* The array of contexts to register for our test.
* To add more contexts, just add more rows to this array.
@@ -300,12 +306,15 @@
/* At this stage, the dialplan is built. Now we iterate over
* the tests array to attempt to find each of the specified
- * extensions.
- */
- for (i = 0; i < ARRAY_LEN(tests); ++i) {
- if (test_exten(&tests[i], test)) {
- res = AST_TEST_FAIL;
- break;
+ * extensions with the old and new pattern matching engines.
+ */
+ for (j = 0; j < 2; j++) {
+ pbx_set_extenpatternmatchnew(j);
+ for (i = 0; i < ARRAY_LEN(tests); ++i) {
+ if (test_exten(&tests[i], test, j)) {
+ res = AST_TEST_FAIL;
+ break;
+ }
}
}
More information about the asterisk-commits
mailing list