From edf5ab9ea923c5364fbf3301100c2e75f1cd8a9e Mon Sep 17 00:00:00 2001
From: Brandon Thomas <echelon@gmail.com>
Date: Sun, 13 Jun 2021 00:39:33 -0400
Subject: [PATCH] pagination on w2l and tts results

---
 vocodes/src/common/ApiConfig.ts               | 35 ++++++++++++++-
 .../Profile_TtsInferenceResultListFc.tsx      | 13 ++++--
 .../Profile_W2lInferenceResultListFc.tsx      | 43 ++++++++++++++++---
 3 files changed, 79 insertions(+), 12 deletions(-)

diff --git a/vocodes/src/common/ApiConfig.ts b/vocodes/src/common/ApiConfig.ts
index 2913a04d1f..26549147c8 100644
--- a/vocodes/src/common/ApiConfig.ts
+++ b/vocodes/src/common/ApiConfig.ts
@@ -7,6 +7,14 @@ export interface ListTtsInferenceResultsForUserArgs {
   limit?: number
 };
 
+export interface ListW2lInferenceResultsForUserArgs {
+  username: string, 
+  cursor?: string,
+  cursor_is_reversed?: boolean,
+  sort_ascending?: boolean,
+  limit?: number
+};
+
 class ApiConfig {
   
   isLocalDev: boolean;
@@ -151,8 +159,31 @@ class ApiConfig {
     return `${this.getScheme()}://${this.getNewApiHost()}/w2l/upload_template_job/${jobToken}`;
   }
 
-  listW2lInferenceResultsForUser(username: string) : string {
-    return `${this.getScheme()}://${this.getNewApiHost()}/user/${username}/w2l_results`;
+  listW2lInferenceResultsForUser(params: ListTtsInferenceResultsForUserArgs) : string {
+    const base_url = `${this.getScheme()}://${this.getNewApiHost()}/user/${params.username}/w2l_results`;
+
+    let query = "";
+    let query_prepend = "?";
+
+    if (params.cursor !== undefined) {
+      query += `${query_prepend}cursor=${params.cursor}`;
+      query_prepend = "&";
+
+      if (params.cursor_is_reversed !== undefined) {
+        query += `${query_prepend}cursor_is_reversed=${params.cursor_is_reversed}`;
+      }
+    }
+
+    if (params.sort_ascending !== undefined) {
+      query += `${query_prepend}sort_ascending=${params.sort_ascending}`;
+      query_prepend = "&";
+    }
+
+    if (params.limit !== undefined) {
+      query += `${query_prepend}limit=${params.limit}`;
+    }
+
+    return base_url + query;
   }
 
   firehoseEvents() : string {
diff --git a/vocodes/src/v2/profile/Profile_TtsInferenceResultListFc.tsx b/vocodes/src/v2/profile/Profile_TtsInferenceResultListFc.tsx
index 935d13fb86..79f389c4a9 100644
--- a/vocodes/src/v2/profile/Profile_TtsInferenceResultListFc.tsx
+++ b/vocodes/src/v2/profile/Profile_TtsInferenceResultListFc.tsx
@@ -70,13 +70,12 @@ function ProfileTtsInferenceResultsListFc(props: Props) {
       setPreviousCursor(modelsResponse.cursor_previous || null)
     })
     .catch(e => {
-      //this.props.onSpeakErrorCallback();
     });
   }, [props.username]);
 
   useEffect(() => {
     getPage(null, false);
-  }, [getPage, props.username]); // NB: Empty array dependency sets to run ONLY on mount
+  }, [getPage, props.username]);
 
   let rows : Array<JSX.Element> = [];
   
@@ -101,6 +100,12 @@ function ProfileTtsInferenceResultsListFc(props: Props) {
     );
   });
 
+  // Disable if there is no "next" or "previous" cursor.
+  // However, let the buttosn show up if there are no results (empty payload) 
+  // to get unstuck. Come up with a better fix for this.
+  let prevDisabled = !previousCursor && rows.length != 0;
+  let nextDisabled = !nextCursor && rows.length != 0;
+
   return (
     <div>
       <table className="table">
@@ -117,8 +122,8 @@ function ProfileTtsInferenceResultsListFc(props: Props) {
         </tbody>
       </table>
 
-      <button className="button is-info" onClick={() => getPage(previousCursor, true)}>&lt; Get newer</button> &nbsp;
-      <button className="button is-info" onClick={() => getPage(nextCursor, false)}>Get older&gt;</button>
+      <button className="button is-info" onClick={() => getPage(previousCursor, true)} disabled={prevDisabled}>&lt; Get newer</button> &nbsp;
+      <button className="button is-info" onClick={() => getPage(nextCursor, false)} disabled={nextDisabled}>Get older&gt;</button>
     </div>
   )
 }
diff --git a/vocodes/src/v2/profile/Profile_W2lInferenceResultListFc.tsx b/vocodes/src/v2/profile/Profile_W2lInferenceResultListFc.tsx
index 96b98a597f..c2a708de7f 100644
--- a/vocodes/src/v2/profile/Profile_W2lInferenceResultListFc.tsx
+++ b/vocodes/src/v2/profile/Profile_W2lInferenceResultListFc.tsx
@@ -1,10 +1,12 @@
-import React, { useState, useEffect } from 'react';
-import { ApiConfig } from '../../common/ApiConfig';
+import React, { useState, useEffect, useCallback } from 'react';
+import { ApiConfig, ListW2lInferenceResultsForUserArgs } from '../../common/ApiConfig';
 import { Link } from "react-router-dom";
 
 interface W2lInferenceResultListResponsePayload {
   success: boolean,
   results: Array<W2lInferenceResult>,
+  cursor_next: string | null | undefined,
+  cursor_previous: string | null | undefined,
 }
 
 interface W2lInferenceResult {
@@ -35,9 +37,24 @@ interface Props {
 function ProfileW2lInferenceResultsListFc(props: Props) {
   const [w2lResults, setW2lResults] = useState<Array<W2lInferenceResult>>([]);
 
-  useEffect(() => {
+  const [nextCursor, setNextCursor] = useState<string|null>(null);
+  const [previousCursor, setPreviousCursor] = useState<string|null>(null);
+
+  const getPage = useCallback((cursor : string|null, reverse: boolean) => {
+    let args : ListW2lInferenceResultsForUserArgs = {
+      username: props.username,
+      limit: 5,
+    };
+
+    if (cursor !== null) {
+      args.cursor = cursor;
+      if (reverse) {
+        args.cursor_is_reversed = true;
+      }
+    }
+
     const api = new ApiConfig();
-    const endpointUrl = api.listW2lInferenceResultsForUser(props.username);
+    const endpointUrl = api.listW2lInferenceResultsForUser(args);
 
     fetch(endpointUrl, {
       method: 'GET',
@@ -54,11 +71,16 @@ function ProfileW2lInferenceResultsListFc(props: Props) {
       }
 
       setW2lResults(templatesResponse.results)
+      setNextCursor(templatesResponse.cursor_next || null)
+      setPreviousCursor(templatesResponse.cursor_previous || null)
     })
     .catch(e => {
-      //this.props.onSpeakErrorCallback();
     });
-  }, [props.username]); // NB: Empty array dependency sets to run ONLY on mount
+  }, [props.username]);
+
+  useEffect(() => {
+    getPage(null, false);
+  }, [getPage, props.username]);
 
   let rows : Array<JSX.Element> = [];
   
@@ -80,6 +102,12 @@ function ProfileW2lInferenceResultsListFc(props: Props) {
     );
   });
 
+  // Disable if there is no "next" or "previous" cursor.
+  // However, let the buttosn show up if there are no results (empty payload) 
+  // to get unstuck. Come up with a better fix for this.
+  let prevDisabled = !previousCursor && rows.length != 0;
+  let nextDisabled = !nextCursor && rows.length != 0;
+
   return (
     <div>
       <table className="table">
@@ -96,6 +124,9 @@ function ProfileW2lInferenceResultsListFc(props: Props) {
           {rows}
         </tbody>
       </table>
+
+      <button className="button is-info" onClick={() => getPage(previousCursor, true)} disabled={prevDisabled}>&lt; Get newer</button> &nbsp;
+      <button className="button is-info" onClick={() => getPage(nextCursor, false)} disabled={nextDisabled}>Get older&gt;</button>
     </div>
   )
 }