From 2971c72f7659afb2f24e2ef1fe7cb91a5c8b9e65 Mon Sep 17 00:00:00 2001
From: Patrick Fic <pfic@outlook.com>
Date: Wed, 15 Jan 2020 22:24:35 -0800
Subject: [PATCH] Fixed scaling of images. Fixed server.js issues for
 production

---
 bodyshop_translations.babel                   | 106 +++++++++++++++++-
 .../job-detail-cards.customer.component.jsx   |   5 +-
 .../jobs-documents.container.jsx              |   2 +-
 .../jobs-documents/jobs-documents.page.jsx    |  78 +++++++++----
 client/src/graphql/documents.queries.js       |   2 +-
 .../pages/jobs-detail/jobs-detail.page.jsx    |   1 -
 client/src/translations/en_us/common.json     |  12 ++
 client/src/translations/es/common.json        |  12 ++
 client/src/translations/fr/common.json        |  12 ++
 s3upload.js                                   |  28 ++++-
 server.js                                     |  28 +++--
 11 files changed, 246 insertions(+), 40 deletions(-)

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index 89f625184..5a81d8485 100644
--- a/bodyshop_translations.babel
+++ b/bodyshop_translations.babel
@@ -1,4 +1,4 @@
-<babeledit_project version="1.2" be_version="2.6.1">
+<babeledit_project be_version="2.6.1" version="1.2">
     <!--
 
     BabelEdit project file
@@ -18,6 +18,110 @@
             <folder_node>
                 <name>translation</name>
                 <children>
+                    <folder_node>
+                        <name>documents</name>
+                        <children>
+                            <folder_node>
+                                <name>errors</name>
+                                <children>
+                                    <concept_node>
+                                        <name>getpresignurl</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                    <concept_node>
+                                        <name>insert</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                </children>
+                            </folder_node>
+                            <folder_node>
+                                <name>labels</name>
+                                <children>
+                                    <concept_node>
+                                        <name>upload</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                </children>
+                            </folder_node>
+                            <folder_node>
+                                <name>successes</name>
+                                <children>
+                                    <concept_node>
+                                        <name>insert</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                </children>
+                            </folder_node>
+                        </children>
+                    </folder_node>
                     <folder_node>
                         <name>general</name>
                         <children>
diff --git a/client/src/components/job-detail-cards/job-detail-cards.customer.component.jsx b/client/src/components/job-detail-cards/job-detail-cards.customer.component.jsx
index b2059bfb3..1ad95037a 100644
--- a/client/src/components/job-detail-cards/job-detail-cards.customer.component.jsx
+++ b/client/src/components/job-detail-cards/job-detail-cards.customer.component.jsx
@@ -10,8 +10,7 @@ export default function JobDetailCardsCustomerComponent({ loading, data }) {
     <CardTemplate
       loading={loading}
       title={t("jobs.labels.cards.customer")}
-      extraLink={data?.owner ? `/manage/owners/${data?.owner?.id}` : null}
-    >
+      extraLink={data?.owner ? `/manage/owners/${data?.owner?.id}` : null}>
       {data ? (
         <span>
           <div>{`${data?.pit_owner_first_name ??
@@ -25,7 +24,7 @@ export default function JobDetailCardsCustomerComponent({ loading, data }) {
             </a>
           ) : null}
 
-          <div>{`${data?.owner.preferred_contact ?? ""}`}</div>
+          <div>{`${data?.owner?.preferred_contact ?? ""}`}</div>
         </span>
       ) : null}
     </CardTemplate>
diff --git a/client/src/components/jobs-documents/jobs-documents.container.jsx b/client/src/components/jobs-documents/jobs-documents.container.jsx
index 268000809..2576b0ce7 100644
--- a/client/src/components/jobs-documents/jobs-documents.container.jsx
+++ b/client/src/components/jobs-documents/jobs-documents.container.jsx
@@ -23,7 +23,7 @@ export default function JobsDocumentsContainer({ jobId }) {
 
   return (
     <JobDocuments
-      data={data}
+      data={data.documents}
       jobId={jobId}
       shopId={
         shopData.data?.bodyshops[0]?.id
diff --git a/client/src/components/jobs-documents/jobs-documents.page.jsx b/client/src/components/jobs-documents/jobs-documents.page.jsx
index 61a68269a..e936447a8 100644
--- a/client/src/components/jobs-documents/jobs-documents.page.jsx
+++ b/client/src/components/jobs-documents/jobs-documents.page.jsx
@@ -1,10 +1,11 @@
 import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
-import { Upload, Modal, Icon, Button } from "antd";
+import { Upload, Modal, Icon, notification } from "antd";
 import axios from "axios";
 import "./jobs-documents.styles.scss";
 import Resizer from "react-image-file-resizer";
-//import { Resizer } from "../../utils/resizer";
+import { useMutation } from "react-apollo";
+import { INSERT_NEW_DOCUMENT } from "../../graphql/documents.queries";
 
 function getBase64(file) {
   return new Promise((resolve, reject) => {
@@ -16,11 +17,25 @@ function getBase64(file) {
 }
 
 function JobsDetailPage({ shopId, jobId, loading, data }) {
-  //const { t } = useTranslation();
+  const { t } = useTranslation();
+  const [insertNewDocument] = useMutation(INSERT_NEW_DOCUMENT);
+
+  console.log("data", data);
 
   const [state, setState] = useState({
     previewVisible: false,
     previewImage: "",
+
+    fl: data.reduce((acc, value) => {
+      acc.push({
+        uid: value.id,
+        url: value.url,
+        name: value.name,
+        status: "done"
+      });
+      return acc;
+    }, []),
+
     fileList: [
       {
         uid: "-1",
@@ -41,24 +56,22 @@ function JobsDetailPage({ shopId, jobId, loading, data }) {
     console.log("handleUpload Props:", ev);
     Resizer.imageFileResizer(
       ev.file,
-      3000,
-      3000,
+      3500,
+      3500,
       "JPEG",
-      85,
+      75,
       0,
       uri => {
-        let file = new File([uri], ev.file.name, {
-          //type: ev.file.type,
-          uid: ev.file.uid
-        });
-
+        let file = new File([uri], ev.file.name, {});
+        file.uid = ev.file.uid;
         // Split the filename to get the name and type
         let fileName = file.name;
         let fileType = file.type;
-        console.log("Preparing the upload ");
+        let key = `${shopId}/${jobId}/${fileName}`;
+        //URL is using the proxy set in pacakges.json.
         axios
           .post("/sign_s3", {
-            fileName: `${shopId}/${jobId}/${fileName}`,
+            fileName: key,
             fileType
           })
           .then(response => {
@@ -66,8 +79,6 @@ function JobsDetailPage({ shopId, jobId, loading, data }) {
             var signedRequest = returnData.signedRequest;
             var url = returnData.url;
             setState({ ...state, url: url });
-            console.log("Recieved a signed request " + signedRequest);
-
             // Put the fileType in the headers for the upload
             var options = {
               headers: {
@@ -79,16 +90,40 @@ function JobsDetailPage({ shopId, jobId, loading, data }) {
               .put(signedRequest, file, options)
               .then(result => {
                 console.log("Response from s3", result);
+
+                insertNewDocument({
+                  variables: {
+                    docInput: [
+                      {
+                        jobid: jobId,
+                        uploaded_by: "patrick@bodyshop.app",
+                        url,
+                        thumb_url: url
+                      }
+                    ]
+                  }
+                }).then(r => {
+                  console.log(r);
+                  notification["success"]({
+                    message: t("documents.successes.insert")
+                  });
+                });
+
                 setState({ ...state, success: true });
               })
               .catch(error => {
-                console.log("Inside Error here.", error);
-                alert("ERROR " + JSON.stringify(error));
+                console.log("Error uploading to S3", error);
+                notification["error"]({
+                  message: t("documents.errors.insert") + JSON.stringify(error)
+                });
               });
           })
           .catch(error => {
             console.log("Outside Error here.", error);
-            alert(JSON.stringify(error));
+            notification["error"]({
+              message:
+                t("documents.errors.getpresignurl") + JSON.stringify(error)
+            });
           });
       },
       "blob"
@@ -115,7 +150,7 @@ function JobsDetailPage({ shopId, jobId, loading, data }) {
   const uploadButton = (
     <div>
       <Icon type='plus' />
-      <div className='ant-upload-text'>Upload</div>
+      <div className='ant-upload-text'>{t("documents.labels.upload")}</div>
     </div>
   );
 
@@ -123,11 +158,6 @@ function JobsDetailPage({ shopId, jobId, loading, data }) {
     <div className='clearfix'>
       <Upload
         customRequest={handleUpload}
-        data={file => ({
-          photoCotent: file, // file is currently uploading pictures
-          photoWidth: file.height, // Get the width and height of the picture by handleBeforeUpload
-          photoHeight: file.width
-        })}
         accept='.pdf,.jpg,.jpeg'
         listType='picture-card'
         fileList={fileList}
diff --git a/client/src/graphql/documents.queries.js b/client/src/graphql/documents.queries.js
index 1217234d6..5a2cc937d 100644
--- a/client/src/graphql/documents.queries.js
+++ b/client/src/graphql/documents.queries.js
@@ -13,7 +13,7 @@ export const GET_DOCUMENTS_BY_JOB = gql`
 
 export const INSERT_NEW_DOCUMENT = gql`
   mutation INSERT_NEW_DOCUMENT($docInput: [documents_insert_input!]!) {
-    insert_ducments(objects: $docInput) {
+    insert_documents(objects: $docInput) {
       returning {
         id
       }
diff --git a/client/src/pages/jobs-detail/jobs-detail.page.jsx b/client/src/pages/jobs-detail/jobs-detail.page.jsx
index 497d4d679..2c3b954c9 100644
--- a/client/src/pages/jobs-detail/jobs-detail.page.jsx
+++ b/client/src/pages/jobs-detail/jobs-detail.page.jsx
@@ -29,7 +29,6 @@ function JobsDetailPage({ match, location }) {
   const [selectedTab, setSelectedTab] = useState(hash ? hash : "#lines");
   if (loading) return <SpinComponent />;
   if (error) return <AlertComponent message={error.message} type='error' />;
-  console.log("selectedTab", selectedTab);
   return (
     <div>
       <Row>
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index 867a99a1a..b0334030f 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1,5 +1,17 @@
 {
 	"translation": {
+		"documents": {
+			"errors": {
+				"getpresignurl": "Error obtaining presigned URL for document. ",
+				"insert": "Unable to upload file."
+			},
+			"labels": {
+				"upload": "Upload"
+			},
+			"successes": {
+				"insert": "Uploaded document successfully. "
+			}
+		},
 		"general": {
 			"labels": {
 				"in": "In",
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index 6d2e29aa6..38bdacf2d 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -1,5 +1,17 @@
 {
 	"translation": {
+		"documents": {
+			"errors": {
+				"getpresignurl": "Error al obtener la URL prescrita para el documento.",
+				"insert": "Incapaz de cargar el archivo."
+			},
+			"labels": {
+				"upload": "Subir"
+			},
+			"successes": {
+				"insert": "Documento cargado con éxito."
+			}
+		},
 		"general": {
 			"labels": {
 				"in": "en",
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index 2445064fe..cf488722e 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -1,5 +1,17 @@
 {
 	"translation": {
+		"documents": {
+			"errors": {
+				"getpresignurl": "Erreur lors de l'obtention de l'URL présignée pour le document.",
+				"insert": "Incapable de télécharger le fichier."
+			},
+			"labels": {
+				"upload": "Télécharger"
+			},
+			"successes": {
+				"insert": "Document téléchargé avec succès."
+			}
+		},
 		"general": {
 			"labels": {
 				"in": "dans",
diff --git a/s3upload.js b/s3upload.js
index 016df8f8a..263f781f3 100644
--- a/s3upload.js
+++ b/s3upload.js
@@ -37,7 +37,33 @@ exports.sign_s3 = (req, res) => {
       res.json({ success: false, error: err });
     }
     // Data payload of what we are sending back, the url of the signedRequest and a URL where we can access the content after its saved.
-    const returnData = { 
+    const returnData = {
+      signedRequest: data,
+      url: `https://${S3_BUCKET}.s3.amazonaws.com/${fileName}`
+    };
+    // Send it all back
+    res.json({ success: true, data: { returnData } });
+  });
+};
+
+exports.get_s3 = (req, res) => {
+  const s3 = new aws.S3(); // Create a new instance of S3
+  const fileName = req.body.fileName;
+  //const fileType = req.body.fileType;
+  // Set up the payload of what we are sending to the S3 api
+  const s3Params = {
+    Bucket: S3_BUCKET,
+    Key: fileName,
+    Expires: 500
+  };
+  // Make a request to the S3 API to get a signed URL which we can use to upload our file
+  s3.getSignedUrl("getObject", s3Params, (err, data) => {
+    if (err) {
+      console.log(err);
+      res.json({ success: false, error: err });
+    }
+    // Data payload of what we are sending back, the url of the signedRequest and a URL where we can access the content after its saved.
+    const returnData = {
       signedRequest: data,
       url: `https://${S3_BUCKET}.s3.amazonaws.com/${fileName}`
     };
diff --git a/server.js b/server.js
index 95c8b337f..3809dcba9 100644
--- a/server.js
+++ b/server.js
@@ -6,8 +6,8 @@ const compression = require("compression");
 var enforce = require("express-sslify");
 if (process.env.NODE_ENV !== "production") require("dotenv").config();
 
-const https = require('https');
-const fs = require('fs');
+const https = require("https");
+const fs = require("fs");
 
 const app = express();
 const port = process.env.PORT || 5000;
@@ -32,17 +32,29 @@ if (process.env.NODE_ENV === "production") {
 
 var s3upload = require("./s3upload");
 app.post("/sign_s3", s3upload.sign_s3);
+app.get("/sign_s3", s3upload.get_s3);
 
 app.get("/test", function(req, res) {
   res.json({ success: true });
 });
 
-https.createServer({
-  key: fs.readFileSync('./key.pem'),
-  cert: fs.readFileSync('./cert.pem'),
-  passphrase: 'Wl0d8k@!'
-}, app)
-.listen(5000);
+if (process.env.NODE_ENV === "production") {
+  app.listen(port, error => {
+    if (error) throw error;
+    console.log("Server running on port " + port);
+  });
+} else {
+  https
+    .createServer(
+      {
+        key: fs.readFileSync("./key.pem"),
+        cert: fs.readFileSync("./cert.pem"),
+        passphrase: "Wl0d8k@!"
+      },
+      app
+    )
+    .listen(port);
+}
 
 // app.listen(port, error => {
 //   if (error) throw error;