From a8080fad5eabd1ffef484e6979bea3c1eebd8978 Mon Sep 17 00:00:00 2001 From: Matthew Somerville Date: Tue, 25 Feb 2020 13:54:07 +0000 Subject: [UK] Use staging hosts on some staging sites. --- perllib/FixMyStreet/Map/MasterMap.pm | 3 ++- t/map/mastermap.t | 2 +- web/cobrands/isleofwight/assets.js | 7 +------ web/cobrands/northamptonshire/assets.js | 5 +---- web/cobrands/peterborough/assets.js | 7 ++----- web/js/map-mastermap.js | 3 ++- 6 files changed, 9 insertions(+), 18 deletions(-) diff --git a/perllib/FixMyStreet/Map/MasterMap.pm b/perllib/FixMyStreet/Map/MasterMap.pm index a2e0acd54..d66234bbf 100644 --- a/perllib/FixMyStreet/Map/MasterMap.pm +++ b/perllib/FixMyStreet/Map/MasterMap.pm @@ -23,7 +23,8 @@ sub map_tiles { my ( $self, %params ) = @_; my ( $x, $y, $z ) = ( $params{x_tile}, $params{y_tile}, $params{zoom_act} ); if ($z >= 17) { - my $base = "//%stilma.mysociety.org/mastermap/%d/%d/%d.png"; + my $layer = FixMyStreet->config('STAGING_SITE') ? 'mastermap-staging' : 'mastermap'; + my $base = "//%stilma.mysociety.org/$layer/%d/%d/%d.png"; return [ sprintf($base, 'a.', $z, $x-1, $y-1), sprintf($base, 'b.', $z, $x, $y-1), diff --git a/t/map/mastermap.t b/t/map/mastermap.t index 109142379..79522cdea 100644 --- a/t/map/mastermap.t +++ b/t/map/mastermap.t @@ -5,7 +5,7 @@ use FixMyStreet::Map::MasterMap; subtest 'correct map tiles used' => sub { my %test = ( 16 => [ '-', 'oml' ], - 20 => [ '.', 'mastermap' ] + 20 => [ '.', 'mastermap-staging' ] ); foreach my $zoom (qw(16 20)) { my $tiles = FixMyStreet::Map::MasterMap->map_tiles(x_tile => 123, y_tile => 456, zoom_act => $zoom); diff --git a/web/cobrands/isleofwight/assets.js b/web/cobrands/isleofwight/assets.js index e8d30e596..2710a9de7 100644 --- a/web/cobrands/isleofwight/assets.js +++ b/web/cobrands/isleofwight/assets.js @@ -4,14 +4,9 @@ if (!fixmystreet.maps) { return; } -var is_live = false; -if ( location.hostname === 'www.fixmystreet.com' || location.hostname === 'fms.islandroads.com' ) { - is_live = true; -} - var defaults = { http_options: { - url: is_live ? "https://tilma.mysociety.org/mapserver/iow": "https://staging.tilma.mysociety.org/mapserver/iow", + url: fixmystreet.staging ? "https://tilma.staging.mysociety.org/mapserver/iow": "https://tilma.mysociety.org/mapserver/iow", params: { SERVICE: "WFS", VERSION: "1.1.0", diff --git a/web/cobrands/northamptonshire/assets.js b/web/cobrands/northamptonshire/assets.js index b6f8f07f3..2b6cdeb69 100644 --- a/web/cobrands/northamptonshire/assets.js +++ b/web/cobrands/northamptonshire/assets.js @@ -4,10 +4,7 @@ if (!fixmystreet.maps) { return; } -var is_live = false; -if ( location.hostname === 'www.fixmystreet.com' || location.hostname == 'fixmystreet.northamptonshire.gov.uk' ) { - is_live = true; -} +var is_live = !fixmystreet.staging; var layers = [ /* diff --git a/web/cobrands/peterborough/assets.js b/web/cobrands/peterborough/assets.js index 2fb9075c8..b3cac5f6c 100644 --- a/web/cobrands/peterborough/assets.js +++ b/web/cobrands/peterborough/assets.js @@ -4,9 +4,10 @@ if (!fixmystreet.maps) { return; } + var defaults = { http_options: { - url: "https://tilma.mysociety.org/mapserver/peterborough", + url: fixmystreet.staging ? "https://tilma.staging.mysociety.org/mapserver/peterborough" : "https://tilma.mysociety.org/mapserver/peterborough", params: { SERVICE: "WFS", VERSION: "1.1.0", @@ -58,9 +59,6 @@ var trees_defaults = $.extend(true, {}, defaults, { asset_found: fixmystreet.message_controller.asset_found, asset_not_found: fixmystreet.message_controller.asset_not_found }, - http_options: { - url: "https://tilma.mysociety.org/mapserver/peterborough" - }, attributes: { tree_code: 'TREE_CODE' }, @@ -96,7 +94,6 @@ fixmystreet.assets.add(trees_defaults, { // The new tree request category is disabled in the other tree point layer. fixmystreet.assets.add(defaults, { http_options: { - url: "https://tilma.mysociety.org/mapserver/peterborough", params: { TYPENAME: "tree_points" } diff --git a/web/js/map-mastermap.js b/web/js/map-mastermap.js index d12a79bd0..bb9adf532 100644 --- a/web/js/map-mastermap.js +++ b/web/js/map-mastermap.js @@ -13,7 +13,8 @@ OpenLayers.Layer.MasterMap = OpenLayers.Class(OpenLayers.Layer.BingUK, { var urls = []; var servers = [ '', 'a.', 'b.', 'c.' ]; - var base = "//{S}tilma.mysociety.org/mastermap/${z}/${x}/${y}.png"; + var layer = fixmystreet.staging ? 'mastermap-staging' : 'mastermap'; + var base = "//{S}tilma.mysociety.org/" + layer + "/${z}/${x}/${y}.png"; for (var i=0; i < servers.length; i++) { urls.push( base.replace('{S}', servers[i]) ); } -- cgit v1.2.3