a few fixes here & there - cache typehinting moved to Forum.ts, don't use button link in PostUser

js/src/common/Application.ts

@@ -67,9 +67,7 @@ export default abstract class Application {
      * A local cache that can be used to store data at the application level, so
      * that is persists between different routes.
      */
-    cache = {
+    cache: { [key: string]: any } = {};
-        notifications: null,
-    };
 
     routes = {};
 
@@ -243,10 +241,6 @@ export default abstract class Application {
 
         // Now make the request. If it's a failure, inspect the error that was
         // returned and show an alert containing its contents.
-        // const deferred = m.deferred();
-
-        // return new Promise((resolve, reject) => )
-
         return m.request(options).then(
             res => res,
             error => {

js/src/common/Model.ts

@@ -12,6 +12,8 @@ export default class Model {
      */
     data: any;
 
+    payload: any;
+
     /**
      * The time at which the model's data was last updated. Watching the value
      * of this property is a fast way to retain/cache a subtree if data hasn't

js/src/forum/Forum.ts

@@ -17,6 +17,7 @@ import CommentPost from './components/CommentPost';
 import User from '../common/models/User';
 import Post from '../common/models/Post';
 import Discussion from '../common/models/Discussion';
+import Notification from '../common/models/Notification';
 
 export default class Forum extends Application {
     routes = {
@@ -40,6 +41,12 @@ export default class Forum extends Application {
      */
     history: History = new History();
 
+    cache: {
+        notifications?: Notification[];
+        discussionList?: DiscussionList;
+        [key: string]: any;
+    } = {};
+
     postComponents = {
         comment: CommentPost,
         // discussionRenamed: DiscussionRenamedPost

js/src/forum/components/CommentPost.tsx

@@ -35,21 +35,14 @@ export default class CommentPost extends Post {
     }
 
     content() {
-        // Note: we avoid using JSX for the <ul> below because it results in some
+        return super.content().concat([
-        // weirdness in Mithril.js 0.1.x (see flarum/core#975). This workaround can
+            <header className="Post-header">
-        // be reverted when we upgrade to Mithril 1.0.
+                <ul>{listItems(this.headerItems().toArray())}</ul>
-        return super
+            </header>,
-            .content()
+            <div className="Post-body">
-            .concat([
+                {this.isEditing() ? <div className="Post-preview" config={this.configPreview.bind(this)} /> : m.trust(this.props.post.contentHtml())}
-                <header className="Post-header">{m('ul', listItems(this.headerItems().toArray()))}</header>,
+            </div>,
-                <div className="Post-body">
+        ]);
-                    {this.isEditing() ? (
-                        <div className="Post-preview" config={this.configPreview.bind(this)} />
-                    ) : (
-                        m.trust(this.props.post.contentHtml())
-                    )}
-                </div>,
-            ]);
     }
 
     onupdate(vnode) {

js/src/forum/components/PostUser.tsx

@@ -43,11 +43,11 @@ export default class PostUser extends Component<PostProps> {
         return (
             <div className="PostUser">
                 <h3>
-                    <LinkButton href={app.route.user(user)}>
+                    <m.route.Link href={app.route.user(user)}>
                         {avatar(user, { className: 'PostUser-avatar' })}
                         {userOnline(user)}
                         {username(user)}
-                    </LinkButton>
+                    </m.route.Link>
                 </h3>
                 <ul className="PostUser-badges badges">{listItems(user.badges().toArray())}</ul>
                 {card}

js/src/forum/utils/History.ts

@@ -43,7 +43,7 @@ export default class History {
      * @param {String} [url] The URL of the route. The current URL will be used if
      *     not provided.
      */
-    push(name: string, title: string, url: string = m.route.get()) {
+    push(name: string, title?: string, url: string = m.route.get()) {
         // If we're pushing an item with the same name as second-to-top item in the
         // stack, we will assume that the user has clicked the 'back' button in
         // their browser. In this case, we don't want to push a new item, so we will