feat(S&P): fix navigation bar if warehouses or branches feature.

src/components/BranchSuggestField.js

@@ -39,10 +39,10 @@ export default function BranchSuggestField({
       <MenuItem
         // active={modifiers.active}
         disabled={modifiers.disabled}
-        label={branch.code.toString()}
+        label={branch.code}
         key={branch.id}
         onClick={handleClick}
-        text={branch.name.toString()}
+        text={branch.name}
       />
     );
   };

src/containers/Accounting/MakeJournal/MakeJournalEntriesHeaderFields.js

@@ -195,32 +195,35 @@ function MakeJournalEntriesHeader({
       </FastField>
 
       {/*------------ Currency  -----------*/}
-      {/* <FeatureCan feature={Features.ManualJournal}> */}
-      <FastField
-        name={'currency_code'}
-        currencies={currencies}
-        shouldUpdate={currenciesFieldShouldUpdate}
-      >
-        {({ form, field: { value }, meta: { error, touched } }) => (
-          <FormGroup
-            label={<T id={'currency'} />}
-            className={classNames('form-group--currency', CLASSES.FILL)}
-            inline={true}
-          >
-            <CurrencySelectList
-              currenciesList={currencies}
-              selectedCurrencyCode={value}
-              onCurrencySelected={(currencyItem) => {
-                form.setFieldValue('currency_code', currencyItem.currency_code);
-                form.setFieldValue('exchange_rate', '');
-                setSelactJournalCurrency(currencyItem);
-              }}
-              defaultSelectText={value}
-            />
-          </FormGroup>
-        )}
-      </FastField>
-      {/* </FeatureCan> */}
+      <FeatureCan feature={Features.ManualJournal}>
+        <FastField
+          name={'currency_code'}
+          currencies={currencies}
+          shouldUpdate={currenciesFieldShouldUpdate}
+        >
+          {({ form, field: { value }, meta: { error, touched } }) => (
+            <FormGroup
+              label={<T id={'currency'} />}
+              className={classNames('form-group--currency', CLASSES.FILL)}
+              inline={true}
+            >
+              <CurrencySelectList
+                currenciesList={currencies}
+                selectedCurrencyCode={value}
+                onCurrencySelected={(currencyItem) => {
+                  form.setFieldValue(
+                    'currency_code',
+                    currencyItem.currency_code,
+                  );
+                  form.setFieldValue('exchange_rate', '');
+                  setSelactJournalCurrency(currencyItem);
+                }}
+                defaultSelectText={value}
+              />
+            </FormGroup>
+          )}
+        </FastField>
+      </FeatureCan>
 
       {/* ----------- Exchange rate ----------- */}
       <If condition={isForeignJournal}>

src/containers/Purchases/Bills/BillForm/BillFormTopBar.js

@@ -38,7 +38,7 @@ export default function BillFormTopBar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }
 

src/containers/Purchases/CreditNotes/CreditNoteForm/VendorCreditNoteFormTopBar.js

@@ -39,7 +39,7 @@ export default function VendorCreditNoteFormTopBar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }
 

src/containers/Sales/CreditNotes/CreditNoteForm/CreditNoteFormTopBar.js

@@ -38,7 +38,7 @@ export default function CreditNoteFormTopbar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }
 

src/containers/Sales/Estimates/EstimateForm/EstimtaeFormTopBar.js

@@ -39,7 +39,7 @@ export default function EstimtaeFormTopBar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }
 

src/containers/Sales/Invoices/InvoiceForm/InvoiceFormTopBar.js

@@ -40,7 +40,7 @@ export default function InvoiceFormTopBar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }
   return (

src/containers/Sales/Receipts/ReceiptForm/ReceiptFormTopbar.js

@@ -40,7 +40,7 @@ export default function ReceiptFormTopBar() {
   useSetPrimaryBranchToForm();
 
   // Can't display the navigation bar if warehouses or branches feature is not enabled.
-  if (!featureCan(Features.Warehouses) || !featureCan(Features.Branches)) {
+  if (!featureCan(Features.Warehouses) && !featureCan(Features.Branches)) {
     return null;
   }