Fix IP ban handling and add delete function.

lib/admin/Controller/bans.pm

@@ -41,6 +41,22 @@ sub users : Chained('base') PathPart('users') Args(0) {
     $c->stash->{banned_users} = $banned_users;
 }
 
+sub unban_user : Chained('base') PathPart('unban_user')  Args(0) {
+    my ($self, $c) = @_;
+
+    my $user = $c->request->params->{'user'} || '';
+
+    $c->model('Provisioning')->call_prov(
+        $c,
+        'voip',
+        'unban_user',
+        { user => $user },
+        undef,
+    );
+
+    $c->response->redirect("/bans/users");
+}
+
 sub ips : Chained('base') PathPart('ips') Args(0) {
     my ($self, $c) = @_;
     
@@ -58,5 +74,21 @@ sub ips : Chained('base') PathPart('ips') Args(0) {
     $c->stash->{banned_ips} = $banned_ips;
 }
 
+sub unban_ip : Chained('base') PathPart('unban_ip')  Args(0) {
+    my ($self, $c) = @_;
+
+    my $user = $c->request->params->{'ip'} || '';
+
+    $c->model('Provisioning')->call_prov(
+        $c,
+        'voip',
+        'unban_ip',
+        { ip => $ip },
+        undef,
+    );
+
+    $c->response->redirect("/bans/ips");
+}
+
 # Ends, some people will rob their mothers for the ends ...
 1

root/tt/bans_ips.tt

@@ -14,10 +14,20 @@
 <ul class="cleanlist">
     
     <li class="ui-state-default">
-        <div class="span-3">IP</div>
+        <div class="span-3">IP Address</div>
     </li>
     [% FOREACH ip IN banned_ips %]
-        <li class="ui-state-default">[% ip.ip %]</li>
+        <li class="ui-state-default">
+          <div class="span-3">[% ip %]</div>
+          [% UNLESS Catalyst.session.admin.read_only %]
+            <div class="prepend-12 span-1 last">
+              <form action="/bans/unban_ip" method="POST">
+                <input type="hidden" name="ip" value="[% ip %]"/>
+                <button class="button-delete">Delete</button>
+              </form>
+            </div>
+          [% END %]
+        </li>
     [% END %]
 
 </ul> 

root/tt/bans_users.tt

@@ -17,15 +17,23 @@
     
     <li class="ui-state-default">
         <div class="span-6">Subscriber</div>
-        <div class="prepend-1 span-5 last">Last auth</div>
-        <div class="prepend-1 span-2">Auth count</div>
+        <div class="span-5">Last auth</div>
+        <div class="span-2 last">Auth count</div>
     </li>
 
     [% FOREACH user IN banned_users %]
         <li class="ui-state-default">
             <div class="span-6">[% user.username %]</div>
-            <div class="prepend-1 span-5">[% date.format (user.last_auth) %]</div>
-            <div class="prepend-1 span-2">[% user.auth_count %]</div>
+            <div class="span-5">[% date.format (user.last_auth) %]</div>
+            <div class="span-2">[% user.auth_count %]</div>
+            [% UNLESS Catalyst.session.admin.read_only %]
+              <div class="prepend-2 span-1 last">
+                <form action="/bans/unban_user" method="POST">
+                  <input type="hidden" name="user" value="[% user.username %]"/>
+                  <button class="button-delete">Delete</button>
+                </form>
+              </div>
+            [% END %]
         </li>
     [% END %]