- ask confirm by mail on host information update to avoid spam

6 files changed, 59 insertions, 18 deletions

diff --git a/root/html/includes/host_information.tt b/root/html/includes/host_information.tt
index 9093f98..b7b02f3 100644
--- a/root/html/includes/host_information.tt
+++ b/root/html/includes/host_information.tt
@@ -1,13 +1,17 @@
 <!-- $Id$ -->
 <div style="float: left">
-[% IF NOT host.readonly %]
+[% IF c.req.param('modify') AND NOT host.readonly %]
+    [% edit = 1 %]
+[% END %]
+
+[% IF edit %]
 <form action="[% action || c.uri_for() %]" method="POST">
 <input type="hidden" name="hostinfo" value="1">
 [% END %]
 <table border=0>
 <tr><th>Country</th>
 <td>
-[% IF host.readonly %]
+[% IF NOT edit %]
 [% c.model('Mirrors').country_info(host.country).name | html %]
 [% ELSE %]
 <select name="country">
@@ -22,38 +26,38 @@
 
 <tr><th>City</th>
 <td>
-[% IF host.readonly %]
-[% host.city | html %]
-[% ELSE %]
+[% IF edit %]
 <input type="text" name="city" value="[% host.city | html %]">
+[% ELSE %]
+[% host.city | html %]
 [% END %]
 </td>
 </tr>
 <tr><th>Latitude</th>
 <td>
-[% IF host.readonly %]
-[% host.latitude | html %]
-[% ELSE %]
+[% IF edit %]
 <input type="text" id="latitude" name="latitude" value="[% host.latitude | html %]">
+[% ELSE %]
+[% host.latitude | html %]
 [% END %]
 </td>
 </tr>
 <tr><th>Longitude</th>
 <td>
-[% IF host.readonly %]
-[% host.longitude | html %]
-[% ELSE %]
+[% IF edit %]
 <input type="text" id="longitude" name="longitude" value="[% host.longitude | html %]">
+[% ELSE %]
+[% host.longitude | html %]
 [% END %]
 </td>
 </tr>
-[% IF NOT host.readonly %]
+[% IF edit %]
 <tr><td colspan=2>Click on the map to update latitude and longitude</td></tr>
 [% END %]
 
 <tr><th>Synchronized from</th>
 <td>
-[% IF host.readonly %]
+[% IF NOT edit %]
 [% host.syncfrom | html %]
 [% ELSE %]
 [% FOREACH mirror = c.model('Mirrors').find_mirrors %]
@@ -74,7 +78,7 @@
 
 <tr><th>Approximated bandwidth</th>
 <td>
-[% IF host.readonly %]
+[% IF NOT edit %]
 [% c.model('Mirrors').bandwidth_name(host.bandwidth) | html %]
 [% ELSE %]
 <select name="bandwidth">
@@ -86,11 +90,27 @@
 [% END %]
 </td>
 </tr>
+[% IF edit AND needmail %]
+<tr>
+<td colspan=2>
+Please enter your e-mail address:<br>
+<input type="text" name="mail"><br>
+a mail will be be sent with the link to confirm<br>
+your update request.<br>
+</td>
+</tr>
+[% END %]
 </table>
-[% IF NOT host.readonly %]
+[% IF edit %]
 <input type="submit">
 <input type="reset">
 </form>
+[% ELSE %]
+[% IF NOT host.readonly %]
+<form action="[% action || c.uri_for() %]" method="POST">
+<input type="submit" name="modify" value="update information">
+</form>
+[% END %]
 [% END %]
 </div>
 
@@ -111,7 +131,7 @@
     var map = new google.maps.Map(document.getElementById("map_canvas"),
         myOptions);
 
-[% IF NOT host.readonly %] 
+[% IF edit %] 
     google.maps.event.addListener(map, 'click', function(event) {
         recordposition(event.latLng);
     });
@@ -120,6 +140,13 @@
         document.getElementById("latitude").value = location.lat();
         document.getElementById("longitude").value = location.lng();
     }
+[% ELSE %]
+    var myLatlng = new google.maps.LatLng([% host.latitude %],[% host.longitude %]);
+    var marker = new google.maps.Marker({
+      position: myLatlng,
+      map: map,
+      title:"[% host.hostname %]"
+    });
 [% END %]
     }
 
diff --git a/root/html/pages/mirrors/mirror.tt b/root/html/pages/mirrors/mirror.tt
index dad43c2..b27cfac 100644
--- a/root/html/pages/mirrors/mirror.tt
+++ b/root/html/pages/mirrors/mirror.tt
@@ -3,7 +3,8 @@
 <h2>Host information</h2>
 
 [% INCLUDE 'host_information.tt' 
-    action = c.uri_for(hostname)
+    action = c.uri_for(hostname),
+    needmail = 1
 %]
 
 <h2>URLs to the distribution</h2>
diff --git a/root/html/pages/new/new_host.tt b/root/html/pages/new/new_host.tt
index d73600c..ecf51b8 100644
--- a/root/html/pages/new/new_host.tt
+++ b/root/html/pages/new/new_host.tt
@@ -11,4 +11,4 @@
 <input type="hidden" name="url" value="[% uri | html %]">
 <input type="hidden" name="hostinfo" value="1">
 
-[% INCLUDE 'host_information.tt' %]
+[% INCLUDE 'host_information.tt' edit = 1 %]
diff --git a/root/html/pages/validate/error.tt b/root/html/pages/validate/error.tt
new file mode 100644
index 0000000..a5b88c4
--- /dev/null
+++ b/root/html/pages/validate/error.tt
@@ -0,0 +1 @@
+<p>No request found</p>
diff --git a/root/html/pages/validate/validate.tt b/root/html/pages/validate/validate.tt
new file mode 100644
index 0000000..d57e10e
--- /dev/null
+++ b/root/html/pages/validate/validate.tt
@@ -0,0 +1 @@
+<p>[% hostname | html %] formation has been updated</p>
diff --git a/root/mail/host_up_request.tt b/root/mail/host_up_request.tt
new file mode 100644
index 0000000..56b4e91
--- /dev/null
+++ b/root/mail/host_up_request.tt
@@ -0,0 +1,11 @@
+Hi,
+
+You (hopefully) asked to update information about Mageia mirror.
+
+To confirm the requst please visit:
+
+[% c.uri_for('/validate', reqid) %]
+
+If this is an unwanted action, please ignore this mail.
+
+Regards.