aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet
diff options
context:
space:
mode:
authorMatthew Somerville <matthew-github@dracos.co.uk>2016-03-15 13:29:42 +0000
committerMatthew Somerville <matthew-github@dracos.co.uk>2016-03-23 08:56:06 +0000
commitcb49284dc0503b2a05ca2bb98b8a1320431c2616 (patch)
treebe4ddb1fc16cf45b67e3cf0683506a493de4b27e /perllib/FixMyStreet
parentd3d0ab6d5a753d1e5c8277db981f03823683ae1f (diff)
Updates to cpanfile and code for perl 5.20/5.22.
The following modules had bugs that have been fixed for working in recent perls: * List::MoreUtils * Guard * PadWalker * aliased * URI * Convert::NLS_DATE_FORMAT The CGI module was removed from core in 5.20, so include it in the snapshot (I don't think it's actually used, but is a dependency). "{" needs to be escaped in regular expressions, and ~~ should not be used. Fix some tests that expect e.g. a certain hash ordering, to use sorted output or better comparisons.
Diffstat (limited to 'perllib/FixMyStreet')
-rwxr-xr-xperllib/FixMyStreet/App/Controller/Rss.pm12
-rw-r--r--perllib/FixMyStreet/TestMech.pm4
2 files changed, 8 insertions, 8 deletions
diff --git a/perllib/FixMyStreet/App/Controller/Rss.pm b/perllib/FixMyStreet/App/Controller/Rss.pm
index 586d5e7ae..6047f063b 100755
--- a/perllib/FixMyStreet/App/Controller/Rss.pm
+++ b/perllib/FixMyStreet/App/Controller/Rss.pm
@@ -260,9 +260,9 @@ sub add_row : Private {
$row->{confirmed} =~ s/^(\d+)/ordinal($1)/e if $c->stash->{lang_code} eq 'en-gb';
}
- (my $title = _($alert_type->item_title)) =~ s/{{(.*?)}}/$row->{$1}/g;
- (my $link = $alert_type->item_link) =~ s/{{(.*?)}}/$row->{$1}/g;
- (my $desc = _($alert_type->item_description)) =~ s/{{(.*?)}}/$row->{$1}/g;
+ (my $title = _($alert_type->item_title)) =~ s/\{\{(.*?)}}/$row->{$1}/g;
+ (my $link = $alert_type->item_link) =~ s/\{\{(.*?)}}/$row->{$1}/g;
+ (my $desc = _($alert_type->item_description)) =~ s/\{\{(.*?)}}/$row->{$1}/g;
my $base_url = $c->cobrand->base_url_for_report($row);
my $url = $base_url . $link;
@@ -317,9 +317,9 @@ sub add_parameters : Private {
$row->{$_} = $c->stash->{title_params}->{$_};
}
- (my $title = _($alert_type->head_title)) =~ s/{{(.*?)}}/$row->{$1}/g;
- (my $link = $alert_type->head_link) =~ s/{{(.*?)}}/$row->{$1}/g;
- (my $desc = _($alert_type->head_description)) =~ s/{{(.*?)}}/$row->{$1}/g;
+ (my $title = _($alert_type->head_title)) =~ s/\{\{(.*?)}}/$row->{$1}/g;
+ (my $link = $alert_type->head_link) =~ s/\{\{(.*?)}}/$row->{$1}/g;
+ (my $desc = _($alert_type->head_description)) =~ s/\{\{(.*?)}}/$row->{$1}/g;
$c->stash->{rss}->channel(
title => ent($title),
diff --git a/perllib/FixMyStreet/TestMech.pm b/perllib/FixMyStreet/TestMech.pm
index 9d836ca4d..f3ee7787b 100644
--- a/perllib/FixMyStreet/TestMech.pm
+++ b/perllib/FixMyStreet/TestMech.pm
@@ -234,8 +234,8 @@ sub get_first_email {
my $mech = shift;
my $email = shift or do { fail 'No email retrieved'; return };
my $email_as_string = $email->as_string;
- ok $email_as_string =~ s{\s+Date:\s+\S.*?$}{}xmsg, "Found and stripped out date";
- ok $email_as_string =~ s{\s+Message-ID:\s+\S.*?$}{}xmsg, "Found and stripped out message ID (contains epoch)";
+ ok $email_as_string =~ s{^Date:\s+\S.*?\r?\n}{}xmsg, "Found and stripped out date";
+ ok $email_as_string =~ s{^Message-ID:\s+\S.*?\r?\n}{}xmsg, "Found and stripped out message ID (contains epoch)";
return $email_as_string;
}