]> git.openstreetmap.org Git - nominatim-ui.git/commitdiff
feat: Removed polygon parameter from API and debug link (#145)
authorYash Srivastava <52625656+darkshredder@users.noreply.github.com>
Sun, 9 May 2021 18:57:26 +0000 (00:27 +0530)
committerGitHub <noreply@github.com>
Sun, 9 May 2021 18:57:26 +0000 (20:57 +0200)
src/components/LastUpdated.svelte
test/search.js

index 44738f204b2608e34a2403f44b233623bc6a2a9f..f03e8cd69da89ca06f147bd2a0f457b85cecb74d 100644 (file)
   last_api_request_url_store.subscribe(url => {
     last_api_request_url = url;
 
+    if (last_api_request_url) {
+      last_api_request_url = new URL(last_api_request_url);
+      last_api_request_url.searchParams.delete('polygon_geojson');
+      last_api_request_url = last_api_request_url.toString();
+    }
+
     if (fetch_running || last_updated_date) return;
 
     fetch_running = true;
index 68273924fd2ba027fa7cc6c3a1699a5f4fda0d68..0e6ec9fd3134ed0f761780416168ed5584c75a1e 100644 (file)
@@ -94,6 +94,15 @@ describe('Search Page', function () {
       assert.deepEqual(link_titles, ['API request', 'debug output']);
     });
 
+    it('should not have polygon params in API request and debug URL', async function () {
+      let links_href = await page.$$eval('#api-request a', links => links.map(l => l.href));
+      let api_request_url = new URL(links_href[0]);
+      let debug_url = new URL(links_href[1]);
+
+      assert.deepStrictEqual(api_request_url.searchParams.has('polygon_geojson'), false);
+      assert.deepStrictEqual(debug_url.searchParams.has('polygon_geojson'), false);
+    });
+
     it('should display a map', async function () {
       await page.waitForSelector('#map');
       assert.equal((await page.$$('#map')).length, 1);