(* COCANWIKI - a wiki written in Objective CAML.
* Written by Richard W.M. Jones <rich@merjis.com>.
* Copyright (C) 2004 Merjis Ltd.
- * $Id: cocanwiki_emailnotify.ml,v 1.1 2004/10/21 11:42:05 rich Exp $
+ * $Id: cocanwiki_emailnotify.ml,v 1.3 2004/10/23 15:00:16 rich Exp $
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
open Cgi
open Printf
+open Cocanwiki
+
(* This is where we coordinate email notification from various
* scripts which create or update the wiki.
*)
-let email_notify ~subject ~body (dbh : Dbi.connection) hostid =
- (* Is anyone listed for email notification at this host? *)
- let sth = dbh#prepare_cached "select email, name from email_notify
- where hostid = ?" in
- sth#execute [`Int hostid];
+let email_notify ~subject ~body ?user (dbh : Dbi.connection) hostid =
+ (* Get own userid, if we have it. Don't want to send email
+ * notification back to the person who changed the page. If
+ * we don't have a userid, set it to 0, because no real user can
+ * have userid = 0.
+ *)
+ let own_userid =
+ match user with
+ | None
+ | Some Anonymous -> 0
+ | Some (User (userid, _, _)) -> userid in
+
+ (* Send a change email to everyone who hasn't opted out using
+ * their preferences. This behaviour replaces the old
+ * 'email_notify' table. Don't send email to invited accounts
+ * who have not yet confirmed.
+ *)
+ let sth = dbh#prepare_cached "select email, name from users
+ where hostid = ? and id <> ? and email_notify
+ and email is not null
+ and invite is null" in
+ sth#execute [`Int hostid; `Int own_userid];
let to_addr = sth#map (function
| [`String email; `String name] ->
- name ^ " <" ^ email ^ ">"
- | [`String email; `Null] ->
- email
+ "\"" ^ name ^ "\" <" ^ email ^ ">"
| _ -> assert false) in
if to_addr <> [] then (
+ (* Get the from address of the user, if available. *)
+ let from =
+ match user with
+ | None
+ | Some Anonymous -> None
+ | Some (User (userid, _, _)) ->
+ let sth = dbh#prepare_cached "select email from users
+ where hostid = ? and id = ?" in
+ sth#execute [`Int hostid; `Int userid];
+
+ match sth#fetch1 () with
+ | [ `Null ] -> None
+ | [ `String email ] -> Some email
+ | _ -> assert false in
+
(* Prepare the body of the message. The assumption is that
* this takes time and database access, so we defer the creation
* of the body until we know that someone needs to be notified.
let subject = "Site notice: " ^ subject in
(* Send the email. *)
- Sendmail.send_mail ~subject ~to_addr ~body ()
+ Sendmail.send_mail ~subject ~to_addr ~body ?from ()
)