From dfe1f5371d01969aad7d7dafe6c7980589935517 Mon Sep 17 00:00:00 2001
From: Daniel Tiefnig <dtiefnig@sipwise.com>
Date: Mon, 24 Oct 2011 15:00:12 +0000
Subject: [PATCH] use urlencoded parameter for ncos pattern edit - see issue
 #763

---
 lib/admin/Controller/ncos.pm | 8 +++++++-
 root/tt/ncos_lists.tt        | 2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/lib/admin/Controller/ncos.pm b/lib/admin/Controller/ncos.pm
index 97699b1..708dcc1 100644
--- a/lib/admin/Controller/ncos.pm
+++ b/lib/admin/Controller/ncos.pm
@@ -3,6 +3,7 @@ package admin::Controller::ncos;
 use strict;
 use warnings;
 use base 'Catalyst::Controller';
+use URI::Escape;;
 
 =head1 NAME
 
@@ -186,7 +187,12 @@ sub lists : Local {
                                                         { level => $level },
                                                         \$patterns
                                                       );
-    $c->stash->{patterns} = $patterns if eval { @$patterns };
+    if(eval { @$patterns }) {
+        for(@$patterns) {
+            $$_{urlenc_pattern} = uri_escape($$_{pattern});
+        }
+        $c->stash->{patterns} = $patterns;
+    }
 
     my $lnpids;
     return unless $c->model('Provisioning')->call_prov( $c, 'billing', 'get_ncos_lnp_list',
diff --git a/root/tt/ncos_lists.tt b/root/tt/ncos_lists.tt
index ce5c478..05b549d 100644
--- a/root/tt/ncos_lists.tt
+++ b/root/tt/ncos_lists.tt
@@ -60,7 +60,7 @@
                   <td>[% pattern.description %]</td>
                   <td>
                   [% UNLESS Catalyst.session.admin.read_only %]
-                    <a href="/ncos/lists?level=[% level.level %]&amp;edit_pattern=[% pattern.pattern %]" class="aaction">edit</a>
+                    <a href="/ncos/lists?level=[% level.level %]&amp;edit_pattern=[% pattern.urlenc_pattern %]" class="aaction">edit</a>
                   [% END %]
                   </td>
                   <form action="/ncos/do_delete_pattern" method="post">