From 9a6cc464ed203df2d0188be60ec7250b468bd33a Mon Sep 17 00:00:00 2001
From: Jason Tieu <tieujason330@gmail.com>
Date: Tue, 30 Jul 2019 11:22:25 -0700
Subject: [PATCH] Remove query parameters from UI browse button

- Fixes #9484
- Removed `queryStringSuffix` for directory rendering since we should
only base it off the input path field

pr-link: Alluxio/alluxio#9551
change-id: cid-7603541c8a06d8c65e347b1ddc33074cc1aedc1b
---
 webui/master/src/containers/pages/Browse/Browse.tsx    | 10 +++++-----
 .../pages/Browse/__snapshots__/Browse.test.tsx.snap    |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/webui/master/src/containers/pages/Browse/Browse.tsx b/webui/master/src/containers/pages/Browse/Browse.tsx
index f20ca3f35065..cbd54bb9e1b9 100644
--- a/webui/master/src/containers/pages/Browse/Browse.tsx
+++ b/webui/master/src/containers/pages/Browse/Browse.tsx
@@ -142,7 +142,7 @@ export class Browse extends React.Component<AllProps, IBrowseState> {
           <div className="row">
             <div className="col-12">
               {!browseData.currentDirectory.isDirectory && this.renderFileView(browseData, queryStringSuffix, initData)}
-              {browseData.currentDirectory.isDirectory && this.renderDirectoryListing(initData, browseData, queryStringSuffix)}
+              {browseData.currentDirectory.isDirectory && this.renderDirectoryListing(initData, browseData)}
             </div>
           </div>
         </div>
@@ -194,7 +194,7 @@ export class Browse extends React.Component<AllProps, IBrowseState> {
     );
   }
 
-  private renderDirectoryListing(initData: IInit, browseData: IBrowse, queryStringSuffix: string) {
+  private renderDirectoryListing(initData: IInit, browseData: IBrowse) {
     const {path, offset, limit} = this.state;
     const {history} = this.props;
     const fileInfos = browseData.fileInfos;
@@ -204,7 +204,7 @@ export class Browse extends React.Component<AllProps, IBrowseState> {
         <Form className="mb-3 browse-directory-form" id="browseDirectoryForm" inline={true}
               onSubmit={disableFormSubmit}>
           <FormGroup className="mb-2 mr-sm-2">
-            <Button tag={Link} to={`/browse?path=/${queryStringSuffix}`} color="secondary"
+            <Button tag={Link} to={`/browse?path=/`} color="secondary"
                     outline={true}>Root</Button>
           </FormGroup>
           <FormGroup className="mb-2 mr-sm-2">
@@ -212,10 +212,10 @@ export class Browse extends React.Component<AllProps, IBrowseState> {
             <Input type="text" id="browsePath" placeholder="Enter a Path" value={path || '/'}
                    onChange={pathInputHandler}
                    onKeyUp={this.createInputEnterHandler(history, () =>
-                   `/browse?path=${path}${queryStringSuffix}`)}/>
+                   `/browse?path=${path}`)}/>
           </FormGroup>
           <FormGroup className="mb-2 mr-sm-2">
-            <Button tag={Link} to={`/browse?path=${encodeURIComponent(path || '/')}${queryStringSuffix}`}
+            <Button tag={Link} to={`/browse?path=${encodeURIComponent(path || '/')}`}
             color="secondary">Go</Button>
           </FormGroup>
         </Form>
diff --git a/webui/master/src/containers/pages/Browse/__snapshots__/Browse.test.tsx.snap b/webui/master/src/containers/pages/Browse/__snapshots__/Browse.test.tsx.snap
index d5df98642043..e6310a732d52 100644
--- a/webui/master/src/containers/pages/Browse/__snapshots__/Browse.test.tsx.snap
+++ b/webui/master/src/containers/pages/Browse/__snapshots__/Browse.test.tsx.snap
@@ -1086,7 +1086,7 @@ exports[`Browse Shallow component Matches snapshot 1`] = `
               color="secondary"
               outline={true}
               tag={[Function]}
-              to="/browse?path=/&offset=0"
+              to="/browse?path=/"
             >
               Root
             </Button>
@@ -1127,7 +1127,7 @@ exports[`Browse Shallow component Matches snapshot 1`] = `
             <Button
               color="secondary"
               tag={[Function]}
-              to="/browse?path=%2F&offset=0"
+              to="/browse?path=%2F"
             >
               Go
             </Button>