X-Git-Url: http://git.annexia.org/?a=blobdiff_plain;f=scripts%2Fpage.ml;h=b6a4185237ae99410cfd922f7f2ba1c10a3410a7;hb=c912392e889fc28c6444b7e38cd92b9d0fd960a8;hp=9e63c07a9497b824464d42a76fcf29aea3c99e47;hpb=4a0ade944a434120218f2083d5ea7558b3e9cf08;p=cocanwiki.git diff --git a/scripts/page.ml b/scripts/page.ml index 9e63c07..b6a4185 100644 --- a/scripts/page.ml +++ b/scripts/page.ml @@ -1,7 +1,7 @@ (* COCANWIKI - a wiki written in Objective CAML. * Written by Richard W.M. Jones . * 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. *)