X-Git-Url: http://git.iain.cx/?a=blobdiff_plain;f=lib%2Fcity.php;h=dd60e360899b5d9d8442f9ba5d06f5905a342b98;hb=f181270af10ecded4b40d9ad3f838216835eb5f4;hp=29f2c753312793dbce5ad4dbc563ac49f0297cdd;hpb=6fdbfe32798452ec1c0fdbda57edabb892518409;p=readifood.git

diff --git a/lib/city.php b/lib/city.php
index 29f2c75..dd60e36 100644
--- a/lib/city.php
+++ b/lib/city.php
@@ -18,10 +18,12 @@
     if (count($p)) {
       foreach ($p as $city) {
         echo "<br>\nCity: " . $city->getStrongLink(get_city_displayname($city));
-        printf(" <a class=\"small\" href=\"/area/in/%s/%d\">Areas</a>", urlencode($city->getName()), $city->getID());
-        printf(" <a class=\"small\" href=\"/contact/in/city/%s/%d\">Contacts</a>", urlencode($city->getName()), $city->getId());
-        printf(" <a class=\"small\" href=\"/donation/in/city/%s/%d\">Donations</a>", urlencode($city->getName()), $city->getId());
-        printf(" <a class=\"small\" href=\"/order/in/city/%s/%d\">Orders</a>", urlencode($city->getName()), $city->getId());
+        $n = $city->getName();
+        $i = $city->getId();
+        echo " " . get_small_link("Areas", "/area/in/city/%s/%d", $n, $i);
+        echo " " . get_small_link("Contacts", "/contact/in/city/%s/%d", $n, $i);
+        echo " " . get_small_link("Donations", "/donation/in/city/%s/%d", $n, $i);
+        echo " " . get_small_link("Orders", "/order/in/city/%s/%d", $n, $i);
         if (check_admin(1)) {
           echo " " . $city->getDeleteLink();
         }
@@ -34,7 +36,7 @@
   function show_new_city_form() {
     if (! check_admin(1)) return;
 
-    echo "<form method=\"POST\" action=\"" . $_SERVER['REQUEST_URI'] . "\">\n";
+    form("noprint standout");
     echo "<p>Add a new city <input name=\"city_name\">\n";
     echo "with postcode prefix <input name=\"city_postcode_area\" size=4 maxlength=4>\n";
     echo "<input type=\"submit\" value=\"Add\"></p>\n";
@@ -74,6 +76,17 @@
     return $city->getId();
   }
 
+  function confirm_delete_city($name, $id = null) {
+    if (! check_admin(1, "delete a city")) return;
+
+    if (isset($id)) $city = get_city_by_id($id);
+    else $city = get_city_by_name($name);
+    if (! $city) return false;
+
+    echo "<h3>Confirm deletion</h3>\n";
+    echo "<p>You must confirm deletion of city " . $city->getName() . ": " . $city->getDeleteLink(true) . "</p>\n";
+  }
+
   function delete_city($name, $id = null) {
     if (! check_admin(1, "delete a city")) return;
 
@@ -100,10 +113,12 @@
     if (! $city) return;
 
     echo "<p>City: <span class=\"strong\">" . get_city_displayname($city) . "</span>";
-    printf(" <a class=\"small\" href=\"/area/in/city/%s/%d\">Areas</a>", urlencode($city->getName), $city->getID());
-    printf(" <a class=\"small\" href=\"/contact/in/city/%s/%d\">Contacts</a>", urlencode($city->getName()), $city->getId());
-    printf(" <a class=\"small\" href=\"/donation/in/city/%s/%d\">Donations</a>", urlencode($city->getName()), $city->getId());
-    printf(" <a class=\"small\" href=\"/order/in/city/%s/%d\">Orders</a>", urlencode($city->getName()), $city->getId());
+    $n = $city->getName();
+    $i = $city->getId();
+    echo " " . get_small_link("Areas", "/area/in/city/%s/%d", $n, $i);
+    echo " " . get_small_link("Contacts", "/contact/in/city/%s/%d", $n, $i);
+    echo " " . get_small_link("Donations", "/donation/in/city/%s/%d", $n, $i);
+    echo " " . get_small_link("Orders", "/order/in/city/%s/%d", $n, $i);
     if (check_admin(1)) {
       echo " " . $city->getDeleteLink();
     }
@@ -124,6 +139,10 @@
   if (count($args)) {
     switch ($args[0]) {
       case "delete":
+        confirm_delete_city($name, $id);
+      break;
+
+      case "confirmdelete":
         delete_city($name, $id);
       break;
 
@@ -134,12 +153,9 @@
     }
   }
   else if (isset($name)) show_city($name, $id);
-  else {
-    show_cities(0, 10);
-    show_new_city_form();
-  }
+  else show_cities(0, 10);
+
+  if (count($parameters)) echo "<p class=\"noprint standout\">Show all <a href=\"/city\">cities</a></p>\n";
+  show_new_city_form();
 
-  # XXX: Format URL in branch case...
-  if (count($parameters))
-  echo "<p>Show all <a href=\"/city\">cities</a></p>\n";
 ?>