<p> Attention is currently required from: Joshua Colp, George Joseph. </p>
<p>N A <strong>uploaded patch set #10</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/17719">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pbx_validate: Add CLI commands to warn of dialplan problems<br><br>Adds a mechanism for applications to register a validation<br>callback function which can be used to preemptively validate<br>registered dialplan application calls and find syntax or usage<br>problems. While it is impossible to find every possible runtime<br>problem that could occur, many applications are characterized<br>by a particular type of issue which can be found in advanced<br>and can help users by finding these issues before they cause<br>problems at runtime. These validation callbacks may thus return<br>some false negatives, but they should not return any false<br>negatives.<br><br>Going forward, any module containing an application can register<br>a callback function which will be used for the preemptive<br>validation.<br><br>res_pbx_validate is a module which can be used to interact with<br>these validation callbacks by allowing users to run these scans<br>from the CLI. In the future, AMI commands or similar functionality<br>could also be included here without touching the core.<br><br>Additionally, some validation functions are added for common<br>applications, including Goto, GotoIf, GotoIfTime, Gosub,<br>GosubIf, Background, BackgroundDetect, ControlPlayback,<br>Playback, Read, and ReadExten.<br><br>ASTERISK-29828 #close<br><br>Change-Id: Iaf7a2c8a6982ee5adaae7f7aa63e311d0e9106e3<br>---<br>M apps/app_controlplayback.c<br>M apps/app_playback.c<br>M apps/app_read.c<br>M apps/app_readexten.c<br>M apps/app_stack.c<br>M apps/app_talkdetect.c<br>M configs/samples/cli_aliases.conf.sample<br>A doc/CHANGES-staging/pbx-cli.txt<br>M include/asterisk/module.h<br>M include/asterisk/pbx.h<br>M main/pbx.c<br>M main/pbx_app.c<br>M main/pbx_builtins.c<br>A res/res_pbx_validate.c<br>14 files changed, 910 insertions(+), 52 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/19/17719/10</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17719">change 17719</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/17719"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iaf7a2c8a6982ee5adaae7f7aa63e311d0e9106e3 </div>
<div style="display:none"> Gerrit-Change-Number: 17719 </div>
<div style="display:none"> Gerrit-PatchSet: 10 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>