Merge branch 'hin-restore-he-rec' into 'develop'

Hin restore he rec

See merge request !84

packages/component-faraday-ui/src/Tabs.js

@@ -4,12 +4,9 @@ const Tabs = ({ items, selectedTab, changeTab, children }) =>
   children({ selectedTab, changeTab })
 
 export default compose(
-  withStateHandlers(
-    { selectedTab: 0 },
-    {
-      changeTab: () => selectedTab => ({
-        selectedTab,
-      }),
-    },
-  ),
+  withStateHandlers(({ selectedTab = 0 }) => ({ selectedTab }), {
+    changeTab: () => selectedTab => ({
+      selectedTab,
+    }),
+  }),
 )(Tabs)

packages/component-faraday-ui/src/contextualBoxes/HERecommendation.js

@@ -20,10 +20,30 @@ import {
 } from 'pubsweet-component-faraday-ui/src'
 
 const options = [
-  { value: 'publish', label: 'Publish' },
-  { value: 'reject', label: 'Reject' },
-  { value: 'minor', label: 'Request Minor Revision' },
-  { value: 'major', label: 'Request Major Revision' },
+  {
+    value: 'publish',
+    label: 'Publish',
+    message: 'Recommend Manuscript for Publishing',
+    button: 'Submit Recommendation',
+  },
+  {
+    value: 'reject',
+    label: 'Reject',
+    message: 'Recommend Manuscript for Rejection',
+    button: 'Submit Recommendation',
+  },
+  {
+    value: 'minor',
+    label: 'Request Minor Revision',
+    message: 'Request Minor Revision',
+    button: 'Request Revision',
+  },
+  {
+    value: 'major',
+    label: 'Request Major Revision',
+    message: 'Request Major Revision',
+    button: 'Request Revision',
+  },
 ]
 
 const parseFormValues = ({ recommendation, ...rest }) => {
@@ -68,17 +88,15 @@ const HERecommendation = ({
         </ItemOverrideAlert>
       </Row>
 
-      {get(formValues, 'recommendation') === 'publish' ||
-      get(formValues, 'recommendation') === 'reject' ? (
+      {get(formValues, 'recommendation') === 'minor' ||
+      get(formValues, 'recommendation') === 'major' ? (
         <Row mt={2}>
-          <ItemOverrideAlert vertical>
-            <Label required>Message for Author</Label>
-            <ValidatedField
-              component={Textarea}
-              name="public"
-              validate={[required]}
-            />
-          </ItemOverrideAlert>
+          <ResponsiveItem mr={1} vertical>
+            <Label>
+              Message for Author <Text secondary>Optional</Text>
+            </Label>
+            <ValidatedField component={Textarea} name="public" />
+          </ResponsiveItem>
         </Row>
       ) : (
         <ResponsiveRow mt={2}>
@@ -100,7 +118,11 @@ const HERecommendation = ({
 
       <Row justify="flex-end" mt={2}>
         <Button onClick={handleSubmit} primary size="medium">
-          Submit recommendation
+          {
+            options.find(
+              o => o.value === get(formValues, 'recommendation', 'publish'),
+            ).button
+          }
         </Button>
       </Row>
     </Root>
@@ -116,17 +138,31 @@ export default compose(
   withProps(({ formValues }) => ({
     modalTitle: options.find(
       o => o.value === get(formValues, 'recommendation', 'publish'),
-    ).label,
+    ).message,
+    confirmMessage: options.find(
+      o => o.value === get(formValues, 'recommendation', 'publish'),
+    ).button,
   })),
   reduxForm({
     form: 'HERecommendation',
     onSubmit: (
       values,
       dispatch,
-      { onRecommendationSubmit, showModal, setFetching, modalTitle },
+      {
+        onRecommendationSubmit,
+        showModal,
+        setFetching,
+        modalTitle,
+        confirmMessage,
+      },
     ) => {
       showModal({
         title: `${modalTitle}?`,
+        confirmText:
+          confirmMessage === 'Submit Recommendation'
+            ? 'Submit'
+            : confirmMessage,
+
         onConfirm: props => {
           onRecommendationSubmit(parseFormValues(values), {
             ...props,

packages/component-faraday-ui/src/contextualBoxes/ReviewerDetails.js

@@ -21,7 +21,7 @@ import {
 
 const ReviewerDetails = ({
   journal,
-  reports,
+  reports = [],
   fragment,
   invitations,
   previewFile,
@@ -47,7 +47,7 @@ const ReviewerDetails = ({
       toggle={toggle}
       {...rest}
     >
-      <Tabs>
+      <Tabs selectedTab={reports.length ? 1 : 0}>
         {({ selectedTab, changeTab }) => (
           <Fragment>
             <TabsHeader>

packages/component-manuscript/src/components/ManuscriptLayout.js

@@ -181,15 +181,17 @@ const ManuscriptLayout = ({
           />
         )}
 
-        {get(currentUser, 'permissions.canMakeHERecommendation', false) && (
-          <HERecommendation
-            formValues={get(formValues, 'editorialRecommendation', {})}
-            hasReviewerReports={reviewerRecommendations.length > 0}
-            highlight={reviewerRecommendations.length > 0}
-            modalKey="heRecommendation"
-            onRecommendationSubmit={onEditorialRecommendation}
-          />
-        )}
+        {get(currentUser, 'permissions.canMakeHERecommendation', false) &&
+          (!invitationsWithReviewers.length ||
+            reviewerRecommendations.length > 0) && (
+            <HERecommendation
+              formValues={get(formValues, 'editorialRecommendation', {})}
+              hasReviewerReports={reviewerRecommendations.length > 0}
+              highlight={reviewerRecommendations.length > 0}
+              modalKey="heRecommendation"
+              onRecommendationSubmit={onEditorialRecommendation}
+            />
+          )}
 
         {get(currentUser, 'permissions.canMakeDecision', false) && (
           <ManuscriptEicDecision

packages/component-manuscript/src/components/ManuscriptPage.js

@@ -170,9 +170,7 @@ export default compose(
         eicDecision: getFormValues('eic-decision')(state),
         reviewerReport: getFormValues('reviewerReport')(state),
         responseToInvitation: getFormValues('answer-invitation')(state),
-        editorialRecommendation: getFormValues('editorialRecommendation')(
-          state,
-        ),
+        editorialRecommendation: getFormValues('HERecommendation')(state),
       },
       invitationsWithReviewers: getInvitationsWithReviewersForFragment(
         state,