<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: Code-Review+1</p><p style="white-space: pre-wrap; word-wrap: break-word;">What was the behavior if more than 1 row was returned before this patch? It looks like we were adding duplicate variables to the list. If so, that was probably not a good thing so I'd make this a bug fix, not an improvement. In that case, you could cherry-pick this to the 16 and 18 branches and change the ASTERISK issue to a bug.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yep, you are right. It was adding duplicate variables to the list with different values. I've corrected the ASTERISK issue.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15446">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15446">change 15446</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/+/15446"/><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: If5a6d4b1072ea2e6e89059b21139d554a74b34f5 </div>
<div style="display:none"> Gerrit-Change-Number: 15446 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Boris P. Korzun <drtr0jan@yandex.ru> </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-Comment-Date: Wed, 24 Feb 2021 11:19:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>