refactor: adjust the search request to allow nullable values as well …

…as undefined

javascript-sdk/logstf/api/logstf.ts

@@ -81,7 +81,7 @@ export async function search(
     params.append("map", map);
   }
 
-  if (player.length > 0) {
+  if (player && player.length > 0) {
     params.append("player", player.join(","));
   }
 

javascript-sdk/logstf/tests/api.test.ts

@@ -44,8 +44,13 @@ Deno.test("Getting an invalid log fails", async () => {
 });
 
 Deno.test("Searching logs works as expected", async () => {
-  const log = await search({ limit: 1 });
+  const searchA = await search({ limit: 1 });
 
-  assertEquals(log.success, true);
-  assertEquals(1, log.logs.length);
+  assertEquals(searchA.success, true);
+  assertEquals(1, searchA.logs.length);
+
+  const searchB = await search({ limit: 1, map: null });
+
+  assertEquals(searchB.success, true);
+  assertEquals(1, searchB.logs.length);
 });

javascript-sdk/logstf/types/endpoints/logSearchRequest.ts

@@ -5,21 +5,22 @@ export type LogSearchRequest = {
   /**
    * The title of the log
    */
-  title?: string;
+  title?: string | null;
 
   /**
    * The map of the log
    */
-  map?: string;
+  map?: string | null;
 
   /**
    * The steamid of the uploader
    */
-  uploader?: string;
+  uploader?: string | null;
+
   /**
    * The list of steamids that played in the log
    */
-  player?: string[];
+  player?: string[] | null;
 
   /**
    * The number of entries to return