(* 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.2 2004/10/21 19:54:29 rich Exp $
+ * $Id: cocanwiki_emailnotify.ml,v 1.8 2006/04/08 14:46:24 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 ExtList
+
open Cocanwiki
(* This is where we coordinate email notification from various
* scripts which create or update the wiki.
*)
-let email_notify ~subject ~body ?user (dbh : Dbi.connection) hostid =
+let email_notify ~subject ~body ?user dbh 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.
+ *
+ * Also get the user's email address.
*)
- let own_userid =
+ let own_userid, from =
match user with
| None
- | Some Anonymous -> 0
- | Some (User (userid, _, _)) -> userid in
+ | Some Anonymous -> 0l, None
+ | Some (User (userid, _, _, prefs)) -> userid, prefs.email in
(* Send a change email to everyone who hasn't opted out using
* their preferences. This behaviour replaces the old
- * 'email_notify' table.
+ * '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" in
- sth#execute [`Int hostid; `Int own_userid];
+ let rows = PGSQL(dbh)
+ "select name, email
+ from users
+ where hostid = $hostid and id <> $own_userid and email_notify
+ and email is not null and invite is null" in
+ let to_addr = List.filter_map (
+ function
+ | (name, Some email) ->
+ Some ("\"" ^ name ^ "\" <" ^ email ^ ">")
+ | (name, None) -> None
+ ) rows in
- let to_addr = sth#map (function
- | [`String email; `String name] ->
- "\"" ^ name ^ "\" <" ^ email ^ ">"
- | _ -> assert false) in
+ (* Get the canonical hostname. *)
+ let canonical_hostname = List.hd (
+ PGSQL(dbh)
+ "select canonical_hostname from hosts where id = $hostid"
+ ) 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 body = body () in
- let subject = "Site notice: " ^ subject in
+ (* Add the opt-out signature to the body. *)
+ let body = body ^ "\n\n-- \n" ^
+ "To stop receiving further emails:\n" ^
+ "(1) Go to http://" ^ canonical_hostname ^ "/_userprefs\n" ^
+ "(2) Uncheck 'Receive email notifications'\n" ^
+ "(3) Press 'Update Preferences'\n" in
+
+ let subject = canonical_hostname ^ ": " ^ subject in
(* Send the email. *)
- Sendmail.send_mail ~subject ~to_addr ~body ?from ()
+ Sendmail.send_mail ~subject ~to_addr ?from body
)