Port number for new services is guessed from protocol if not specified.
[irspy-moved-to-github.git] / web / htdocs / details / edit.mc
index ba46e68..e435624 100644 (file)
@@ -1,7 +1,7 @@
-%# $Id: edit.mc,v 1.22 2006-12-05 12:21:39 mike Exp $
+%# $Id: edit.mc,v 1.36 2007-07-17 13:04:45 mike Exp $
 <%args>
 $op
-$id => undef
+$id => undef ### should be extracted using utf8param()
 $update => undef
 </%args>
 <%doc>
@@ -32,66 +32,89 @@ duplicate ID.
 die "op = new but id defined" if $op eq "new" && defined $id;
 die "op != new but id undefined" if $op ne "new" && !defined $id;
 
-my $conn = new ZOOM::Connection("localhost:3313/IR-Explain---1", 0,
+my $conn = new ZOOM::Connection("localhost:8018/IR-Explain---1", 0,
                                user => "admin", password => "fruitbat",
                                elementSetName => "zeerex");
+
+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";
+    $r->param(port => $port);
+}
+
+my $newid;
+if (defined $protocol && $protocol ne "" &&
+    defined $host && $host ne "" &&
+    defined $port && $port ne "" &&
+    defined $dbname && $dbname ne "") {
+    $newid = irspy_make_identifier($protocol, $host, $port, $dbname);
+}
+
 my $rec = '<explain xmlns="http://explain.z3950.org/dtd/2.0/"/>';
-if (defined $id && ($op ne "copy" || !$update)) {
-    # Existing record
-    my $query = 'rec.id="' . cql_quote($id) . '"';
-    my $rs = $conn->search(new ZOOM::Query::CQL($query));
-    if ($rs->size() > 0) {
-       $rec = $rs->record(0);
-    } else {
-       ### Is this an error?  I don't think the UI will ever provoke it
-       print qq[<p class="error">(New ID specified.)</p>\n];
-       $id = undef;
-    }
 
-} else {
-    # No ID supplied -- this is a brand new record
-    my $host = $r->param("host");
-    my $port = $r->param("port");
-    my $dbname = $r->param("dbname");
-    if (!defined $host || $host eq "" ||
-       !defined $port || $port eq "" ||
-       !defined $dbname || $dbname eq "") {
+if (!defined $id) {
+    if (!$update) {
+       # About to enter data for a new record
+       # Nothing to do at this stage
+    } elsif (!defined $newid) {
+       # Tried to create new record but data is insufficient
        print qq[<p class="error">
-You must specify host, port and database name.</p>\n] if $update;
+               Please specify protocol, host, port and database name.</p>\n];
        undef $update;
     } else {
-       my $query = cql_target($host, $port, $dbname);
-       my $rs = $conn->search(new ZOOM::Query::CQL($query));
+       # Creating new record, all necessary data is present.  Check
+       # that the new record is not a duplicate of an existing one.
+       my $rs = $conn->search(new ZOOM::Query::CQL(cql_target($newid)));
        if ($rs->size() > 0) {
-           my $fakeid = xml_encode(uri_escape("$host:$port/$dbname"));
+           my $qnewid = xml_encode(uri_escape($newid));
            print qq[<p class="error">
-There is already
-<a href='?op=edit&amp;id=$fakeid'>a record</a>
-for this host, port and database name.
-</p>\n];
+               There is already
+               <a href='?op=edit&amp;id=$newid'>a record</a>
+               for this protocol, host, port and database name.
+               </p>\n];
            undef $update;
        }
     }
+} else {
+    # assert(defined $id);
+    # Copying or editing an existing record: fetch it for editing
+    my $query = cql_target($id);
+    my $rs = $conn->search(new ZOOM::Query::CQL($query));
+    if ($rs->size() > 0) {
+       $rec = $rs->record(0);
+    } else {
+       ### Is this an error?  I don't think the UI will ever provoke it
+       print qq[<p class="error">(New ID specified.)</p>\n];
+       $id = undef;
+    }
 }
 
 my $xc = irspy_xpath_context($rec);
 my @fields =
     (
-     [ protocol     => [ qw(Z39.50 SRW SRU SRW/U) ],
+     [ title        => 0, "Name", "e:databaseInfo/e:title",
+       qw() ],
+     [ country      => $m->comp("country-list.mc"),
+       "Country", "i:status/i:country" ],
+     [ protocol     => [ qw(Z39.50 SRW SRU) ],
        "Protocol", "e:serverInfo/\@protocol" ],
      [ host         => 0, "Host", "e:serverInfo/e:host" ],
      [ port         => 0, "Port", "e:serverInfo/e:port" ],
      [ dbname       => 0, "Database Name", "e:serverInfo/e:database",
        qw(e:host e:port) ],
-     [ type         => [ qw(Academic Public Corporate Special National Education Other) ],
+     [ type         => $m->comp("libtype-list.mc"),
        "Type of Library", "i:status/i:libraryType" ],
-     [ country      => 0, "Country", "i:status/i:country" ],
      [ username     => 0, "Username (if needed)", "e:serverInfo/e:authentication/e:user",
        qw() ],
      [ password     => 0, "Password (if needed)", "e:serverInfo/e:authentication/e:password",
        qw(e:user) ],
-     [ title        => 0, "Title", "e:databaseInfo/e:title",
-       qw() ],
      [ description  => 5, "Description", "e:databaseInfo/e:description",
        qw(e:title) ],
      [ author       => 0, "Author", "e:databaseInfo/e:author",
@@ -103,7 +126,200 @@ my @fields =
        qw(e:title e:description) ],
      [ history      => 5, "History", "e:databaseInfo/e:history",
        qw(e:title e:description) ],
-     [ language     => 0, "Language of Records", "e:databaseInfo/e:langUsage",
+     [ language     => [
+# This list was produced by feeding
+#      http://www.loc.gov/standards/iso639-2/ISO-639-2_values_8bits.txt
+# through the filter
+#      awk -F'|' '$3 {print$4}'
+# and shortening some of the longer names by hand
+                       "",
+                       "English",
+                       "Afar",
+                       "Abkhazian",
+                       "Afrikaans",
+                       "Akan",
+                       "Albanian",
+                       "Amharic",
+                       "Arabic",
+                       "Aragonese",
+                       "Armenian",
+                       "Assamese",
+                       "Avaric",
+                       "Avestan",
+                       "Aymara",
+                       "Azerbaijani",
+                       "Bashkir",
+                       "Bambara",
+                       "Basque",
+                       "Belarusian",
+                       "Bengali",
+                       "Bihari",
+                       "Bislama",
+                       "Bosnian",
+                       "Breton",
+                       "Bulgarian",
+                       "Burmese",
+                       "Catalan; Valencian",
+                       "Chamorro",
+                       "Chechen",
+                       "Chinese",
+                       "Church Slavic; Old Slavonic",
+                       "Chuvash",
+                       "Cornish",
+                       "Corsican",
+                       "Cree",
+                       "Czech",
+                       "Danish",
+                       "Divehi; Dhivehi; Maldivian",
+                       "Dutch; Flemish",
+                       "Dzongkha",
+                       "Esperanto",
+                       "Estonian",
+                       "Ewe",
+                       "Faroese",
+                       "Fijian",
+                       "Finnish",
+                       "French",
+                       "Western Frisian",
+                       "Fulah",
+                       "Georgian",
+                       "German",
+                       "Gaelic; Scottish Gaelic",
+                       "Irish",
+                       "Galician",
+                       "Manx",
+                       "Greek, Modern (1453-)",
+                       "Guarani",
+                       "Gujarati",
+                       "Haitian; Haitian Creole",
+                       "Hausa",
+                       "Hebrew",
+                       "Herero",
+                       "Hindi",
+                       "Hiri Motu",
+                       "Hungarian",
+                       "Igbo",
+                       "Icelandic",
+                       "Ido",
+                       "Sichuan Yi",
+                       "Inuktitut",
+                       "Interlingue",
+                       "Interlingua",
+                       "Indonesian",
+                       "Inupiaq",
+                       "Italian",
+                       "Javanese",
+                       "Japanese",
+                       "Kalaallisut; Greenlandic",
+                       "Kannada",
+                       "Kashmiri",
+                       "Kanuri",
+                       "Kazakh",
+                       "Khmer",
+                       "Kikuyu; Gikuyu",
+                       "Kinyarwanda",
+                       "Kirghiz",
+                       "Komi",
+                       "Kongo",
+                       "Korean",
+                       "Kuanyama; Kwanyama",
+                       "Kurdish",
+                       "Lao",
+                       "Latin",
+                       "Latvian",
+                       "Limburgan; Limburger; Limburgish",
+                       "Lingala",
+                       "Lithuanian",
+                       "Luxembourgish; Letzeburgesch",
+                       "Luba-Katanga",
+                       "Ganda",
+                       "Macedonian",
+                       "Marshallese",
+                       "Malayalam",
+                       "Maori",
+                       "Marathi",
+                       "Malay",
+                       "Malagasy",
+                       "Maltese",
+                       "Moldavian",
+                       "Mongolian",
+                       "Nauru",
+                       "Navajo; Navaho",
+                       "Ndebele, South; South Ndebele",
+                       "Ndebele, North; North Ndebele",
+                       "Ndonga",
+                       "Nepali",
+                       "Norwegian Nynorsk",
+                       "Norwegian Bokmål",
+                       "Norwegian",
+                       "Chichewa; Chewa; Nyanja",
+                       "Occitan (post 1500); Provençal",
+                       "Ojibwa",
+                       "Oriya",
+                       "Oromo",
+                       "Ossetian; Ossetic",
+                       "Panjabi; Punjabi",
+                       "Persian",
+                       "Pali",
+                       "Polish",
+                       "Portuguese",
+                       "Pushto",
+                       "Quechua",
+                       "Raeto-Romance",
+                       "Romanian",
+                       "Rundi",
+                       "Russian",
+                       "Sango",
+                       "Sanskrit",
+                       "Serbian",
+                       "Croatian",
+                       "Sinhala; Sinhalese",
+                       "Slovak",
+                       "Slovenian",
+                       "Northern Sami",
+                       "Samoan",
+                       "Shona",
+                       "Sindhi",
+                       "Somali",
+                       "Sotho, Southern",
+                       "Spanish; Castilian",
+                       "Sardinian",
+                       "Swati",
+                       "Sundanese",
+                       "Swahili",
+                       "Swedish",
+                       "Tahitian",
+                       "Tamil",
+                       "Tatar",
+                       "Telugu",
+                       "Tajik",
+                       "Tagalog",
+                       "Thai",
+                       "Tibetan",
+                       "Tigrinya",
+                       "Tonga (Tonga Islands)",
+                       "Tswana",
+                       "Tsonga",
+                       "Turkmen",
+                       "Turkish",
+                       "Twi",
+                       "Uighur; Uyghur",
+                       "Ukrainian",
+                       "Urdu",
+                       "Uzbek",
+                       "Venda",
+                       "Vietnamese",
+                       "Volapük",
+                       "Welsh",
+                       "Walloon",
+                       "Wolof",
+                       "Xhosa",
+                       "Yiddish",
+                       "Yoruba",
+                       "Zhuang; Chuang",
+                       "Zulu",
+                       ],
+       "Language of Records", "e:databaseInfo/e:langUsage",
        qw(e:title e:description) ],
      [ restrictions => 2, "Restrictions", "e:databaseInfo/e:restrictions",
        qw(e:title e:description) ],
@@ -116,7 +332,7 @@ my %fieldsByKey = map { ( $_->[0], $_) } @fields;
 my %data;
 foreach my $key ($r->param()) {
     next if grep { $key eq $_ } qw(op id update);
-    $data{$key} = $r->param($key);
+    $data{$key} = utf8param($r, $key);
 }
 my @changedFields = modify_xml_document($xc, \%fieldsByKey, \%data);
 if ($update && @changedFields) {
@@ -126,7 +342,8 @@ 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::_really_rewrite_record($conn, $xc->getContextNode(),
+                                       $op eq "edit" ? $id : undef);
 }
 
 </%perl>
@@ -138,7 +355,14 @@ if ($update && @changedFields) {
   Changed <% $nchanges %> field<% $nchanges == 1 ? "" : "s" %>:
   <% join(", ", map { xml_encode($_->[2]) } @changedFields) %>.
  </p>
+% return if $op eq "new";
 % }
+ <p>
+  Although anyone is allowed to add a new target, please note that
+  <b>you will not be able to edit the newly added target unless you
+  have administrator privileges</b>.  So please be sure that the
+  details are correct before submitting them.
+ </p>
  <form method="get" action="">
   <table class="fullrecord" border="1" cellspacing="0" cellpadding="5" width="100%">
 <%perl>
@@ -172,7 +396,9 @@ foreach my $ref (@fields) {
    <tr>
     <td align="right" colspan="2">
      <input type="submit" name="update" value="Update"/>
+% $op = "edit" if $op eq "new" && defined $update;
      <input type="hidden" name="op" value="<% xml_encode($op) %>"/>
+% $id = $newid if defined $newid;
 % if (defined $id) {
      <input type="hidden" name="id" value="<% xml_encode($id) %>"/>
 % }