This feature was a patch we probably should have looked at closer?<br><br>I reverted it and wrote a new one, please test as i am not able to atm this is a blind commit.<br><br>vm-mailto (default undef) &quot;the addr you want main email&quot;<br>
vm-notify-mailto (default same as vm-mailto) &quot;the addr you want notifications sent to&quot;<br>vm-email-all-messages (default false) &quot;send all messages to vm-mailto addr (attachment based on vm-attach-file param)&quot;<br>
vm-notify-email-all-messages (default false) &quot;send a notify email to vm-notify-mailto when a vm is left (never has attachment)&quot;<br>vm-keep-local-after-email (default true) when you email the main message also keep it locally as new vm for the phone.<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; when false means delete it as if it never happened and just email it.<br>vm-attach-file (default true) attach the audio file to the main email<br><br>the notify email will use the same template as the main email unless you define the new profile param &quot;notify-template-file&quot;<br>
<br><br><br><br><br><div class="gmail_quote">On Tue, Jun 3, 2008 at 1:25 PM, Peder @ NetworkOblivion &lt;<a href="mailto:peder@networkoblivion.com" target="_blank">peder@networkoblivion.com</a>&gt; wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Right, but if you don&#39;t have &quot;vm-email-all-messages&quot; and &quot;vm-mailto&quot;,<br>


you can&#39;t send a notify. &nbsp;In other words it is either email only, or<br>
email and notify. &nbsp;You can&#39;t just do notify.<br>
<div><br>
Anthony Minessale wrote:<br>
&gt; vm-mailto is just the value of your email address if you want to send or<br>
&gt; not which is more specific than email-addr<br>
&gt; vm-email-all-messages is the actual param to adjust the functionality.<br>
&gt;<br>
&gt;<br>
&gt; On Tue, Jun 3, 2008 at 12:39 PM, Peder @ NetworkOblivion<br>
</div><div><div></div><div>&gt; &lt;<a href="mailto:peder@networkoblivion.com" target="_blank">peder@networkoblivion.com</a> &lt;mailto:<a href="mailto:peder@networkoblivion.com" target="_blank">peder@networkoblivion.com</a>&gt;&gt; wrote:<br>


&gt;<br>
&gt; &nbsp; &nbsp; So I was messing with the voicemail to email and notify and ran into an<br>
&gt; &nbsp; &nbsp; &quot;issue&quot;. &nbsp;It appears that you have to have vm-email-all-messages set to<br>
&gt; &nbsp; &nbsp; true and have an address in vm-mailto to send a notify. &nbsp;This may or may<br>
&gt; &nbsp; &nbsp; not be the intention, but it just doesn&#39;t sound right to me. &nbsp;I should<br>
&gt; &nbsp; &nbsp; be able to send a notify without sending an email as well. &nbsp;I think I<br>
&gt; &nbsp; &nbsp; found the culprit at line 1926, it is:<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp;if (send_mail &amp;&amp; !switch_strlen_zero(email_vm) &amp;&amp;<br>
&gt; &nbsp; &nbsp; switch_file_exists(file_path, switch_core_session_get_pool(session)) ==<br>
&gt; &nbsp; &nbsp; SWITCH_STATUS_SUCCESS) {<br>
&gt;<br>
&gt;<br>
&gt; &nbsp; &nbsp; My thought was to instead say &quot;if you have email-all and an email, or<br>
&gt; &nbsp; &nbsp; you have notify&quot;, then you are ok, which I think the below will do:<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp;if (((send_mail &amp;&amp; !switch_strlen_zero(email_vm)) ||<br>
&gt; &nbsp; &nbsp; !switch_strlen_zero(email_vm_notify)) &amp;&amp; switch_file_exists(file_path,<br>
&gt; &nbsp; &nbsp; switch_core_session_get_pool(session)) == SWITCH_STATUS_SUCCESS) {<br>
&gt;<br>
&gt;<br>
&gt; &nbsp; &nbsp; The issue is that I kind of got lost in the logic below as to how I<br>
&gt; &nbsp; &nbsp; would change these lines. &nbsp;This sends an email with attach AND a notify<br>
&gt; &nbsp; &nbsp; if notify &nbsp;is set. &nbsp;If both are set, that is fine, but if only notify is<br>
&gt; &nbsp; &nbsp; set, I don&#39;t want to send an email. Line 2014:<br>
&gt;<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp; if (email_attach) {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm, from,<br>
&gt; &nbsp; &nbsp; header_string, body, file_path);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; } else {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm, from,<br>
&gt; &nbsp; &nbsp; header_string, body, NULL);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (!switch_strlen_zero(email_vm_notify)) {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm_notify, from,<br>
&gt; &nbsp; &nbsp; header_string, body, NULL);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG,<br>
&gt; &nbsp; &nbsp; &quot;Sending message to %s\n&quot;, email_vm);<br>
&gt;<br>
&gt;<br>
&gt; &nbsp; &nbsp; I wasn&#39;t sure if I needed to say &quot;if email_attach and send_mail and<br>
&gt; &nbsp; &nbsp; !zero(email_vm)&quot; and then the else would be an elseif of &quot;if<br>
&gt; &nbsp; &nbsp; email_attach and send_mail and zero(email_vm)&quot; to make sure that I don&#39;t<br>
&gt; &nbsp; &nbsp; match either of these with only notify set. &nbsp;Or is there an easier way<br>
&gt; &nbsp; &nbsp; to say it to make sure I don&#39;t miss something?<br>
&gt;<br>
&gt; &nbsp; &nbsp; Also, I think the log message would be better if there was one per<br>
&gt; &nbsp; &nbsp; if/then/if so that we can see if it is with an attach, without, or just<br>
&gt; &nbsp; &nbsp; a notify.<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;if (email_attach) {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm, from,<br>
&gt; &nbsp; &nbsp; header_string, body, file_path);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_log_printf(SWITCH_CHANNEL_LOG,<br>
&gt; &nbsp; &nbsp; SWITCH_LOG_DEBUG, &quot;Sending email with attachment to %s\n&quot;, email_vm);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; } else {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm, from,<br>
&gt; &nbsp; &nbsp; header_string, body, NULL);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_log_printf(SWITCH_CHANNEL_LOG,<br>
&gt; &nbsp; &nbsp; SWITCH_LOG_DEBUG, &quot;Sending email without attachment to %s\n&quot;, email_vm);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (!switch_strlen_zero(email_vm_notify)) {<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_simple_email(email_vm_notify, from,<br>
&gt; &nbsp; &nbsp; header_string, body, NULL);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_log_printf(SWITCH_CHANNEL_LOG,<br>
&gt; &nbsp; &nbsp; SWITCH_LOG_DEBUG, &quot;Sending email notify to %s\n&quot;, email_vm_notify);<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch_safe_free(body);<br>
&gt;<br>
&gt;<br>
&gt; &nbsp; &nbsp; And before anybody says it, I don&#39;t have cvs commit access and I am a<br>
&gt; &nbsp; &nbsp; rookie programmer and don&#39;t want to commit something that is totally<br>
&gt; &nbsp; &nbsp; f&#39;d up.<br>
&gt;<br>
&gt; &nbsp; &nbsp; Peder<br>
&gt;<br>
&gt; &nbsp; &nbsp; _______________________________________________<br>
&gt; &nbsp; &nbsp; Freeswitch-dev mailing list<br>
&gt; &nbsp; &nbsp; <a href="mailto:Freeswitch-dev@lists.freeswitch.org" target="_blank">Freeswitch-dev@lists.freeswitch.org</a><br>
</div></div>&gt; &nbsp; &nbsp; &lt;mailto:<a href="mailto:Freeswitch-dev@lists.freeswitch.org" target="_blank">Freeswitch-dev@lists.freeswitch.org</a>&gt;<br>
<div>&gt; &nbsp; &nbsp; <a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev</a><br>
&gt; &nbsp; &nbsp; UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-dev</a><br>
&gt; &nbsp; &nbsp; <a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; Anthony Minessale II<br>
&gt;<br>
&gt; FreeSWITCH <a href="http://www.freeswitch.org/" target="_blank">http://www.freeswitch.org/</a><br>
&gt; ClueCon <a href="http://www.cluecon.com/" target="_blank">http://www.cluecon.com/</a><br>
&gt;<br>
&gt; AIM: anthm<br>
&gt; <a href="mailto:MSN%3Aanthony_minessale@hotmail.com" target="_blank">MSN:anthony_minessale@hotmail.com</a><br>
</div>&gt; &lt;mailto:<a href="mailto:MSN%253Aanthony_minessale@hotmail.com" target="_blank">MSN%3Aanthony_minessale@hotmail.com</a>&gt;<br>
<div>&gt; GTALK/JABBER/<a href="mailto:PAYPAL%3Aanthony.minessale@gmail.com" target="_blank">PAYPAL:anthony.minessale@gmail.com</a><br>
</div>&gt; &lt;mailto:<a href="mailto:PAYPAL%253Aanthony.minessale@gmail.com" target="_blank">PAYPAL%3Aanthony.minessale@gmail.com</a>&gt;<br>
&gt; IRC: <a href="http://irc.freenode.net" target="_blank">irc.freenode.net</a> &lt;<a href="http://irc.freenode.net" target="_blank">http://irc.freenode.net</a>&gt; #freeswitch<br>
<div>&gt;<br>
&gt; FreeSWITCH Developer Conference<br>
&gt; <a href="mailto:sip%3A888@conference.freeswitch.org" target="_blank">sip:888@conference.freeswitch.org</a><br>
</div>&gt; &lt;mailto:<a href="mailto:sip%253A888@conference.freeswitch.org" target="_blank">sip%3A888@conference.freeswitch.org</a>&gt;<br>
<div>&gt; <a href="http://iax:guest@conference.freeswitch.org/888" target="_blank">iax:guest@conference.freeswitch.org/888</a><br>
</div>&gt; &lt;<a href="http://iax:guest@conference.freeswitch.org/888" target="_blank">http://iax:guest@conference.freeswitch.org/888</a>&gt;<br>
<div>&gt; <a href="mailto:googletalk%3Aconf%2B888@conference.freeswitch.org" target="_blank">googletalk:conf+888@conference.freeswitch.org</a><br>
</div>&gt; &lt;mailto:<a href="mailto:googletalk%253Aconf%252B888@conference.freeswitch.org" target="_blank">googletalk%3Aconf%2B888@conference.freeswitch.org</a>&gt;<br>
&gt; pstn:213-799-1400<br>
&gt;<br>
&gt;<br>
&gt; ------------------------------------------------------------------------<br>
<div><div></div><div>&gt;<br>
&gt; _______________________________________________<br>
&gt; Freeswitch-dev mailing list<br>
&gt; <a href="mailto:Freeswitch-dev@lists.freeswitch.org" target="_blank">Freeswitch-dev@lists.freeswitch.org</a><br>
&gt; <a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev</a><br>
&gt; UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-dev</a><br>
&gt; <a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<br>
_______________________________________________<br>
Freeswitch-dev mailing list<br>
<a href="mailto:Freeswitch-dev@lists.freeswitch.org" target="_blank">Freeswitch-dev@lists.freeswitch.org</a><br>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev</a><br>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-dev" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-dev</a><br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Anthony Minessale II<br><br>FreeSWITCH <a href="http://www.freeswitch.org/" target="_blank">http://www.freeswitch.org/</a><br>ClueCon <a href="http://www.cluecon.com/" target="_blank">http://www.cluecon.com/</a><br>

<br>AIM: anthm<br><a href="mailto:MSN%3Aanthony_minessale@hotmail.com" target="_blank">MSN:anthony_minessale@hotmail.com</a><br>GTALK/JABBER/<a href="mailto:PAYPAL%3Aanthony.minessale@gmail.com" target="_blank">PAYPAL:anthony.minessale@gmail.com</a><br>

IRC: <a href="http://irc.freenode.net" target="_blank">irc.freenode.net</a> #freeswitch<br><br>FreeSWITCH Developer Conference<br><a href="mailto:sip%3A888@conference.freeswitch.org" target="_blank">sip:888@conference.freeswitch.org</a><br>
<a href="http://iax:guest@conference.freeswitch.org/888" target="_blank">iax:guest@conference.freeswitch.org/888</a><br>
<a href="mailto:googletalk%3Aconf%2B888@conference.freeswitch.org" target="_blank">googletalk:conf+888@conference.freeswitch.org</a><br>pstn:213-799-1400