From 4ea2cddf6b780145e26146dccd9b9c76de4491b3 Mon Sep 17 00:00:00 2001
From: Giannis Kopanas <jkopanas@gmail.com>
Date: Wed, 9 May 2018 10:31:22 +0300
Subject: [PATCH] fix(components): review page layout

---
 .../xpub-review/src/components/ReviewPage.js        |  8 ++++----
 .../src/components/review/ReviewLayout.js           | 13 +++++++------
 packages/components/xpub-selectors/src/index.js     |  4 +---
 .../components/xpub-submit/src/components/Submit.js |  1 +
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/packages/components/xpub-review/src/components/ReviewPage.js b/packages/components/xpub-review/src/components/ReviewPage.js
index b321f88f1..621a2ba1a 100644
--- a/packages/components/xpub-review/src/components/ReviewPage.js
+++ b/packages/components/xpub-review/src/components/ReviewPage.js
@@ -9,7 +9,7 @@ import {
   selectCurrentUser,
   selectCollection,
   selectFragments,
-  selectCurrentVersion,
+  selectLastSubmittedVersion,
   selectFragment,
   selectUser,
   getReviewerFromUser,
@@ -86,7 +86,7 @@ export default compose(
       const project = selectCollection(state, match.params.project)
       const versions = selectFragments(state, project.fragments)
       const version = selectFragment(state, match.params.version)
-      const currentVersion = selectCurrentVersion(state, project)
+      const lastSubmitted = selectLastSubmittedVersion(state, project)[0]
 
       let handlingEditors
       const editors = state.teams.find(
@@ -99,10 +99,10 @@ export default compose(
       if (editors) {
         handlingEditors = editors.members.map(id => selectUser(state, id))
       }
-      const reviewer = getReviewerFromUser(project, currentVersion, currentUser)
+      const reviewer = getReviewerFromUser(project, lastSubmitted, currentUser)
 
       return {
-        currentVersion,
+        lastSubmitted,
         handlingEditors,
         project,
         reviewer,
diff --git a/packages/components/xpub-review/src/components/review/ReviewLayout.js b/packages/components/xpub-review/src/components/review/ReviewLayout.js
index 0c0248569..9d9f47465 100644
--- a/packages/components/xpub-review/src/components/review/ReviewLayout.js
+++ b/packages/components/xpub-review/src/components/review/ReviewLayout.js
@@ -13,7 +13,7 @@ import Tabs from '../atoms/Tabs'
 const ReviewLayout = ({
   project,
   versions,
-  currentVersion,
+  lastSubmitted,
   handlingEditors,
   reviewer,
   valid,
@@ -25,6 +25,7 @@ const ReviewLayout = ({
 
   versions.forEach(version => {
     let review
+    console.log(version.reviewers,reviewer)
     if (version.reviewers) {
       review = version.reviewers.find(
         review => review.reviewer === reviewer._reviewer.id,
@@ -67,13 +68,13 @@ const ReviewLayout = ({
     }
   }, [])
 
-  const review = currentVersion.reviewers.find(
+  const review = lastSubmitted.reviewers.find(
     review => review.id === reviewer.id,
   )
 
-  if (currentVersion.submitted && (!review || !review.submitted)) {
+  if (lastSubmitted.submitted && (!review || !review.submitted)) {
     const submittedMoment = moment()
-    const key = currentVersion.id
+    const key = lastSubmitted.id
     const label = submittedMoment.format('YYYY-MM-DD')
 
     reviewSections.push({
@@ -81,7 +82,7 @@ const ReviewLayout = ({
         <div>
           <ReviewMetadata
             handlingEditors={handlingEditors}
-            version={currentVersion}
+            version={lastSubmitted}
           />
           <ReviewForm
             handleSubmit={handleSubmit}
@@ -98,7 +99,7 @@ const ReviewLayout = ({
     editorSections.push({
       content: (
         <SimpleEditor
-          content={currentVersion.source}
+          content={lastSubmitted.source}
           editing="selection"
           key={key}
           layout="bare"
diff --git a/packages/components/xpub-selectors/src/index.js b/packages/components/xpub-selectors/src/index.js
index 885d28186..155c938e5 100644
--- a/packages/components/xpub-selectors/src/index.js
+++ b/packages/components/xpub-selectors/src/index.js
@@ -13,9 +13,7 @@ export const newestDecidedFirst = items =>
   )
 
 export const newestSubmittedFirst = items =>
-  orderBy(
-    filter(items, data => data.subbmitted === undefined, ['created'], ['desc']),
-  )
+  orderBy(filter(items, data => !!data.submitted), ['submitted'], ['desc'])
 
 export const selectCurrentUser = state =>
   state.currentUser.isAuthenticated ? state.currentUser.user : null
diff --git a/packages/components/xpub-submit/src/components/Submit.js b/packages/components/xpub-submit/src/components/Submit.js
index a624f77c4..934bb85b9 100644
--- a/packages/components/xpub-submit/src/components/Submit.js
+++ b/packages/components/xpub-submit/src/components/Submit.js
@@ -55,6 +55,7 @@ const Submit = ({
       <CurrentVersion
         {...formProps}
         project={project}
+        readonly={false}
         version={currentVersion}
       />
     ),
-- 
GitLab