Non-existant files and images throw 404, instead of 500, errors.
[cocanwiki.git] / scripts / user_prefs.ml
index 41d2f9d..3c1f713 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: user_prefs.ml,v 1.1 2004/10/23 15:00:16 rich Exp $
+ * $Id: user_prefs.ml,v 1.2 2004/10/23 15:05:07 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
@@ -62,9 +62,9 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid {hostname=hostname} user =
       (* Is the new email address different from the one currently recorded
        * in the database?
        *)
-      let sth = dbh#prepare_cached "select ? = coalesce (email, '')
+      let sth = dbh#prepare_cached "select ? <> coalesce (email, '')
                                      from users where hostid = ? and id = ?" in
-      sth#execute [`Int hostid; `Int userid];
+      sth#execute [`String new_email; `Int hostid; `Int userid];
 
       let changed =
        match sth#fetch1 () with [ `Bool b ] -> b | _ -> assert false in
@@ -96,7 +96,8 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid {hostname=hostname} user =
   (* Commit and finish off. *)
   dbh#commit ();
 
-  ok ~title:"Preferences updated"
+  let buttons = [ ok_button "/_userprefs" ] in
+  ok ~title:"Preferences updated" ~buttons
     q ("Your user preferences were updated.  " ^
        if confirm_needed then
         ("Because you changed your email address, we have sent a " ^