X-Git-Url: http://git.indexdata.com/?p=irspy-moved-to-github.git;a=blobdiff_plain;f=web%2Fhtdocs%2Fdetails%2Ffound.mc;h=dc7caeebff04aec40e3edea7cc5d6b18d90e58d8;hp=4bdf8d697eab7d6d44849d90e7d3b5b00125278b;hb=c1c34528dbc95608f3dd6f6e32ab7dc97e53699e;hpb=383ee7cb98db086570669da52c86f2b37f29599d

diff --git a/web/htdocs/details/found.mc b/web/htdocs/details/found.mc
index 4bdf8d6..dc7caee 100644
--- a/web/htdocs/details/found.mc
+++ b/web/htdocs/details/found.mc
@@ -1,9 +1,5 @@
-%# $Id: found.mc,v 1.11 2006-09-25 16:52:30 mike Exp $
+%# $Id: found.mc,v 1.30 2007-05-03 14:15:56 mike Exp $
 <%once>
-use XML::LibXML;
-use XML::LibXML::XPathContext;
-use URI::Escape;
-
 sub print_navlink {
     my($params, $cond, $caption, $skip) = @_;
 
@@ -23,18 +19,40 @@ sub navlink {
     return $url;
 }
 
+# Identical to the same-named function in full.mc
+# So maybe this should go into IRSpy::Utils.pm?
+#
+sub calc_reliability {
+    my($xc) = @_;
+
+    my @allpings = $xc->findnodes("i:status/i:probe");
+    my $nall = @allpings;
+    return "[untested]" if $nall == 0;
+    my @okpings = $xc->findnodes('i:status/i:probe[@ok = "1"]');
+    my $nok = @okpings;
+    return "$nok/$nall = " . int(100*$nok/$nall) . "%";
+}
+
+
+# Just make this once; forge the connection on first use
+our $conn = undef;
 </%once>
 <%perl>
-my %params = map { ( $_, $r->param($_)) } grep { $r->param($_) } $r->param();
-my $query = "";
-foreach my $key (keys %params) {
-    next if $key =~ /^_/;
-    my $val = $params{$key};
-    next if $val eq "";
-    $query .= " and " if $query ne "";
-    $query .= "$key = ($val)";
+my %params = map { ( $_, utf8param($r, $_)) } grep { $r->param($_) } $r->param();
+my $query;
+if ($params{_query}) {
+    $query = $params{_query};
+} else {
+    $query = "";
+    foreach my $key (keys %params) {
+	next if $key =~ /^_/;
+	my $val = $params{$key};
+	next if $val eq "";
+	$query .= " and " if $query ne "";
+	$query .= "$key = ($val)";
+    }
 }
-$query = 'cql.allRecords=x' if $query eq "";
+$query = 'cql.allRecords=1' if $query eq "";
 
 my $sort = $params{"_sort"};
 if ($sort) {
@@ -48,14 +66,23 @@ if ($sort) {
     $query .= " 0";
 }
 
-### We can think about keeping the Connection object open to re-use
-# for multiple requests, but that may not get us much.  Same applies
-# for the XML parser.
-my $conn = new ZOOM::Connection("localhost:3313/IR-Explain---1");
-$conn->option(elementSetName => "zeerex");
-my $parser = new XML::LibXML();
+my $tried_to_open = 0;
+if (!defined $conn) {
+  OPEN:
+    $conn = new ZOOM::Connection("localhost:8018/IR-Explain---1");
+    $conn->option(elementSetName => "zeerex");
+}
+
+my $rs;
+eval { $rs = $conn->search(new ZOOM::Query::CQL($query)) };
+if ($@ && ref $@ && $@->isa('ZOOM::Exception') &&
+    $@->code() == ZOOM::Error::CONNECTION_LOST && !$tried_to_open) {
+    $tried_to_open = 1;
+    goto OPEN;
+} elsif ($@) {
+    die $@;
+}
 
-my $rs = $conn->search(new ZOOM::Query::CQL($query));
 my $n = $rs->size();
 
 my $skip = $params{"_skip"} || 0;
@@ -65,7 +92,10 @@ my $first = $skip+1;
 my $last = $first+$count-1;
 $last = $n if $last > $n;
 </%perl>
-     <h2><% xml_encode($query) %></h2>
+     <form method="get" action=""><p>
+      <input type="text" name="_query" size="60" value="<% xml_encode($query) %>"/>
+      <input type="submit" name="_search" value="Search"/>
+     </p></form>
      <p>
 % if ($n == 0) {
       No matches
@@ -85,7 +115,8 @@ print_navlink(\%params, $last < $n, "Next", $skip+$count);
       <tr class="thleft">
        <th>#</th>
        <th>Title</th>
-       <th>Author</th>
+       <th>Reliability <& /help/link.mc, help => "info/reliability" &>
+       </th>
        <th>Host</th>
        <th>Port</th>
        <th>DB</th>
@@ -95,33 +126,32 @@ print_navlink(\%params, $last < $n, "Next", $skip+$count);
 % my @ids;
 % foreach my $i ($first .. $last) {
 <%perl>
-my $rec = $rs->record($i-1);
-my $xml = $rec->render();
-my $doc = $parser->parse_string($xml);
-my $root = $doc->getDocumentElement();
-my $xc = XML::LibXML::XPathContext->new($root);
-$xc->registerNs(e => 'http://explain.z3950.org/dtd/2.0/');
-my $title = $xc->find("e:databaseInfo/e:title");
-my $author = $xc->find("e:databaseInfo/e:author");
+my $xc = irspy_xpath_context($rs->record($i-1));
+my $title = $xc->find("e:databaseInfo/e:title") || "[UNTITLED]";
+my $reliability = calc_reliability($xc);
 my $host = $xc->find("e:serverInfo/e:host");
 my $port = $xc->find("e:serverInfo/e:port");
 my $db = $xc->find("e:serverInfo/e:database");
-my $id = $xc->find("concat(e:serverInfo/e:host, ':',
-                           e:serverInfo/e:port, '/',
-                           e:serverInfo/e:database)");
+my $id = irspy_record2identifier($xc);
 push @ids, $id;
 </%perl>
       <tr style="background: <% ($i % 2) ? '#ffffc0' : 'white' %>">
        <td><% $i %></td>
-       <td><% xml_encode($title) %></td>
-       <td><% xml_encode($author) %></td>
-       <td><% xml_encode($host) %></td>
-       <td><% xml_encode($port) %></td>
-       <td><% xml_encode($db) %></td>
-       <td><a href="<% xml_encode("/check.html?id=" . uri_escape($id))
-	%>">[Test]</a></td>
-       <td><a href="<% xml_encode("/raw.html?id=" . uri_escape($id))
-	%>">[Raw]</a></td>
+       <td><a href="<% xml_encode("/full.html?id=" . uri_escape($id))
+		%>"><% xml_encode($title) %></a></td>
+       <td><% xml_encode($reliability, "", { nbsp => 1 }) %></td>
+       <td><% xml_encode($host, "") %></td>
+       <td><% xml_encode($port, "") %></td>
+       <td><% xml_encode($db, "") %></td>
+       <td>
+	<a href="<% xml_encode("/admin/check.html?id=" . uri_escape($id))
+		%>" title="Test this target">Test</a
+	>&nbsp;<a href="<% xml_encode("/admin/edit.html?op=edit&id=" .
+		uri_escape($id))
+		%>" title="Edit this target's record">Edit</a
+	>&nbsp;<a href="<% xml_encode("/raw.html?id=" . uri_escape($id))
+		%>" title="Raw XML record">XML</a>
+       </td>
       </tr>
 % }
      </table>
@@ -130,8 +160,12 @@ print_navlink(\%params, $skip > 0, "Prev", $count < $skip ? $skip-$count : 0);
 print_navlink(\%params, $last < $n, "Next", $skip+$count);
 </%perl>
      <p>
-      <a href="<% "/check.html?" .
+      <a href="<% "/admin/check.html?" .
 	xml_encode(join("&", map { "id=" . uri_escape($_) } @ids))
 	%>">[Test all targets on this list]</a>
      </p>
+     <p>
+      <a href="<% "/stats.html?query=" . xml_encode(uri_escape($query))
+	%>">[Statistics for targets on this list]</a>
+     </p>
 % }