From 576a3f0fe74f23bada3222adb197a7df4a6da5f1 Mon Sep 17 00:00:00 2001 From: marudor Date: Tue, 6 Apr 2021 15:55:30 +0200 Subject: [PATCH] fix: allow all length for evaNumbers (#480) --- packages/server/API/routes.ts | 6 +++--- packages/server/StopPlace/search.ts | 3 +++ packages/types/common.ts | 9 +++------ public/swagger.json | 7 +++---- 4 files changed, 12 insertions(+), 13 deletions(-) diff --git a/packages/server/API/routes.ts b/packages/server/API/routes.ts index 4d374dddd..fa6b57fe2 100644 --- a/packages/server/API/routes.ts +++ b/packages/server/API/routes.ts @@ -45,7 +45,7 @@ const models: TsoaRoute.Models = { // WARNING: This file was auto-generated with tsoa. Please do not modify it. Re-run tsoa to re-generate this file: https://github.com/lukeautry/tsoa "EvaNumber": { "dataType": "refAlias", - "type": {"dataType":"string","validators":{"minLength":{"value":7},"maxLength":{"value":7}}}, + "type": {"dataType":"string","validators":{}}, }, // WARNING: This file was auto-generated with tsoa. Please do not modify it. Re-run tsoa to re-generate this file: https://github.com/lukeautry/tsoa "OccupancyPeriod": { @@ -1573,8 +1573,8 @@ const models: TsoaRoute.Models = { "numF": {"dataType":"double"}, "ctxScr": {"dataType":"string"}, "ushrp": {"dataType":"boolean"}, - "start": {"dataType":"string","required":true}, - "destination": {"dataType":"string","required":true}, + "start": {"ref":"EvaNumber","required":true}, + "destination": {"ref":"EvaNumber","required":true}, "time": {"dataType":"datetime"}, "transferTime": {"dataType":"double"}, "maxChanges": {"dataType":"double"}, diff --git a/packages/server/StopPlace/search.ts b/packages/server/StopPlace/search.ts index 417da1ff8..ca614a95a 100644 --- a/packages/server/StopPlace/search.ts +++ b/packages/server/StopPlace/search.ts @@ -117,6 +117,9 @@ export async function geoSearchStopPlace( } async function byRl100WithSpaceHandling(rl100: string) { + if (rl100.length > 5) { + return Promise.resolve(undefined); + } const rl100Promise = byRl100(rl100.toUpperCase()); let rl100DoubleSpacePromise: typeof rl100Promise = Promise.resolve(undefined); if (rl100.length < 5 && rl100.includes(' ')) { diff --git a/packages/types/common.ts b/packages/types/common.ts index e0d21987b..c71cf1ac5 100644 --- a/packages/types/common.ts +++ b/packages/types/common.ts @@ -1,13 +1,10 @@ export interface CommonRoutingOptions { - // evaid - start: string; - // evaid - destination: string; + start: EvaNumber; + destination: EvaNumber; time?: Date; } /** - * @minLength 7 - * @maxLength 7 + * Usually 7 digits */ export type EvaNumber = string; diff --git a/public/swagger.json b/public/swagger.json index d36d26d9e..06ef74e0d 100644 --- a/public/swagger.json +++ b/public/swagger.json @@ -17,8 +17,7 @@ }, "EvaNumber": { "type": "string", - "minLength": 7, - "maxLength": 7 + "description": "Usually 7 digits" }, "OccupancyPeriod": { "properties": { @@ -3717,10 +3716,10 @@ "description": "Is a station nearby enough for routing?" }, "start": { - "type": "string" + "$ref": "#/components/schemas/EvaNumber" }, "destination": { - "type": "string" + "$ref": "#/components/schemas/EvaNumber" }, "time": { "type": "string",