Change subject line so it shows the name of the site.

[cocanwiki.git] / scripts / edit_user.ml

diff --git a/scripts/edit_user.ml b/scripts/edit_user.ml
index b46b905..bfbd191 100644 (file)
--- a/scripts/edit_user.ml
+++ b/scripts/edit_user.ml
@@ -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: edit_user.ml,v 1.9 2004/10/30 10:16:10 rich Exp $
+ * $Id: edit_user.ml,v 1.12 2006/03/27 19:10:29 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
@@ -28,42 +28,42 @@ open Cocanwiki
 open Cocanwiki_strings
 open Cocanwiki_ok
 
-let run r (q : cgi) (dbh : Dbi.connection) hostid _ self =
-  let userid = int_of_string (q#param "userid") in
+let run r (q : cgi) dbh hostid _ self =
+  let userid = Int32.of_string (q#param "userid") in
 
   (* Get the user's original name.  If we're going to change the
    * name, we need to do additional checks.
    *)
-  let sth = dbh#prepare_cached "select name from users
-                                 where hostid = ? and id = ?" in
-  sth#execute [`Int hostid; `Int userid];
-  let original_name = sth#fetch1string () in
+  let original_name =
+    List.hd (
+      PGSQL(dbh)
+       "select name from users where hostid = $hostid and id = $userid"
+    ) in
 
   let name = trim (q#param "name") in
 
   if original_name <> name then (
     if name = "" then (
       error ~back_button:true ~title:"Bad username"
-        q "The username you gave is empty.";
+        dbh hostid q "The username you gave is empty.";
       return ()
     );
 
     (* Check it's not a duplicate, then change it. *)
-    let sth = dbh#prepare_cached "select id from users
-                                   where hostid = ? and name = ?" in
-    sth#execute [`Int hostid; `String name];
+    let rows = PGSQL(dbh)
+      "select 1 from users where hostid = $hostid and name = $name" in
 
-    (try
-       sth#fetch1 ();
-       error ~back_button:true ~title:"Username already taken"
-        q ("That username has already been taken by another user.");
-       return ()
-     with
-        Not_found -> ());
+    (match rows with
+     | [Some 1l] ->
+        error ~back_button:true ~title:"Username already taken"
+          dbh hostid q
+          ("That username has already been taken by another user.");
+        return ()
+     | _ -> ()
+    );
 
-    let sth = dbh#prepare_cached "update users set name = ?
-                                   where hostid = ? and id = ?" in
-    sth#execute [`String name; `Int hostid; `Int userid]
+    PGSQL(dbh) "update users set name = $name
+                 where hostid = $hostid and id = $userid"
   );
 
   (* Change permissions. *)
@@ -78,30 +78,28 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ self =
   (match can_manage_users, self with
      | false, User (id, _, _, _) when id = userid ->
         error ~back_button:true ~title:"Remove manage users from self"
-          q ("You tried to remove 'Manage users' permission from yourself. "^
-             "You can't do this.  You'll have to do it from another "^
-             "user account.");
+          dbh hostid q
+          ("You tried to remove 'Manage users' permission from yourself. " ^
+           "You can't do this.  You'll have to do it from another " ^
+           "user account.");
         return ()
      | _ -> ());
 
-  let sth = dbh#prepare_cached "update users set
-                                       can_edit = ?, can_manage_users = ?,
-                                       can_manage_contacts = ?,
-                                       can_manage_site = ?,
-                                       can_edit_global_css = ?,
-                                       can_import_mail = ?
-                                 where hostid = ? and id = ?" in
-  sth#execute [`Bool can_edit; `Bool can_manage_users;
-              `Bool can_manage_contacts; `Bool can_manage_site;
-              `Bool can_edit_global_css; `Bool can_import_mail;
-              `Int hostid; `Int userid];
+  PGSQL(dbh)
+    "update users set
+       can_edit = $can_edit, can_manage_users = $can_manage_users,
+       can_manage_contacts = $can_manage_contacts,
+       can_manage_site = $can_manage_site,
+       can_edit_global_css = $can_edit_global_css,
+       can_import_mail = $can_import_mail
+     where hostid = $hostid and id = $userid";
 
   (* Finish up. *)
-  dbh#commit ();
+  PGOCaml.commit dbh;
 
   let buttons = [ ok_button "/_users" ] in
   ok ~buttons ~title:"Saved"
-    q "Changes were saved."
+    dbh hostid q "Changes were saved."
 
 let () =
   register_script ~restrict:[CanManageUsers] run