Make compose form also use normalized data

diff --git a/app/assets/javascripts/components/actions/compose.jsx b/app/assets/javascripts/components/actions/compose.jsx
index de4fe7445373eed68e087a84a247f9b25f2fccb8..1efb1ba9d5c8d7a20dde28438fc155f434675e2a 100644 (file)
--- a/app/assets/javascripts/components/actions/compose.jsx
+++ b/app/assets/javascripts/components/actions/compose.jsx
@@ -15,10 +15,10 @@ export function changeCompose(text) {
   };
 }
 
-export function replyCompose(payload) {
+export function replyCompose(status) {
   return {
     type: COMPOSE_REPLY,
-    payload: payload
+    status: status
   };
 }
 

diff --git a/app/assets/javascripts/components/containers/compose_form_container.jsx b/app/assets/javascripts/components/containers/compose_form_container.jsx
index 534830c278cf1a2ab0c349a3ede7b4d4bb07bdbd..d49217a9037833f3169aa7c0abbe317d7c1d441a 100644 (file)
--- a/app/assets/javascripts/components/containers/compose_form_container.jsx
+++ b/app/assets/javascripts/components/containers/compose_form_container.jsx
@@ -2,11 +2,24 @@ import { connect }                                          from 'react-redux';
 import ComposeForm                                          from '../components/compose_form';
 import { changeCompose, submitCompose, cancelReplyCompose } from '../actions/compose';
 
+function selectStatus(state) {
+  let statusId = state.getIn(['compose', 'in_reply_to'], null);
+
+  if (statusId === null) {
+    return null;
+  }
+
+  let status = state.getIn(['timelines', 'statuses', statusId]);
+  status = status.set('account', state.getIn(['timelines', 'accounts', status.get('account')]));
+
+  return status;
+};
+
 const mapStateToProps = function (state, props) {
   return {
     text: state.getIn(['compose', 'text']),
     is_submitting: state.getIn(['compose', 'is_submitting']),
-    in_reply_to: state.getIn(['compose', 'in_reply_to'])
+    in_reply_to: selectStatus(state)
   };
 };
 

diff --git a/app/assets/javascripts/components/reducers/compose.jsx b/app/assets/javascripts/components/reducers/compose.jsx
index abd99ee68f971f832897beb6931b1131e072187b..5dd093e9a791a2fe2085a357a7bec38aafa0987c 100644 (file)
--- a/app/assets/javascripts/components/reducers/compose.jsx
+++ b/app/assets/javascripts/components/reducers/compose.jsx
@@ -1,5 +1,6 @@
-import * as constants from '../actions/compose';
-import Immutable                                                                                              from 'immutable';
+import * as constants      from '../actions/compose';
+import { TIMELINE_DELETE } from '../actions/timelines';
+import Immutable           from 'immutable';
 
 const initialState = Immutable.Map({
   text: '',
@@ -13,7 +14,8 @@ export default function compose(state = initialState, action) {
       return state.set('text', action.text);
     case constants.COMPOSE_REPLY:
       return state.withMutations(map => {
-        map.set('in_reply_to', action.payload).set('text', `@${action.payload.getIn(['account', 'acct'])} `);
+        map.set('in_reply_to', action.status.get('id'));
+        map.set('text', `@${action.status.getIn(['account', 'acct'])} `);
       });
     case constants.COMPOSE_REPLY_CANCEL:
       return state.withMutations(map => {
@@ -27,6 +29,12 @@ export default function compose(state = initialState, action) {
       });
     case constants.COMPOSE_SUBMIT_FAIL:
       return state.set('is_submitting', false);
+    case TIMELINE_DELETE:
+      if (action.id === state.get('in_reply_to')) {
+        return state.set('in_reply_to', null);
+      } else {
+        return state;
+      }
     default:
       return state;
   }