diff --git a/src/containers/FinancialStatements/BalanceSheet/components.js b/src/containers/FinancialStatements/BalanceSheet/components.js index 8c86f1c88..cf135ae5d 100644 --- a/src/containers/FinancialStatements/BalanceSheet/components.js +++ b/src/containers/FinancialStatements/BalanceSheet/components.js @@ -1,7 +1,6 @@ import React from 'react'; import { Button } from '@blueprintjs/core'; -import { useAppIntlContext } from 'components/AppIntlProvider'; import { FormattedMessage as T, Icon, If } from 'components'; import { useBalanceSheetContext } from './BalanceSheetProvider'; @@ -59,10 +58,8 @@ export const useBalanceSheetColumns = () => { balanceSheet: { table }, } = useBalanceSheetContext(); - const { direction } = useAppIntlContext() - return React.useMemo( - () => dynamicColumns(direction, table.columns, table.rows), - [direction, table], + () => dynamicColumns(table.columns, table.rows), + [table], ); }; diff --git a/src/containers/FinancialStatements/BalanceSheet/dynamicColumns.js b/src/containers/FinancialStatements/BalanceSheet/dynamicColumns.js index ff0134d32..119fec54c 100644 --- a/src/containers/FinancialStatements/BalanceSheet/dynamicColumns.js +++ b/src/containers/FinancialStatements/BalanceSheet/dynamicColumns.js @@ -16,10 +16,11 @@ const getReportColWidth = (data, accessor, headerText) => { ); }; + /** * Account name column mapper. */ -const accountNameMapper = R.curry((direction, data, column) => { +const accountNameMapper = R.curry((data, column) => { const accessor = getTableCellValueAccessor(column.cell_index); const width = getReportColWidth(data, accessor, column.label); @@ -30,7 +31,7 @@ const accountNameMapper = R.curry((direction, data, column) => { className: column.key, textOverview: true, width: Math.max(width, 300), - sticky: direction === 'rtl' ? Align.Right : Align.Left, + sticky: Align.Left, }; }); @@ -312,9 +313,9 @@ const isMatchesDateRange = (r) => R.match(/^date-range/g, r).length > 0; /** * Dynamic column mapper. */ -const dynamicColumnMapper = R.curry((direction, data, column) => { +const dynamicColumnMapper = R.curry((data, column) => { const indexTotalMapper = totalMapper(data); - const indexAccountNameMapper = accountNameMapper(direction, data); + const indexAccountNameMapper = accountNameMapper(data); const indexDatePeriodMapper = dateRangeMapper(data); return R.compose( @@ -327,6 +328,6 @@ const dynamicColumnMapper = R.curry((direction, data, column) => { /** * Cash flow dynamic columns. */ -export const dynamicColumns = (direction, columns, data) => { - return R.map(dynamicColumnMapper(direction, data), columns); +export const dynamicColumns = (columns, data) => { + return R.map(dynamicColumnMapper(data), columns); }; diff --git a/src/containers/FinancialStatements/ProfitLossSheet/dynamicColumns.js b/src/containers/FinancialStatements/ProfitLossSheet/dynamicColumns.js index 7daa3bbb4..337b1ba00 100644 --- a/src/containers/FinancialStatements/ProfitLossSheet/dynamicColumns.js +++ b/src/containers/FinancialStatements/ProfitLossSheet/dynamicColumns.js @@ -295,7 +295,7 @@ const totalColumnCompose = R.curry((data, column) => { /** * Account name column mapper. */ -const accountNameColumn = R.curry((direction, data, column) => { +const accountNameColumn = R.curry((data, column) => { const accessor = getTableCellValueAccessor(column.cell_index); const width = getReportColWidth(data, accessor, column.label); @@ -306,7 +306,7 @@ const accountNameColumn = R.curry((direction, data, column) => { className: column.key, textOverview: true, width: Math.max(width, 300), - sticky: direction === 'rtl' ? Align.Right : Align.Left, + sticky: Align.Left, }; }); @@ -360,9 +360,9 @@ const isMatchesDateRange = (r) => R.match(/^date-range/g, r).length > 0; * @param {} data * @param {} column */ -const dynamicColumnMapper = R.curry((direction, data, column) => { +const dynamicColumnMapper = R.curry((data, column) => { const indexTotalColumn = totalColumnCompose(data); - const indexAccountNameColumn = accountNameColumn(direction, data); + const indexAccountNameColumn = accountNameColumn(data); const indexDatePeriodMapper = dateRangeColumn(data); return R.compose( @@ -373,8 +373,11 @@ const dynamicColumnMapper = R.curry((direction, data, column) => { }); /** - * Retrieves the dynamic columns of profit/loss sheet. + * + * @param {*} columns + * @param {*} data + * @returns */ -export const dynamicColumns = (direction, columns, data) => { - return R.map(dynamicColumnMapper(direction, data), columns); +export const dynamicColumns = (columns, data) => { + return R.map(dynamicColumnMapper(data), columns); }; diff --git a/src/containers/FinancialStatements/ProfitLossSheet/hooks.js b/src/containers/FinancialStatements/ProfitLossSheet/hooks.js index 8ca8ea7d2..aab7adb02 100644 --- a/src/containers/FinancialStatements/ProfitLossSheet/hooks.js +++ b/src/containers/FinancialStatements/ProfitLossSheet/hooks.js @@ -1,23 +1,19 @@ import React from 'react'; import { dynamicColumns } from './dynamicColumns'; - import { useProfitLossSheetContext } from './ProfitLossProvider'; -import { useAppIntlContext } from '../../../components/AppIntlProvider'; /** * Retrieves the profit/loss table columns. - * @returns + * @returns */ export const useProfitLossSheetColumns = () => { const { profitLossSheet: { table }, } = useProfitLossSheetContext(); - const { direction } = useAppIntlContext(); - return React.useMemo( - () => dynamicColumns(direction, table.columns, table.rows), - [direction, table], + () => dynamicColumns(table.columns || [], table.rows || []), + [table], ); };