From 857df18cbfdcbeaef143cb184b6c33bb44d3013f Mon Sep 17 00:00:00 2001
From: Mitchell Gerber <mgerb42@gmail.com>
Date: Wed, 17 Jan 2018 22:50:22 -0600
Subject: [PATCH] client/server - update reply

---
 client/app/components/editor/editor.scss |  2 +-
 client/app/components/editor/editor.tsx  | 19 ++++++++++--
 client/app/pages/forum/forum.tsx         |  4 +--
 client/app/pages/thread/thread.tsx       | 39 +++++++++++++++++++-----
 client/index.html                        |  2 +-
 lib/myapp/data/reply.ex                  | 22 ++++++-------
 6 files changed, 64 insertions(+), 24 deletions(-)

diff --git a/client/app/components/editor/editor.scss b/client/app/components/editor/editor.scss
index 9ff1239..8a30dd8 100644
--- a/client/app/components/editor/editor.scss
+++ b/client/app/components/editor/editor.scss
@@ -25,7 +25,7 @@
   margin-bottom: 0;
 
   &__title {
-    height: 25px;
+    height: 30px;
     margin-bottom: 10px;
   }
 
diff --git a/client/app/components/editor/editor.tsx b/client/app/components/editor/editor.tsx
index 16c1399..d6002f9 100644
--- a/client/app/components/editor/editor.tsx
+++ b/client/app/components/editor/editor.tsx
@@ -9,6 +9,7 @@ import './editor.scss';
 interface Props {
   categoryId?: string;
   onClose: (cancel: boolean) => any;
+  editingReply?: ReplyModel;
   quotedReply?: ReplyModel;
   threadId?: string;
 }
@@ -40,6 +41,11 @@ export class Editor extends React.Component<Props, State> {
   componentDidMount() {
     document.body.style.overflow = 'hidden';
 
+    // set content if user is editing a reply
+    if (this.props.editingReply) {
+      this.onContentChange({ target: { value: this.props.editingReply.content } });
+    }
+
     if (this.titleRef) {
       this.titleRef.focus();
     } else {
@@ -84,12 +90,13 @@ export class Editor extends React.Component<Props, State> {
 
     const data = {
       content,
+      id: get(this.props, 'editingReply.id'),
       thread_id: this.props.threadId,
       quote_id: get(this.props, 'quotedReply.id') || undefined,
     };
 
     try {
-      await axios.post('/api/reply', data);
+      this.props.editingReply ? await axios.put('/api/reply', data) : await axios.post('/api/reply', data);
       this.props.onClose(false);
     } catch (e) {
       this.setState({ errorMessage: 'Server error. Please try again later.' });
@@ -140,17 +147,25 @@ export class Editor extends React.Component<Props, State> {
     );
   }
 
+  getEditorTitle(): string {
+    if (!this.props.threadId) {
+      return 'New Topic';
+    }
+    return `${this.props.editingReply ? 'Edit' : 'New'} Reply`;
+  }
+
   render() {
     return (
       <div className="editor-background">
         <ContentContainer className="editor-container">
           <form className="editor" onSubmit={event => this.onSubmit(event)} onReset={() => this.props.onClose(true)}>
 
-            <h2 style={{ color: 'white' }}>New {this.props.threadId ? 'Reply' : 'Topic'}</h2>
+            <h2 style={{ color: 'white' }}>{this.getEditorTitle()}</h2>
             {!this.props.threadId && this.renderTopicInput()}
 
             <div><label>Content</label></div>
             <textarea className="input editor__text-area flex-1"
+              value={this.state.content}
               onChange={event => this.onContentChange(event)} maxLength={2000}
               ref={ref => this.contentRef = ref}
             />
diff --git a/client/app/pages/forum/forum.tsx b/client/app/pages/forum/forum.tsx
index 5e81ed1..bb3ae17 100644
--- a/client/app/pages/forum/forum.tsx
+++ b/client/app/pages/forum/forum.tsx
@@ -6,8 +6,8 @@ import { ThreadService } from '../../services';
 import { Editor, ForumNav, LoginButton, ScrollToTop } from '../../components';
 import { ThreadModel } from '../../model';
 import { UserStore } from '../../stores/user-store';
-import './forum.scss';
 import { Oauth } from '../../util';
+import './forum.scss';
 
 const stickyImage = require('../../assets/sticky.gif');
 
@@ -102,7 +102,7 @@ export class Forum extends React.Component<Props, State> {
 
   renderCell(content: JSX.Element | string, style: any, center?: boolean, header?: boolean) {
     let classNames: string = '';
-    classNames += center ? ' forum-cell--center': '';
+    classNames += center ? ' forum-cell--center' : '';
     classNames += header ? ' forum-cell--header' : ' forum-cell--body';
     return <div className={`forum-cell flex-1 ${classNames}`} style={style}>{content}</div>;
   }
diff --git a/client/app/pages/thread/thread.tsx b/client/app/pages/thread/thread.tsx
index 09547b8..dcc2a62 100644
--- a/client/app/pages/thread/thread.tsx
+++ b/client/app/pages/thread/thread.tsx
@@ -1,21 +1,29 @@
 import React from 'react';
 import { RouteComponentProps } from 'react-router-dom';
-import { get, find, map } from 'lodash';
+import { get, find, orderBy } from 'lodash';
 import marked from 'marked';
 import { DateTime } from 'luxon';
+import { inject, observer } from 'mobx-react';
 import { CharacterService, ThreadService } from '../../services';
 import { Editor, Portrait, ScrollToTop } from '../../components';
 import { ReplyModel, ThreadModel } from '../../model';
+import { UserStore } from '../../stores/user-store';
+import { Oauth } from '../../util';
 import './thread.scss';
 
-interface Props extends RouteComponentProps<any> {}
+interface Props extends RouteComponentProps<any> {
+  userStore: UserStore;
+}
 
 interface State {
+  editingReply?: ReplyModel;
   quotedReply?: ReplyModel;
   showEditor: boolean;
   thread?: ThreadModel;
 }
 
+@inject('userStore')
+@observer
 export class Thread extends React.Component<Props, State> {
 
   constructor(props: Props) {
@@ -31,20 +39,27 @@ export class Thread extends React.Component<Props, State> {
 
   private async getReplies() {
     const thread = await ThreadService.getThread(this.props.match.params['threadId']);
-    thread.replies = map(thread.replies, (reply) => { // add the thread topic to the front of the list
-      return reply;
-    });
+    thread.replies = orderBy(thread.replies, ['inserted_at']);
     this.setState({ thread });
   }
 
   private onReplyClick() {
-    this.setState({ showEditor: true });
+    this.props.userStore.user ? this.setState({ showEditor: true }) : Oauth.openOuathWindow();
   }
 
   private onQuoteClick(reply: ReplyModel) {
+    this.props.userStore.user ?
     this.setState({
       showEditor: true,
       quotedReply: reply,
+    }) :
+    Oauth.openOuathWindow();
+  }
+
+  private onEditClick(reply: ReplyModel) {
+    this.setState({
+      editingReply: reply,
+      showEditor: true,
     });
   }
 
@@ -52,6 +67,7 @@ export class Thread extends React.Component<Props, State> {
     this.setState({
       showEditor: false,
       quotedReply: undefined,
+      editingReply: undefined,
     });
     if (!cancel) {
       this.getReplies();
@@ -99,6 +115,12 @@ export class Thread extends React.Component<Props, State> {
     );
   }
 
+  renderEditbutton(reply: ReplyModel): any {
+    if (get(this.props, 'userStore.user.id') === reply.user_id) {
+      return <a style={{ paddingRight: '10px' }} onClick={() => this.onEditClick(reply)}>Edit</a>;
+    }
+  }
+
   renderReplies(): any {
     return this.state.thread!.replies.map((reply, index) => {
       const replyDark = index % 2 === 0 ? 'reply--dark' : '';
@@ -114,7 +136,8 @@ export class Thread extends React.Component<Props, State> {
                   <b>{`${index + 1}. `}{index > 0 && 'Re: '}{this.state.thread!.title}</b>
                   <small style={{ paddingLeft: '5px' }}>| {this.getTimeFormat(reply.inserted_at)}</small>
                 </div>
-                <div>
+                <div className="flex flex--center">
+                  {this.renderEditbutton(reply)}
                   <img src={require('../../assets/quote-button.gif')}
                     className="reply__title__button"
                     onClick={() => this.onQuoteClick(reply)}/>
@@ -127,6 +150,7 @@ export class Thread extends React.Component<Props, State> {
               <div className="reply__content markdown-container">
                 {this.renderQuotedReply(reply)}
                 <div className={bluePost} dangerouslySetInnerHTML={{ __html: marked(reply.content, { sanitize: true }) }}/>
+                {reply.edited && <small className="red">[ post edited by {reply.user.character_name || reply.user.battletag} ]</small>}
               </div>
 
             </div>
@@ -150,6 +174,7 @@ export class Thread extends React.Component<Props, State> {
           <Editor threadId={this.props.match.params['threadId']}
             onClose={cancel => this.onEditorClose(cancel)}
             quotedReply={this.state.quotedReply}
+            editingReply={this.state.editingReply}
           />
         }
         <div className="topic-bg">
diff --git a/client/index.html b/client/index.html
index 4fba242..4c7743f 100644
--- a/client/index.html
+++ b/client/index.html
@@ -3,7 +3,7 @@
 <head>
     <title>Classic WoW Forums</title>
     <link href="https://fonts.googleapis.com/css?family=Roboto:100" rel="stylesheet">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
+    <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0"/>
 </head>
 
 <body>
diff --git a/lib/myapp/data/reply.ex b/lib/myapp/data/reply.ex
index ea46a31..d58e76b 100644
--- a/lib/myapp/data/reply.ex
+++ b/lib/myapp/data/reply.ex
@@ -50,7 +50,7 @@ defmodule MyApp.Data.Reply do
 
     case Repo.update_all(query, []) do
       nil -> {:error, "update thread error"}
-      _ -> {:ok, reply}
+      {1, _} -> {:ok, reply}
     end
   end
 
@@ -58,20 +58,20 @@ defmodule MyApp.Data.Reply do
   def user_update(params) do
     id = Map.get(params, "id")
     user_id = Map.get(params, "user_id")
+    content = Map.get(params, "content")
 
-    if is_nil(id) || is_nil(user_id) do
+    if is_nil(id) || is_nil(user_id) || is_nil(content) do
       {:error, "Invalid reply"}
     else
-      Repo.get_by(Data.Reply, %{id: id, user_id: user_id})
-      |> process_user_update(params)
+      query = from r in Data.Reply,
+        where: r.id == ^id and r.user_id == ^user_id,
+        update: [set: [content: ^content, edited: true]]
+
+      case Repo.update_all(query, []) do
+        nil -> {:error, "update reply error"}
+        {1, _} -> {:ok, "ok"}
+      end
     end
   end
 
-  defp process_user_update(reply, _params) when is_nil(reply), do: {:error, "Invalid reply"}
-  defp process_user_update(reply, params) when not is_nil(reply) do
-    user_update_changeset(reply, params)
-    |> Repo.update
-    |> Data.Util.process_insert_or_update
-  end
-
 end