<p> Attention is currently required from: N A, Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16070">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16070?tab=comments">Patch Set #1:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Whoops, missed this one.<br>So something like SetDTMFStore(direction,var,maxdigits) basically?<br>I thought this seemed more functiony, but I don't care either way and could certainly do that.<br>One consideration is this could be added via an AMI Setvar, or manipulated in other ways, presumably but that would not be possible with an application, since the channel needs to actually go there.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This is more like "taking a control action on a channel" rather than setting or getting a channel variable (especially since this function doesn't return anything) so I think an app is more appropriate.  To be honest, I think the "frame drop" function probably should have been an app as well but since "frame trace" was already a function it made more sense to be consistent.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for AMI, you could add an AMI command for this easily.  Take a look at app_senddtmf for a good basic example.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File funcs/func_dtmfstore.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16070/comment/940b0a60_85acd9b9">Patch Set #1, Line 62:</a> <code style="font-family:monospace,monospace">=CDR(digits)</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">This dictates that any digits dialed in the specified direction will by appended to this variable. So in this case, since I want them in a CDR record (custom column), I just specify this column and when the call is over, it will automatically be there with no further processing required.<br>A non-CDR variable could be used, but I think CDR variables would be the most useful case.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Ah, custom column.  It was the "digits" what was throwing me because I couldn't find that in the normal CDR. 😊</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16070/comment/08894d75_b62cb912">Patch Set #1, Line 168:</a> <code style="font-family:monospace,monospace">                     remove_dtmf_store(chan); /* reached max digit count, stop now */</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The digits go directly into the variable during the callback, so if maximum # of digits is reached (e.g., the database column only stores X digits), then we may as well get rid of this entirely at this point so it won't do anything useful for us afterwards. Destroying the framehook doesn't modify the collected digits because they are not stored as part of it, but in the specified variable.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Ah yeah, gotcha.  You're just disabling any further gathering of digits.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16070">change 16070</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/+/16070"/><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: I51aa93fc9507f7636ac44806c4420ce690423e6f </div>
<div style="display:none"> Gerrit-Change-Number: 16070 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </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-CC: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 24 Jun 2021 12:29:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>