fix(search): Fixed some cases where search wouldn't work correctly (#4398)

[skip ci]
pull/4400/head
sephrat 3 years ago committed by GitHub
parent 033697ef00
commit 4410790bc0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -66,6 +66,8 @@ namespace Ombi.Api
startingTag = builder.Query.Contains("?") ? "&" : "?"; startingTag = builder.Query.Contains("?") ? "&" : "?";
} }
value = Uri.EscapeDataString(value);
builder.Query = hasQuery builder.Query = hasQuery
? $"{builder.Query}{startingTag}{parameter}={value}" ? $"{builder.Query}{startingTag}{parameter}={value}"
: $"{startingTag}{parameter}={value}"; : $"{startingTag}{parameter}={value}";

@ -21,7 +21,7 @@ export class SearchV2Service extends ServiceHelpers {
} }
public multiSearch(searchTerm: string, filter: SearchFilter): Observable<IMultiSearchResult[]> { public multiSearch(searchTerm: string, filter: SearchFilter): Observable<IMultiSearchResult[]> {
return this.http.post<IMultiSearchResult[]>(`${this.url}/multi/${searchTerm}`, filter); return this.http.post<IMultiSearchResult[]>(`${this.url}/multi/${encodeURIComponent(searchTerm)}`, filter);
} }
public getFullMovieDetails(theMovieDbId: number): Observable<ISearchMovieResultV2> { public getFullMovieDetails(theMovieDbId: number): Observable<ISearchMovieResultV2> {
return this.http.get<ISearchMovieResultV2>(`${this.url}/Movie/${theMovieDbId}`); return this.http.get<ISearchMovieResultV2>(`${this.url}/Movie/${theMovieDbId}`);

@ -52,7 +52,7 @@ namespace Ombi.Controllers.V2
[HttpPost("multi/{searchTerm}")] [HttpPost("multi/{searchTerm}")]
public async Task<List<MultiSearchResult>> MultiSearch(string searchTerm, [FromBody] MultiSearchFilter filter) public async Task<List<MultiSearchResult>> MultiSearch(string searchTerm, [FromBody] MultiSearchFilter filter)
{ {
return await _multiSearchEngine.MultiSearch(searchTerm, filter, Request.HttpContext.RequestAborted); return await _multiSearchEngine.MultiSearch(Uri.UnescapeDataString(searchTerm), filter, Request.HttpContext.RequestAborted);
} }
/// <summary> /// <summary>

Loading…
Cancel
Save