Skip to content

Commit ccb3e48

Browse files
committed
fix(base-client): fix falsy filter for param interpolation
1 parent e748f20 commit ccb3e48

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

src/services/common/base-client.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,7 @@ export const parseUrl = <P extends RecursiveRecord = RecursiveRecord, O extends
567567
}
568568
return segment;
569569
})
570-
.filter(Boolean)
570+
.filter(_segment => ![undefined, null, ''].includes(_segment))
571571
.join('/');
572572
}
573573

src/services/trakt-client/api/endpoints/seasons.endpoint.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ export const seasons = {
6262
season: new TraktClientEndpoint<
6363
{
6464
/** Trakt ID, Trakt slug, or IMDB ID */
65-
id: string;
65+
id: string | number;
6666
/** season number */
6767
season: number;
6868
/** include episode translations - 2 character language code (ISO 639-1) */

src/services/trakt-client/clients/base-trakt-client.ts

+5-3
Original file line numberDiff line numberDiff line change
@@ -192,9 +192,11 @@ export class BaseTraktClient extends BaseClient<TraktApiQuery, TraktApiResponse,
192192
}
193193

194194
// Extended
195-
if (template.opts?.extended && params.extended) {
196-
if (!template.opts.extended.includes(params.extended)) {
197-
throw Error(`Invalid value '${params.extended}', extended should be '${template.opts.extended}'`);
195+
if (template.opts?.extended?.length && params.extended) {
196+
const templateExtended = template.opts.extended;
197+
const paramsExtended = Array.isArray(params.extended) ? params.extended : [params.extended];
198+
if (paramsExtended.some(e => !templateExtended.includes(e))) {
199+
throw Error(`Invalid value '${params.extended}', extended should be '${template.opts.extended.join(', ')}'`);
198200
}
199201
queryParams.set('extended', `${params.extended}`);
200202
}

0 commit comments

Comments
 (0)