Skip to content

Commit

Permalink
Rename handle methods to better match property names
Browse files Browse the repository at this point in the history
  • Loading branch information
@dlabrecq
dlabrecq committed May 4, 2023
1 parent 62b9f96 commit 5ac3d08
Show file tree
Hide file tree
Showing 13 changed files with 134 additions and 132 deletions.
28 changes: 14 additions & 14 deletions src/routes/views/details/awsDetails/awsDetails.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,13 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { ExportModal } from 'routes/views/components/export';
import { getGroupByCostCategory, getGroupByOrgValue, getGroupByTagKey } from 'routes/views/utils/groupBy';
import {
handleCostTypeSelected,
handleCurrencySelected,
handleFilterAdded,
handleFilterRemoved,
handlePerPageSelect,
handleSetPage,
handleSort,
handleOnCostTypeSelected,
handleOnCurrencySelected,
handleOnFilterAdded,
handleOnFilterRemoved,
handleOnPerPageSelect,
handleOnSetPage,
handleOnSort,
} from 'routes/views/utils/handles';
import { filterProviders, hasCurrentMonthData } from 'routes/views/utils/providers';
import { getRouteForQuery } from 'routes/views/utils/query';
Expand Down Expand Up @@ -199,8 +199,8 @@ class AwsDetails extends React.Component<AwsDetailsProps, AwsDetailsState> {
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => handlePerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleSetPage(query, router, report, pageNumber)}
onPerPageSelect={(event, perPage) => handleOnPerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleOnSetPage(query, router, report, pageNumber)}
page={page}
perPage={limit}
titles={{
Expand Down Expand Up @@ -239,7 +239,7 @@ class AwsDetails extends React.Component<AwsDetailsProps, AwsDetailsState> {
isAllSelected={isAllSelected}
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSelected={this.handleSelected}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
selectedItems={selectedItems}
Expand Down Expand Up @@ -273,8 +273,8 @@ class AwsDetails extends React.Component<AwsDetailsProps, AwsDetailsState> {
itemsTotal={itemsTotal}
onBulkSelected={this.handleBulkSelected}
onExportClicked={this.handleExportModalOpen}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
selectedItems={selectedItems}
Expand Down Expand Up @@ -393,8 +393,8 @@ class AwsDetails extends React.Component<AwsDetailsProps, AwsDetailsState> {
costType={costType}
currency={currency}
groupBy={groupById}
onCostTypeSelected={value => handleCostTypeSelected(query, router, value)}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCostTypeSelected={value => handleOnCostTypeSelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
onGroupBySelected={this.handleGroupBySelected}
report={report}
/>
Expand Down
4 changes: 2 additions & 2 deletions src/routes/views/details/awsDetails/detailsHeader.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ const resourcePathsType = ResourcePathsType.aws;
const tagPathsType = TagPathsType.aws;

class DetailsHeaderBase extends React.Component<DetailsHeaderProps, any> {
private handleCostTypeSelected = (value: string) => {
private handleOnCostTypeSelected = (value: string) => {
const { onCostTypeSelected } = this.props;

if (onCostTypeSelected) {
Expand Down Expand Up @@ -115,7 +115,7 @@ class DetailsHeaderBase extends React.Component<DetailsHeaderProps, any> {
tagPathsType={tagPathsType}
/>
<div style={styles.costType}>
<CostType costType={costType} onSelect={this.handleCostTypeSelected} />
<CostType costType={costType} onSelect={this.handleOnCostTypeSelected} />
</div>
</div>
{showContent && (
Expand Down
24 changes: 12 additions & 12 deletions src/routes/views/details/azureDetails/azureDetails.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { ExportModal } from 'routes/views/components/export';
import { getGroupByTagKey } from 'routes/views/utils/groupBy';
import {
handleCurrencySelected,
handleFilterAdded,
handleFilterRemoved,
handlePerPageSelect,
handleSetPage,
handleSort,
handleOnCurrencySelected,
handleOnFilterAdded,
handleOnFilterRemoved,
handleOnPerPageSelect,
handleOnSetPage,
handleOnSort,
} from 'routes/views/utils/handles';
import { filterProviders, hasCurrentMonthData } from 'routes/views/utils/providers';
import { getRouteForQuery } from 'routes/views/utils/query';
Expand Down Expand Up @@ -186,8 +186,8 @@ class AzureDetails extends React.Component<AzureDetailsProps, AzureDetailsState>
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => handlePerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleSetPage(query, router, report, pageNumber)}
onPerPageSelect={(event, perPage) => handleOnPerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleOnSetPage(query, router, report, pageNumber)}
page={page}
perPage={limit}
titles={{
Expand Down Expand Up @@ -216,7 +216,7 @@ class AzureDetails extends React.Component<AzureDetailsProps, AzureDetailsState>
isAllSelected={isAllSelected}
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSelected={this.handleSelected}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
selectedItems={selectedItems}
Expand All @@ -243,8 +243,8 @@ class AzureDetails extends React.Component<AzureDetailsProps, AzureDetailsState>
itemsTotal={itemsTotal}
onBulkSelected={this.handleBulkSelected}
onExportClicked={this.handleExportModalOpen}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
selectedItems={selectedItems}
Expand Down Expand Up @@ -352,7 +352,7 @@ class AzureDetails extends React.Component<AzureDetailsProps, AzureDetailsState>
<DetailsHeader
currency={currency}
groupBy={groupById}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
onGroupBySelected={this.handleGroupBySelected}
report={report}
/>
Expand Down
12 changes: 6 additions & 6 deletions src/routes/views/details/components/breakdown/breakdownBase.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ import { NoData } from 'routes/state/noData';
import { NoProviders } from 'routes/state/noProviders';
import { NotAvailable } from 'routes/state/notAvailable';
import {
handleCostDistributionSelected,
handleCostTypeSelected,
handleCurrencySelected,
handleOnCostDistributionSelected,
handleOnCostTypeSelected,
handleOnCurrencySelected,
} from 'routes/views/utils/handles';
import { hasCurrentMonthData } from 'routes/views/utils/providers';
import { FetchStatus } from 'store/common';
Expand Down Expand Up @@ -303,9 +303,9 @@ class BreakdownBase extends React.Component<BreakdownProps, BreakdownState> {
description={description}
detailsURL={detailsURL}
groupBy={groupBy}
onCostDistributionSelected={value => handleCostDistributionSelected(query, router, value)}
onCostTypeSelected={value => handleCostTypeSelected(query, router, value)}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCostDistributionSelected={value => handleOnCostDistributionSelected(query, router, value)}
onCostTypeSelected={value => handleOnCostTypeSelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
query={query}
report={report}
showCostDistribution={showCostDistribution}
Expand Down
24 changes: 12 additions & 12 deletions src/routes/views/details/gcpDetails/gcpDetails.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { ExportModal } from 'routes/views/components/export';
import { getGroupByTagKey } from 'routes/views/utils/groupBy';
import {
handleCurrencySelected,
handleFilterAdded,
handleFilterRemoved,
handlePerPageSelect,
handleSetPage,
handleSort,
handleOnCurrencySelected,
handleOnFilterAdded,
handleOnFilterRemoved,
handleOnPerPageSelect,
handleOnSetPage,
handleOnSort,
} from 'routes/views/utils/handles';
import { filterProviders, hasCurrentMonthData } from 'routes/views/utils/providers';
import { getRouteForQuery } from 'routes/views/utils/query';
Expand Down Expand Up @@ -186,8 +186,8 @@ class GcpDetails extends React.Component<GcpDetailsProps, GcpDetailsState> {
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => handlePerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleSetPage(query, router, report, pageNumber)}
onPerPageSelect={(event, perPage) => handleOnPerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleOnSetPage(query, router, report, pageNumber)}
page={page}
perPage={limit}
titles={{
Expand Down Expand Up @@ -215,7 +215,7 @@ class GcpDetails extends React.Component<GcpDetailsProps, GcpDetailsState> {
isAllSelected={isAllSelected}
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSelected={this.handleSelected}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
selectedItems={selectedItems}
Expand All @@ -242,8 +242,8 @@ class GcpDetails extends React.Component<GcpDetailsProps, GcpDetailsState> {
itemsTotal={itemsTotal}
onBulkSelected={this.handleBulkSelected}
onExportClicked={this.handleExportModalOpen}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
selectedItems={selectedItems}
Expand Down Expand Up @@ -341,7 +341,7 @@ class GcpDetails extends React.Component<GcpDetailsProps, GcpDetailsState> {
<DetailsHeader
currency={currency}
groupBy={groupById}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
onGroupBySelected={this.handleGroupBySelected}
report={report}
/>
Expand Down
24 changes: 12 additions & 12 deletions src/routes/views/details/ibmDetails/ibmDetails.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { ExportModal } from 'routes/views/components/export';
import { getGroupByTagKey } from 'routes/views/utils/groupBy';
import {
handleCurrencySelected,
handleFilterAdded,
handleFilterRemoved,
handlePerPageSelect,
handleSetPage,
handleSort,
handleOnCurrencySelected,
handleOnFilterAdded,
handleOnFilterRemoved,
handleOnPerPageSelect,
handleOnSetPage,
handleOnSort,
} from 'routes/views/utils/handles';
import { hasCurrentMonthData } from 'routes/views/utils/providers';
import { filterProviders } from 'routes/views/utils/providers';
Expand Down Expand Up @@ -187,8 +187,8 @@ class IbmDetails extends React.Component<IbmDetailsProps, IbmDetailsState> {
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => handlePerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleSetPage(query, router, report, pageNumber)}
onPerPageSelect={(event, perPage) => handleOnPerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleOnSetPage(query, router, report, pageNumber)}
page={page}
perPage={limit}
titles={{
Expand Down Expand Up @@ -217,7 +217,7 @@ class IbmDetails extends React.Component<IbmDetailsProps, IbmDetailsState> {
isAllSelected={isAllSelected}
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSelected={this.handleSelected}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
selectedItems={selectedItems}
Expand All @@ -244,8 +244,8 @@ class IbmDetails extends React.Component<IbmDetailsProps, IbmDetailsState> {
itemsTotal={itemsTotal}
onBulkSelected={this.handleBulkSelected}
onExportClicked={this.handleExportModalOpen}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
selectedItems={selectedItems}
Expand Down Expand Up @@ -343,7 +343,7 @@ class IbmDetails extends React.Component<IbmDetailsProps, IbmDetailsState> {
<DetailsHeader
currency={currency}
groupBy={groupById}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
onGroupBySelected={this.handleGroupBySelected}
report={report}
/>
Expand Down
24 changes: 12 additions & 12 deletions src/routes/views/details/ociDetails/ociDetails.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { ExportModal } from 'routes/views/components/export';
import { getGroupByTagKey } from 'routes/views/utils/groupBy';
import {
handleCurrencySelected,
handleFilterAdded,
handleFilterRemoved,
handlePerPageSelect,
handleSetPage,
handleSort,
handleOnCurrencySelected,
handleOnFilterAdded,
handleOnFilterRemoved,
handleOnPerPageSelect,
handleOnSetPage,
handleOnSort,
} from 'routes/views/utils/handles';
import { filterProviders, hasCurrentMonthData } from 'routes/views/utils/providers';
import { getRouteForQuery } from 'routes/views/utils/query';
Expand Down Expand Up @@ -186,8 +186,8 @@ class OciDetails extends React.Component<OciDetailsProps, OciDetailsState> {
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => handlePerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleSetPage(query, router, report, pageNumber)}
onPerPageSelect={(event, perPage) => handleOnPerPageSelect(query, router, perPage)}
onSetPage={(event, pageNumber) => handleOnSetPage(query, router, report, pageNumber)}
page={page}
perPage={limit}
titles={{
Expand Down Expand Up @@ -216,7 +216,7 @@ class OciDetails extends React.Component<OciDetailsProps, OciDetailsState> {
isAllSelected={isAllSelected}
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSelected={this.handleSelected}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
selectedItems={selectedItems}
Expand All @@ -243,8 +243,8 @@ class OciDetails extends React.Component<OciDetailsProps, OciDetailsState> {
itemsTotal={itemsTotal}
onBulkSelected={this.handleBulkSelected}
onExportClicked={this.handleExportModalOpen}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
selectedItems={selectedItems}
Expand Down Expand Up @@ -352,7 +352,7 @@ class OciDetails extends React.Component<OciDetailsProps, OciDetailsState> {
<DetailsHeader
currency={currency}
groupBy={groupById}
onCurrencySelected={value => handleCurrencySelected(query, router, value)}
onCurrencySelected={value => handleOnCurrencySelected(query, router, value)}
onGroupBySelected={this.handleGroupBySelected}
report={report}
/>
Expand Down
16 changes: 8 additions & 8 deletions src/routes/views/details/ocpBreakdown/optimizationsBreakdown.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import { NotAvailable } from 'routes/state/notAvailable';
import { OptimizationsTable, OptimizationsToolbar } from 'routes/views/components/optimizations';
import { styles } from 'routes/views/optimizations/optimizations.styles';
import { getGroupById, getGroupByValue } from 'routes/views/utils/groupBy';
import { handleFilterAdded, handleFilterRemoved, handleSort } from 'routes/views/utils/handles';
import { handleOnFilterAdded, handleOnFilterRemoved, handleOnSort } from 'routes/views/utils/handles';
import { getOrderById, getOrderByValue } from 'routes/views/utils/orderBy';
import { getRouteForQuery } from 'routes/views/utils/query';
import { createMapStateToProps, FetchStatus } from 'store/common';
Expand Down Expand Up @@ -95,8 +95,8 @@ class OptimizationsBreakdownBase extends React.Component<OptimizationsBreakdownP
isCompact={!isBottom}
isDisabled={isDisabled}
itemCount={count}
onPerPageSelect={(event, perPage) => this.handlePerPageSelect(perPage)}
onSetPage={(event, pageNumber) => this.handleSetPage(pageNumber)}
onPerPageSelect={(event, perPage) => this.handleOnPerPageSelect(perPage)}
onSetPage={(event, pageNumber) => this.handleOnSetPage(pageNumber)}
page={page}
perPage={limit}
titles={{
Expand All @@ -117,7 +117,7 @@ class OptimizationsBreakdownBase extends React.Component<OptimizationsBreakdownP
return (
<OptimizationsTable
isLoading={reportFetchStatus === FetchStatus.inProgress}
onSort={(sortType, isSortAscending) => handleSort(query, router, sortType, isSortAscending)}
onSort={(sortType, isSortAscending) => handleOnSort(query, router, sortType, isSortAscending)}
report={report}
reportQueryString={reportQueryString}
/>
Expand All @@ -136,15 +136,15 @@ class OptimizationsBreakdownBase extends React.Component<OptimizationsBreakdownP
isDisabled={isDisabled}
itemsPerPage={itemsPerPage}
itemsTotal={itemsTotal}
onFilterAdded={filter => handleFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleFilterRemoved(query, router, filter)}
onFilterAdded={filter => handleOnFilterAdded(query, router, filter)}
onFilterRemoved={filter => handleOnFilterRemoved(query, router, filter)}
pagination={this.getPagination(isDisabled)}
query={query}
/>
);
};

private handlePerPageSelect = (perPage: number) => {
private handleOnPerPageSelect = (perPage: number) => {
const { query, router } = this.props;

const newQuery = {
Expand All @@ -155,7 +155,7 @@ class OptimizationsBreakdownBase extends React.Component<OptimizationsBreakdownP
router.navigate(filteredQuery, { replace: true });
};

private handleSetPage = (pageNumber: number) => {
private handleOnSetPage = (pageNumber: number) => {
const { query, report, router } = this.props;

const limit = report && report.meta && report.meta.limit ? report.meta.limit : baseQuery.limit;
Expand Down
Loading

0 comments on commit 5ac3d08

Please sign in to comment.