Change subject line so it shows the name of the site.
[cocanwiki.git] / scripts / recently_visited.ml
index 64d0638..8522f6c 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: recently_visited.ml,v 1.3 2006/03/27 18:09:46 rich Exp $
+ * $Id: recently_visited.ml,v 1.5 2006/03/28 18:40:50 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
@@ -27,6 +27,8 @@ open Printf
 open Cocanwiki
 open Cocanwiki_template
 
+let limit = 50_l
+
 let run r (q : cgi) dbh hostid _ user =
   let template = get_template dbh hostid "recently_visited.html" in
 
@@ -35,17 +37,19 @@ let run r (q : cgi) dbh hostid _ user =
        User (userid, _, _, _) -> userid
       | _ -> assert false 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.hostid = p.hostid and rv.url = p.url
-                                 order by 3 desc" in
-  sth#execute [Some hostid; Some userid];
-
-  let table = sth#map (function [Some page; Some title; _] ->
-                        [ "page", Template.VarString page;
-                          "title", Template.VarString title ]
-                        | _ -> assert false) 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.hostid = p.hostid and rv.url = p.url
+      order by 3 desc
+      limit $limit" in
+
+  let table = List.map (
+    fun (page, title, _) ->
+      [ "page", Template.VarString page;
+       "title", Template.VarString title ]
+  ) rows in
   template#table "recently_visited" table;
 
   q#template template