<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/1589/">https://reviewboard.asterisk.org/r/1589/</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 and mjordan.</div>
<div>By jrose.</div>


<p style="color: grey;"><i>Updated Nov. 15, 2011, 3:20 p.m.</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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 slightly cleaner by only doing the urgent flag check once and setting a string instead.</pre>
  </td>
 </tr>
</table>


<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;">When sending a voicemail with multiple recipients and using the urgent option, all flags applied to the message are copied and should be used in the copy_message function.  Setup was done actually to check whether the message was urgent or not and then to use create_dirpath to set up the appropiate folders, but then a little further down the road make_dir would be invoked in a way that ignores this flag and set the todir to INBOX even when the Urgent flag was raised.  Because of this, the extra recipients would always have their voicemails copied to the inbox folder and if the message was marked as urgent, this would only work if the inbox folder had already been created by previous messages.

One of the bug reports suggests that we should only be sending to the urgent folder on the primary recipient unless the auto forwarding as urgent flag is set, but I maintain that this is not a question of forwarding, which is a whole other feature.  This is simply a message with multiple recipients, and all the flags applied to the message should be honored for this purpose, and the intent for that is pretty clear in the copy_message comment &quot;If urgent, copy to Urgent folder&quot;.</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 multiple recipients with the urgent option and without the urgent option.  In both cases, the files are sent to their respective folders.  Also checked to make sure it worked with unmade directory paths each time.</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-18245">ASTERISK-18245</a>, 

 <a href="https://issues.asterisk.org/jira/browse/ASTERISK-18246">ASTERISK-18246</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/branches/1.8/apps/app_voicemail.c <span style="color: grey">(345120)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1589/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>




  </div>
 </body>
</html>