<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/1581/">https://reviewboard.asterisk.org/r/1581/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Sounds like the right fix.</pre>
<br />
<p>- wdoekes</p>
<br />
<p>On November 11th, 2011, 3:46 p.m., jrose wrote:</p>
<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 and mjordan.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated Nov. 11, 2011, 3:46 p.m.</i></p>
<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;">The reporter (Matt Jordan in this case) mentioned that when using the force name entry and force greeting options in voicemail that a new user can skip those by simply hanging up after password entry and then his password will be set and he never has to bother with those things. Technically, this is a pretty minor hole since there is also nothing stopping the user from simply being silent during the recordings of those as well, but that's aside the point...
This patch simply moves password setting to the end of the setup procedure. I thought about simply not setting the user's password until the end, but doing password entry at the start anyway, but that would do one of the following things...
A. Make the Allison voice prompt "password has been changed" a lie, if mentioned immediately after setting password while it was still at the start.
B. Make the voice prompt triggered at an awkward moment some time after the password was entered.
C. Made us remove the voice prompt because it would either be a lie or unintuitive.
By moving the password entry to the end, all of that is obviated and this fix is made really simple.</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;">Tested new user entry on voicemail with a number of possible stopping points.</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-18282">ASTERISK-18282</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>/branches/1.8/apps/app_voicemail.c <span style="color: grey">(344811)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1581/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>