From 94bd80b59a24d18ebd0ce60c6e78ab16f015cd57 Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Thu, 5 Dec 2019 11:59:23 +0100
Subject: [PATCH] Add production mode to ctb

---
 .../admin/src/components/CustomTable/Row.js   |  2 +-
 .../ComponentSelect/MultipleMenuList.js       |  2 +-
 .../src/components/ComponentSelect/Value.js   |  2 +-
 .../src/components/ComponentSelect/index.js   |  4 +-
 .../containers/DataManagerProvider/index.js   | 43 +++++++++-------
 .../admin/src/containers/LeftMenu/index.js    | 49 ++++++++++---------
 .../src/containers/TempListView/index.js      | 39 ++++++++-------
 7 files changed, 74 insertions(+), 67 deletions(-)

diff --git a/packages/strapi-plugin-content-manager/admin/src/components/CustomTable/Row.js b/packages/strapi-plugin-content-manager/admin/src/components/CustomTable/Row.js
index 44888f2d13..be7170802c 100644
--- a/packages/strapi-plugin-content-manager/admin/src/components/CustomTable/Row.js
+++ b/packages/strapi-plugin-content-manager/admin/src/components/CustomTable/Row.js
@@ -31,7 +31,7 @@ const getDisplayedValue = (type, value, name) => {
     case 'time':
     case 'datetime':
     case 'timestamp': {
-      if (value === null) {
+      if (value == null) {
         return '-';
       }
 
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/MultipleMenuList.js b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/MultipleMenuList.js
index 3cd0fb6ff3..51dd9076c2 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/MultipleMenuList.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/MultipleMenuList.js
@@ -88,7 +88,7 @@ const MultipleMenuList = ({
               <div style={{ marginTop: 3 }}>
                 <CheckboxWrapper>
                   <Label
-                    htmlFor="overrideReactSelectBehaviour"
+                    htmlFor="overrideReactSelectBehavior"
                     onClick={() => {
                       handleChangeCategory({ target });
                     }}
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/Value.js b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/Value.js
index cda89838b6..3684a9e4a7 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/Value.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/Value.js
@@ -35,7 +35,7 @@ const Value = ({ children, ...props }) => {
   const message =
     valueLength > '0'
       ? getTrad('components.componentSelect.value-components')
-      : getTrad('components.componentSelect.value-components');
+      : getTrad('components.componentSelect.value-component');
 
   return (
     <SingleValue {...props}>
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/index.js b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/index.js
index 8cc1f57b0e..34a35609d8 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/index.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/components/ComponentSelect/index.js
@@ -20,8 +20,7 @@ const ComponentSelect = ({
   // Create a ref in order to access the StateManager
   // So we can close the menu after clicking on a menu item
   // This allows us to get rid of the menuIsOpen state management
-  // So we let the lib taking care of it
-  // It's not the best practice but it is efficient
+  // So we let the custom components taking care of it
   const ref = useRef();
   const handleChange = (inputValue, actionMeta) => {
     const { action } = actionMeta;
@@ -54,7 +53,6 @@ const ComponentSelect = ({
       value={{ label: value, value }}
       options={[]}
       ref={ref}
-      // menuIsOpen
       refState={ref}
       components={{
         MenuList: MenuListCompo,
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/index.js b/packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/index.js
index 7ea76b7af7..7da34e00b4 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/index.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/index.js
@@ -37,7 +37,7 @@ import getTrad from '../../utils/getTrad';
 const DataManagerProvider = ({ allIcons, children }) => {
   const [reducerState, dispatch] = useReducer(reducer, initialState, init);
   const [infoModals, toggleInfoModal] = useState({ cancel: false });
-  const { updatePlugin } = useGlobalContext();
+  const { currentEnvironment, updatePlugin } = useGlobalContext();
   const {
     components,
     contentTypes,
@@ -54,6 +54,8 @@ const DataManagerProvider = ({ allIcons, children }) => {
   const componentMatch = useRouteMatch(
     `/plugins/${pluginId}/component-categories/:categoryUid/:componentUid`
   );
+  const isInDevelopmentMode = currentEnvironment === 'development';
+
   const isInContentTypeView = contentTypeMatch !== null;
   const firstKeyToMainSchema = isInContentTypeView
     ? 'contentType'
@@ -424,8 +426,6 @@ const DataManagerProvider = ({ allIcons, children }) => {
     });
   };
 
-  console.log({ modifiedData });
-
   return (
     <DataManagerContext.Provider
       value={{
@@ -449,6 +449,7 @@ const DataManagerProvider = ({ allIcons, children }) => {
         deleteCategory,
         deleteData,
         editCategory,
+        isInDevelopmentMode,
         initialData,
         isInContentTypeView,
         modifiedData,
@@ -467,22 +468,26 @@ const DataManagerProvider = ({ allIcons, children }) => {
       ) : (
         <>
           {children}
-          <FormModal />
-          <PopUpWarning
-            isOpen={infoModals.cancel}
-            toggleModal={toggleModalCancel}
-            content={{
-              message: getTrad(
-                `popUpWarning.bodyMessage.cancel-modifications${
-                  didModifiedComponents ? '.with-components' : ''
-                }`
-              ),
-            }}
-            popUpWarningType="danger"
-            onConfirm={() => {
-              cancelChanges();
-            }}
-          />
+          {isInDevelopmentMode && (
+            <>
+              <FormModal />
+              <PopUpWarning
+                isOpen={infoModals.cancel}
+                toggleModal={toggleModalCancel}
+                content={{
+                  message: getTrad(
+                    `popUpWarning.bodyMessage.cancel-modifications${
+                      didModifiedComponents ? '.with-components' : ''
+                    }`
+                  ),
+                }}
+                popUpWarningType="danger"
+                onConfirm={() => {
+                  cancelChanges();
+                }}
+              />
+            </>
+          )}
         </>
       )}
     </DataManagerContext.Provider>
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/containers/LeftMenu/index.js b/packages/strapi-plugin-content-type-builder/admin/src/containers/LeftMenu/index.js
index 3063060571..3d609b7b34 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/containers/LeftMenu/index.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/containers/LeftMenu/index.js
@@ -27,18 +27,17 @@ function LeftMenu({ wait }) {
     components,
     componentsGroupedByCategory,
     contentTypes,
+    isInDevelopmentMode,
     sortedContentTypesList,
   } = useDataManager();
-  const { currentEnvironment, formatMessage } = useGlobalContext();
+  const { formatMessage } = useGlobalContext();
   const { push } = useHistory();
 
-  const isProduction = currentEnvironment === 'production';
-
   const componentsData = sortBy(
     Object.keys(componentsGroupedByCategory).map(category => ({
       name: category,
       title: category,
-      isEditable: true,
+      isEditable: isInDevelopmentMode,
       onClickEdit: (e, data) => {
         e.stopPropagation();
         const search = makeSearch({
@@ -95,16 +94,17 @@ function LeftMenu({ wait }) {
         id: `${pluginId}.menu.section.models.name.`,
       },
       searchable: true,
-      customLink: {
-        Component: CustomLink,
-        componentProps: {
-          disabled: isProduction,
-          id: `${pluginId}.button.model.create`,
-          onClick: () => {
-            handleClickOpenModal('contentType');
-          },
-        },
-      },
+      customLink: isInDevelopmentMode
+        ? {
+            Component: CustomLink,
+            componentProps: {
+              id: `${pluginId}.button.model.create`,
+              onClick: () => {
+                handleClickOpenModal('contentType');
+              },
+            },
+          }
+        : null,
       links: sortedContentTypesList,
     },
     {
@@ -113,16 +113,17 @@ function LeftMenu({ wait }) {
         id: `${pluginId}.menu.section.components.name.`,
       },
       searchable: true,
-      customLink: {
-        Component: CustomLink,
-        componentProps: {
-          disabled: isProduction,
-          id: `${pluginId}.button.component.create`,
-          onClick: () => {
-            handleClickOpenModal('component');
-          },
-        },
-      },
+      customLink: isInDevelopmentMode
+        ? {
+            Component: CustomLink,
+            componentProps: {
+              id: `${pluginId}.button.component.create`,
+              onClick: () => {
+                handleClickOpenModal('component');
+              },
+            },
+          }
+        : null,
       links: componentsData,
     },
   ];
diff --git a/packages/strapi-plugin-content-type-builder/admin/src/containers/TempListView/index.js b/packages/strapi-plugin-content-type-builder/admin/src/containers/TempListView/index.js
index 6783d28210..5009cc1faa 100644
--- a/packages/strapi-plugin-content-type-builder/admin/src/containers/TempListView/index.js
+++ b/packages/strapi-plugin-content-type-builder/admin/src/containers/TempListView/index.js
@@ -37,6 +37,7 @@ const ListPage = () => {
     isInContentTypeView,
     // removeAttribute,
     // removeComponentFromDynamicZone,
+    isInDevelopmentMode,
     submitData,
     toggleModalCancel,
   } = useDataManager();
@@ -168,26 +169,28 @@ const ListPage = () => {
   const label = get(modifiedData, [firstMainDataPath, 'schema', 'name'], '');
 
   const headerProps = {
-    actions: getActions(),
+    actions: isInDevelopmentMode ? getActions() : [],
     title: {
       label,
-      cta: {
-        icon: 'pencil-alt',
-        onClick: async () => {
-          await wait();
+      cta: isInDevelopmentMode
+        ? {
+            icon: 'pencil-alt',
+            onClick: async () => {
+              await wait();
 
-          push({
-            search: makeSearch({
-              modalType: firstMainDataPath,
-              actionType: 'edit',
-              settingType: 'base',
-              forTarget: firstMainDataPath,
-              targetUid,
-              headerDisplayName: label,
-            }),
-          });
-        },
-      },
+              push({
+                search: makeSearch({
+                  modalType: firstMainDataPath,
+                  actionType: 'edit',
+                  settingType: 'base',
+                  forTarget: firstMainDataPath,
+                  targetUid,
+                  headerDisplayName: label,
+                }),
+              });
+            },
+          }
+        : null,
     },
     content: getDescription(),
   };
@@ -210,7 +213,7 @@ const ListPage = () => {
     onClick: () => handleClickAddAttributeMainData(),
   };
 
-  const listActions = [{ ...addButtonProps }];
+  const listActions = isInDevelopmentMode ? [{ ...addButtonProps }] : [];
 
   const convertDataToArray = () => {
     return Object.keys(attributes).map((key, index) => {