<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1524/">https://reviewboard.asterisk.org/r/1524/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Kevin Fleming and David Vossel.</div>
<div>By jrose.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Changes menuselect applications so that they can reorder member lists from category trees so that they get separated by support level. Also adds separators splitting the selection groups in an easy to visually parse manner.
Just applying this patch won't perfectly sort every item. Items that don't receive labels are dumped into an 'other' category, but appear in menuselect as having core support. Another review will be posted which cleans up support level tags so that these will appear in the right place.
Fixes are made against cmenuselect, nmenuselect, and gmenuselect (which turns out to still work after the latest round of changes).
One issue with gmenuselect is that the checkboxes for disabling/enabling members are still visible. Clicking on them doesn't do anything, but their appearance remains annoying. It's not really any kind of big problem though.
In addition to these, a couple long-standing bugs in the cmenuselect easteregg were fixed. One prevented the use of the 'q' key to return to the main menu. The other involves setting a certain value back to zero if the easteregg is ran multiple times in a single session.
I'm still unsure whether this change should be made against the menuselect in 1.8 or if it should just apply to later versions of Asterisk. The report's description wasn't sure either.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Made sure all existing modules made the list.
Made sure disabling/enabling separators wouldn't work.
Made sure modules being enabled did in fact enable those specific modules.
Made sure menuselect didn't produce unusual error messages.
This change could probably do with some more extensive testing, and I'm sure there are probably some quirks with the way I went about doing this.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-18268">ASTERISK-18268</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/menuselect.c <span style="color: grey">(943)</span></li>
<li>/trunk/menuselect.h <span style="color: grey">(943)</span></li>
<li>/trunk/menuselect_curses.c <span style="color: grey">(943)</span></li>
<li>/trunk/menuselect_gtk.c <span style="color: grey">(943)</span></li>
<li>/trunk/menuselect_newt.c <span style="color: grey">(943)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1524/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>