aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Arter <davea@mysociety.org>2019-10-28 20:16:49 +0000
committerDave Arter <davea@mysociety.org>2019-12-09 12:48:12 +0000
commitc9fbc19964f7dd31fee15c821a52a4030695af9a (patch)
treed1688e40e2a6cf2358af5bfd52135a4ed51903a0
parent8422f7f53ebcf2e59627c30a15f4ee5952e85217 (diff)
[UK Councils] Factor out redirect to fms.com in area_check
-rw-r--r--perllib/FixMyStreet/Cobrand/UKCouncils.pm9
1 files changed, 7 insertions, 2 deletions
diff --git a/perllib/FixMyStreet/Cobrand/UKCouncils.pm b/perllib/FixMyStreet/Cobrand/UKCouncils.pm
index 3a831b431..bb3f6bbbd 100644
--- a/perllib/FixMyStreet/Cobrand/UKCouncils.pm
+++ b/perllib/FixMyStreet/Cobrand/UKCouncils.pm
@@ -146,6 +146,12 @@ sub area_check {
if ($council_match) {
return 1;
}
+ return ( 0, $self->area_check_error_message($params, $context) );
+}
+
+sub area_check_error_message {
+ my ( $self, $params, $context ) = @_;
+
my $url = 'https://www.fixmystreet.com/';
if ($context eq 'alert') {
$url .= 'alert';
@@ -157,9 +163,8 @@ sub area_check {
$url .= '?latitude=' . URI::Escape::uri_escape( $self->{c}->get_param('latitude') )
. '&amp;longitude=' . URI::Escape::uri_escape( $self->{c}->get_param('longitude') )
if $self->{c}->get_param('latitude');
- my $error_msg = "That location is not covered by " . $self->council_name . ".
+ return "That location is not covered by " . $self->council_name . ".
Please visit <a href=\"$url\">the main FixMyStreet site</a>.";
- return ( 0, $error_msg );
}
# All reports page only has the one council.