Changes done on the Mac.
[cocanwiki.git] / scripts / page.ml
index 9e63c07..b6a4185 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: page.ml,v 1.43 2006/03/27 18:09:46 rich Exp $
+ * $Id: page.ml,v 1.44 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
@@ -76,15 +76,13 @@ let run r (q : cgi) dbh hostid
   let template_404  = get_template dbh hostid "page_404.html" in
 
   (* Host-specific fields. *)
-  let sth = dbh#prepare_cached "select css is not null,
-                                       feedback_email is not null,
-                                       mailing_list, navigation
-                                  from hosts where id = ?" in
-  sth#execute [Some hostid];
+  let rows = PGSQL(dbh)
+    "select css is not null, feedback_email is not null, mailing_list, navigation
+       from hosts where id = $hostid" in
   let has_host_css, has_feedback_email, mailing_list, navigation =
-    match sth#fetch1 () with
-      | [ `Bool has_host_css; `Bool has_feedback_email; `Bool mailing_list;
-         `Bool navigation ] ->
+    match rows with
+      | [Some has_host_css, Some has_feedback_email,
+        mailing_list, navigation] ->
          has_host_css, has_feedback_email, mailing_list, navigation
       | _ -> assert false in
 
@@ -167,17 +165,17 @@ let run r (q : cgi) dbh hostid
 
   (* Check the templates table for extensions. *)
   let get_extension url =
-    let sth = dbh#prepare_cached "select extension from templates
-                                   where ? ~ url_regexp
-                                   order by ordering
-                                   limit 1" in
-    sth#execute [Some url];
-
     try
-      let name = sth#fetch1string () in
+      let name =
+       List.hd (
+         PGSQL(dbh) "select extension from templates
+                       where $url ~ url_regexp
+                       order by ordering
+                       limit 1"
+       ) in
       Some (List.assoc name !extensions)
     with
-       Not_found -> None
+      Not_found | ExtList.List.Empty_list -> None
   in
 
   (* This code generates ordinary pages. *)
@@ -241,24 +239,20 @@ let run r (q : cgi) dbh hostid
       match pageid with
          None -> []
        | Some pageid ->
-           let sth = dbh#prepare_cached
-                       "select ordering, sectionname, content, divname
-                           from contents where pageid = ? order by ordering" in
-           sth#execute [Some pageid];
-
-           sth#map
-             (function [Some ordering;
-                        (None | Some _) as sectionname;
-                        Some content;
-                        (None | Some _) as divname] ->
+           let rows = PGSQL(dbh)
+             "select ordering, sectionname, content, divname
+                 from contents where pageid = $pageid order by ordering" in
+
+           List.map
+             (fun (ordering, sectionname, content, divname) ->
                 let divname, has_divname =
                   match divname with
-                      None -> "", false
-                    | Some divname -> divname, true in
+                  | None -> "", false
+                  | Some divname -> divname, true in
                 let sectionname, has_sectionname =
                   match sectionname with
-                      None -> "", false
-                    | Some sectionname -> sectionname, true in
+                  | None -> "", false
+                  | Some sectionname -> sectionname, true in
                 let linkname = linkname_of_sectionname sectionname in
                 [ "ordering", Template.VarString (Int32.to_string ordering);
                   "has_sectionname",
@@ -269,8 +263,7 @@ let run r (q : cgi) dbh hostid
                     Template.VarString
                       (Wikilib.xhtml_of_content dbh hostid content);
                   "has_divname", Template.VarConditional has_divname;
-                  "divname", Template.VarString divname ]
-                | _ -> assert false) in
+                  "divname", Template.VarString divname ]) rows in
 
     (* Call an extension to generate the first section in this page? *)
     let sections =
@@ -302,15 +295,14 @@ let run r (q : cgi) dbh hostid
     if pageid <> None then (
       match user with
        | User (userid, _, _, _) ->
-           let sth = dbh#prepare_cached "delete from recently_visited
-                                           where hostid = ? and userid = ?
-                                             and url = ?" in
-           sth#execute [Some hostid; Some userid; Some page'];
-           let sth = dbh#prepare_cached
-                       "insert into recently_visited (hostid, userid, url)
-                         values (?, ?, ?)" in
-           sth#execute [Some hostid; Some userid; Some page'];
-           PGOCaml.commit dbh
+           PGSQL(dbh)
+             "delete from recently_visited
+                where hostid = $hostid and userid = $userid and url = $page'";
+           PGSQL(dbh)
+             "insert into recently_visited (hostid, userid, url)
+               values ($hostid, $userid, $page')";
+           PGOCaml.commit dbh;
+           PGOCaml.begin_work dbh;
        | _ -> ()
     );
 
@@ -330,24 +322,19 @@ let run r (q : cgi) dbh hostid
               * links here' section, and don't link to self.
               *)
              let not_urls = page' :: wlh_urls in
-             let limit = max_links - List.length wlh_urls in
-             let qs = Dbi.placeholders (List.length not_urls) in
-             let sth =
-               dbh#prepare_cached
-                 ("select rv.url, p.title, rv.visit_time
-                      from recently_visited rv, pages p
-                     where rv.hostid = ? and rv.userid = ?
-                       and rv.url not in " ^ qs ^ "
-                       and rv.hostid = p.hostid and rv.url = p.url
-                     order by 3 desc
-                     limit ?") in
-             let args = List.map (fun s -> Some s) not_urls in
-             sth#execute
-               ([Some hostid; Some userid] @ args @ [Some limit]);
-             sth#map
-               (function [Some url; Some title; _] ->
-                  url, title
-                  | _ -> assert false)
+             let limit = Int32.of_int (max_links - List.length wlh_urls) in
+             let rows =
+               PGSQL(dbh)
+                 "select rv.url, p.title, rv.visit_time
+                     from recently_visited rv, pages p
+                    where rv.hostid = $hostid and rv.userid = $userid
+                      and rv.url not in $@not_urls
+                      and rv.hostid = p.hostid and rv.url = p.url
+                    order by 3 desc
+                    limit $limit" in
+             List.map (
+               fun (url, title, _) -> url, title
+             ) rows
          | _ -> [] in
 
       (* Links to page. *)
@@ -417,61 +404,53 @@ let run r (q : cgi) dbh hostid
     match version with
       | None ->
          if allow_redirect then (
-           let sth =
-             dbh#prepare_cached
-               "select url, redirect, id, title, description,
-                        last_modified_date, css is not null
-                   from pages where hostid = ? and lower (url) = lower (?)" in
-           sth#execute [Some hostid; Some page];
-           (try
-              (match sth#fetch1 () with
-               | Some page' :: _ when page <> page' -> (* different case *)
-                   FPExternalRedirect page'
-               | [ _; None; Some id; Some title; Some description;
-                   `Timestamp last_modified_date; `Bool has_page_css ] ->
-                   FPOK (id, title, description, last_modified_date,
-                         has_page_css)
-               | _ :: Some redirect :: _ ->
-                   FPInternalRedirect redirect
-               | xs -> failwith (Dbi.sdebug xs))
-            with
-              Not_found -> FPNotFound)
+           let rows = PGSQL(dbh)
+             "select url, redirect, id, title, description,
+                      last_modified_date, css is not null
+                 from pages
+                where hostid = $hostid and lower (url) = lower ($page)" in
+           match rows with
+           | [page', _, _, _, _, _, _]
+               when page <> page' -> (* different case *)
+               FPExternalRedirect page'
+           | [ _, None, id, title, description,
+               last_modified_date, has_page_css ] ->
+               let has_page_css = Option.get has_page_css in
+               FPOK (id, title, description, last_modified_date,
+                     has_page_css)
+           | [_, Some redirect, _, _, _, _, _] ->
+               FPInternalRedirect redirect
+           | [] -> FPNotFound
+           | _ -> assert false
          ) else (* redirects not allowed ... *) (
-           let sth =
-             dbh#prepare_cached
-               "select id, title, description, last_modified_date,
-                        css is not null
-                   from pages where hostid = ? and url = ?" in
-           sth#execute [Some hostid; Some page];
-           (try
-              (match sth#fetch1 () with
-               | [ Some id; Some title; Some description;
-                   `Timestamp last_modified_date; `Bool has_page_css ] ->
-                   FPOK (id, title, description, last_modified_date,
-                         has_page_css)
-               | xs -> failwith (Dbi.sdebug xs))
-            with
-                Not_found -> FPNotFound)
+           let rows = PGSQL(dbh)
+             "select id, title, description, last_modified_date,
+                      css is not null
+                 from pages where hostid = $hostid and url = $page" in
+           match rows with
+           | [ id, title, description,
+               last_modified_date, has_page_css ] ->
+               let has_page_css = Option.get has_page_css in
+               FPOK (id, title, description, last_modified_date,
+                     has_page_css)
+           | [] -> FPNotFound
+           | _ -> assert false
          )
       | Some version ->
-           let sth =
-             dbh#prepare_cached
-               "select id, title, description, last_modified_date,
-                        css is not null
-                   from pages
-                  where hostid = ? and id = ? and
-                        (url = ? or url_deleted = ?)" in
-           sth#execute [Some hostid; Some version;
-                        Some page; Some page];
-           (try
-              (match sth#fetch1 () with
-               | [ Some id; Some title; Some description;
-                   `Timestamp last_modified_date; `Bool has_page_css ] ->
-                   FPOK (id, title, description, last_modified_date,
-                         has_page_css)
-               | xs -> failwith (Dbi.sdebug xs))
-            with
-                Not_found -> FPNotFound)
+         let rows = PGSQL(dbh)
+           "select id, title, description, last_modified_date,
+                    css is not null
+               from pages
+              where hostid = $hostid and id = $version and
+                    (url = $page or url_deleted = $page)" in
+         match rows with
+         | [ id, title, description,
+             last_modified_date, has_page_css ] ->
+             let has_page_css = Option.get has_page_css in
+             FPOK (id, title, description, last_modified_date,
+                   has_page_css)
+         | [] -> FPNotFound
+         | _ -> assert false
   in
 
   (* Here we deal with the complex business of redirects and versions. *)