From 5786f1a10b58189287fcad639fb602db0b8eb6f3 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <38059171+askvortsov1@users.noreply.github.com>
Date: Fri, 3 Jul 2020 01:16:08 -0400
Subject: [PATCH] Fix discussions user page (#2225)

* Fixed up discussions user page, improve discussion list state signature
---
 js/src/forum/ForumApplication.js              |  2 +-
 .../forum/components/DiscussionsUserPage.js   | 23 +++++++++++--------
 js/src/forum/states/DiscussionListState.js    |  4 ++--
 3 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/js/src/forum/ForumApplication.js b/js/src/forum/ForumApplication.js
index 57f74ce53..b034d350e 100644
--- a/js/src/forum/ForumApplication.js
+++ b/js/src/forum/ForumApplication.js
@@ -84,7 +84,7 @@ export default class ForumApplication extends Application {
      *
      * @type {DiscussionListState}
      */
-    this.discussions = new DiscussionListState({ forumApp: this });
+    this.discussions = new DiscussionListState({}, this);
 
     /**
      * @deprecated beta 14, remove in beta 15.
diff --git a/js/src/forum/components/DiscussionsUserPage.js b/js/src/forum/components/DiscussionsUserPage.js
index 616f01ea9..e909a7d1e 100644
--- a/js/src/forum/components/DiscussionsUserPage.js
+++ b/js/src/forum/components/DiscussionsUserPage.js
@@ -1,5 +1,6 @@
 import UserPage from './UserPage';
 import DiscussionList from './DiscussionList';
+import DiscussionListState from '../states/DiscussionListState';
 
 /**
  * The `DiscussionsUserPage` component shows a discussion list inside of a user
@@ -12,16 +13,18 @@ export default class DiscussionsUserPage extends UserPage {
     this.loadUser(m.route.param('username'));
   }
 
+  show(user) {
+    super.show(user);
+
+    this.state = new DiscussionListState({
+      q: 'author:' + user.username(),
+      sort: 'newest',
+    });
+
+    this.state.refresh();
+  }
+
   content() {
-    return (
-      <div className="DiscussionsUserPage">
-        {DiscussionList.component({
-          params: {
-            q: 'author:' + this.user.username(),
-            sort: 'newest',
-          },
-        })}
-      </div>
-    );
+    return <div className="DiscussionsUserPage">{DiscussionList.component({ state: this.state })}</div>;
   }
 }
diff --git a/js/src/forum/states/DiscussionListState.js b/js/src/forum/states/DiscussionListState.js
index 38a87bf3d..787560507 100644
--- a/js/src/forum/states/DiscussionListState.js
+++ b/js/src/forum/states/DiscussionListState.js
@@ -1,8 +1,8 @@
 export default class DiscussionListState {
-  constructor({ params = {}, forumApp = app } = {}) {
+  constructor(params = {}, app = window.app) {
     this.params = params;
 
-    this.app = forumApp;
+    this.app = app;
 
     this.discussions = [];