X-Git-Url: http://git.indexdata.com/?p=irspy-moved-to-github.git;a=blobdiff_plain;f=web%2Fhtdocs%2Fdetails%2Fedit.mc;h=e0403a4c21ba9b2a4faf139bb42f41cabb6b306a;hp=038923537463fc19cd97eadcf53da261f1c368f7;hb=4666eaaa660f6f1072acdab8677478cba42a32da;hpb=fad1a8886f1f255bcc19bb02883aba38545d2bcf

diff --git a/web/htdocs/details/edit.mc b/web/htdocs/details/edit.mc
index 0389235..e0403a4 100644
--- a/web/htdocs/details/edit.mc
+++ b/web/htdocs/details/edit.mc
@@ -1,4 +1,4 @@
-%# $Id: edit.mc,v 1.35 2007-07-16 11:56:14 mike Exp $
+%# $Id: edit.mc,v 1.40 2009-04-15 18:16:46 wosch Exp $
 <%args>
 $op
 $id => undef ### should be extracted using utf8param()
@@ -40,6 +40,15 @@ my $protocol = utf8param($r, "protocol");
 my $host = utf8param($r, "host");
 my $port = utf8param($r, "port");
 my $dbname = utf8param($r, "dbname");
+
+if ((!defined $port || $port eq "") &&
+    (defined $protocol && $protocol ne "")) {
+    # Port-guessing based on defaults for each protocol
+    $port = $protocol eq "Z39.50" ? 210 : 80;
+    warn "guessed port $port";
+    &utf8param($r, port => $port);
+}
+
 my $newid;
 if (defined $protocol && $protocol ne "" &&
     defined $host && $host ne "" &&
@@ -59,6 +68,16 @@ if (!defined $id) {
 	print qq[<p class="error">
 		Please specify protocol, host, port and database name.</p>\n];
 	undef $update;
+    } elsif ($host !~ /^\w+\.[\w.]*\w$/i) {
+	print qq[<p class="error">
+		This host name is not valid.</p>\n];
+	undef $update;
+	sleep 25;
+    } elsif ($port !~ /^\d*$/i) {
+	print qq[<p class="error">
+		This port number is not valid.</p>\n];
+	undef $update;
+	sleep 25;
     } else {
 	# Creating new record, all necessary data is present.  Check
 	# that the new record is not a duplicate of an existing one.
@@ -321,7 +340,7 @@ my @fields =
 # Update record with submitted data
 my %fieldsByKey = map { ( $_->[0], $_) } @fields;
 my %data;
-foreach my $key ($r->param()) {
+foreach my $key (&utf8param($r)) {
     next if grep { $key eq $_ } qw(op id update);
     $data{$key} = utf8param($r, $key);
 }
@@ -333,7 +352,7 @@ if ($update && @changedFields) {
 					     "e:metaInfo/e:dateModified" ] },
 				{ dateModified => isodate(time()) });
     die "Didn't set dateModified!" if !@x;
-    ZOOM::IRSpy::_really_rewrite_record($conn, $xc->getContextNode(),
+    ZOOM::IRSpy::_rewrite_zeerex_record($conn, $xc->getContextNode(),
 					$op eq "edit" ? $id : undef);
 }