Changes done on the Mac.
[cocanwiki.git] / scripts / mail_import.ml
index 0cd7a67..f609062 100644 (file)
@@ -1,7 +1,7 @@
 (* COCANWIKI - a wiki written in Objective CAML.
  * Written by Richard W.M. Jones <rich@merjis.com>.
  * Copyright (C) 2004 Merjis Ltd.
- * $Id: mail_import.ml,v 1.4 2004/10/12 10:00:38 rich Exp $
+ * $Id: mail_import.ml,v 1.11 2006/03/28 13:20:00 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
@@ -36,13 +36,14 @@ open Cocanwiki_template
 open Cocanwiki_date
 open Cocanwiki_strings
 open Cocanwiki_pages
+open Cocanwiki_mail
 
 let irt_re = Pcre.regexp "<.*?>"
 let ws_re = Pcre.regexp "\\s+"
 let comma_re = Pcre.regexp "\\s*,\\s*"
 let lines_re = Pcre.regexp "\\r?\\n"
 
-let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
+let run r (q : cgi) dbh hostid _ user =
   let hdr_template = get_template dbh hostid "mail_import_header.txt" in
 
   (* Overwrite old messages? *)
@@ -58,7 +59,7 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
     with
        Not_found ->
          error ~back_button:true ~title:"No message"
-           q "No message was uploaded.";
+           dbh hostid q "No message was uploaded.";
          return () in
 
   (* Parse the message. *)
@@ -85,27 +86,31 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
    *)
   if date = "" || inet_message_id = "" then (
     error ~back_button:true ~title:"Headers missing"
-      q "Date or Message-ID header missing.  Cannot handle this message. ";
+      dbh hostid q
+      "Date or Message-ID header missing.  Cannot handle this message. ";
     return ()
   );
 
+  (* Can't handle funny characters in subject lines - remove them. *)
+  let subject = String.map (fun c ->
+                             if Char.code c < 32 then ' ' else c) subject in
+
   (* Parse the date field. *)
-  let date, time =
+  let message_date =
     try
       let date = Netdate.parse date in
-      let date, time =
-       { Dbi.year = date.Netdate.year;
-         Dbi.month = date.Netdate.month;
-         Dbi.day = date.Netdate.day; },
-       { Dbi.hour = date.Netdate.hour;
-         Dbi.min = date.Netdate.minute;
-         Dbi.sec = date.Netdate.second;
-         Dbi.microsec = 0;
-         Dbi.timezone = Some (date.Netdate.zone / 60); } in
-      date, time
+      let cal = Calendar.make
+       date.Netdate.year
+       date.Netdate.month
+       date.Netdate.day
+       date.Netdate.hour
+       date.Netdate.minute
+       date.Netdate.second in
+      let tz = Time_Zone.UTC_Plus (date.Netdate.zone / 60) in
+      cal, tz
     with
-       Invalid_argument _ ->
-         failwith ("cannot parse date: " ^ date) in
+      Invalid_argument _ ->
+       failwith ("cannot parse date: " ^ date) in
 
   (* Find the first thing in the In-Reply-To field which looks like a
    * message ID.
@@ -143,41 +148,38 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
    * else 'None' if this is a never-seen-before message.
    *)
   let overwrite =
-    let sth = dbh#prepare_cached "select id from messages
-                                   where hostid = ? and inet_message_id = ?" in
-    sth#execute [`Int hostid; `String inet_message_id];
-    try
-      let id = sth#fetch1int () in
-      if not overwrite then (
-       ok ~title:"Message exists"
-          q "Message already imported";
-       return ()
-      );
-      Some id
-    with
-       Not_found -> None in
+    let rows = PGSQL(dbh)
+      "select id from messages
+        where hostid = $hostid and inet_message_id = $inet_message_id" in
+    match rows with
+    | [id] ->
+       if not overwrite then (
+         ok ~title:"Message exists"
+            dbh hostid q "Message already imported";
+         return ()
+       );
+       Some id
+    | [] -> None
+    | _ -> assert false in
 
   (* Save all of this in the database. *)
   let msgid =
     match overwrite with
        None ->                         (* Never-seen-before message. *)
-         let sth =
-           dbh#prepare_cached
-             "insert into messages (hostid, subject, inet_message_id,
-                 message_date) values (?, ?, ?, ?)" in
-         sth#execute [`Int hostid; `String subject; `String inet_message_id;
-                      `Timestamp (date, time)];
-         let msgid = sth#serial "messages_id_seq" in
-
-         let sth =
-           dbh#prepare_cached
-             "insert into msg_references (message_id, inet_message_id,
-                 ordering) values (?, ?, ?)" in
+         PGSQL(dbh)
+           "insert into messages (hostid, subject, inet_message_id,
+               message_date)
+             values ($hostid, $subject, $inet_message_id, $message_date)";
+         let msgid = PGOCaml.serial4 dbh "messages_id_seq" in
+
          let ordering = ref 0 in
          List.iter (fun inet_message_id ->
-                      incr ordering; let ordering = !ordering in
-                      sth#execute [`Int msgid; `String inet_message_id;
-                                   `Int ordering]) references;
+                      incr ordering; let ordering = Int32.of_int !ordering in
+                      PGSQL(dbh)
+                        "insert into msg_references (message_id,
+                            inet_message_id, ordering)
+                          values ($msgid, $inet_message_id, $ordering)"
+                   ) references;
 
          msgid
 
@@ -187,7 +189,10 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
           *)
          msgid in
 
-  (* The message is referred to by a unique title: *)
+  (* The message is referred to by a unique title.
+   * NB. Do not change this unique title - it is also used during thread
+   * indexing.
+   *)
   let title = sprintf "Mail/%s (%d)" subject msgid in
 
   (* Choose a suitable URL. *)
@@ -267,12 +272,11 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
       (* Get all the titles from the database! We're going to exclude
        * mail messages from this.
        *)
-      let sth =
-       dbh#prepare_cached
-         "select lower (title) from pages where hostid = ?
+      let links =
+       PGSQL(dbh)
+         "select lower (title) from pages where hostid = $hostid
               and url is not null and title not like 'Mail/%'" in
-      sth#execute [`Int hostid];
-      let links = sth#map (function [`String s] -> s | _ -> assert false) in
+      let links = List.map Option.get links in
 
       (* This code cannot find titles which are split across multiple lines.
        * XXX
@@ -290,7 +294,7 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
               try
                 String.find (String.lowercase str) sub_lc
               with
-                  String.Invalid_string -> -1
+                  Invalid_string -> -1
             in
             let rec loop line = function
                 [] -> line
@@ -312,9 +316,7 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
             loop line links) lines in
 
       let lines = List.map trim lines in
-      let lines =
-       List.map (fun line -> if line <> "" then line ^ " <br>" else "")
-         lines in
+      let lines = List.map (fun line -> line ^ "<br>") lines in
 
       String.concat "\n" lines
     in
@@ -348,12 +350,16 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ user =
    *)
   save_page dbh hostid ~user ~r model;
 
+  (* Rebuild threads? *)
+  if rebuild then
+    thread_mail dbh hostid ~user ~r date.Dbi.year date.Dbi.month;
+
   (* Commit to the database. *)
-  dbh#commit ();
+  PGOCaml.commit dbh;
 
   (* Finish off. *)
   ok ~title:"Imported"
-    q ("Message " ^ inet_message_id ^ " was imported.")
+    dbh hostid q ("Message " ^ inet_message_id ^ " was imported.")
 
 let () =
   register_script ~restrict:[CanImportMail] run