m/metamask-extension
1
0
Fork 0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-12-23 09:52:26 +01:00
Code

Ensure the advanced gas modal displays a zero priority fee if that is how it is set in the transaction (#17559)

Browse Source 
Co-authored-by: Pedro Figueiredo <pedro.figueiredo@consensys.net>
This commit is contained in:
Dan J Miller 2023-02-03 08:00:49 -08:00 committed by GitHub
parent 6b076fa346
commit 0c24448037
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
ui/hooks/gasFeeInput/useMaxPriorityFeePerGasInput.js
View File

@ -7,6 +7,8 @@ import { isLegacyTransaction } from '../../helpers/utils/transactions.util';
import { hexWEIToDecGWEI } from '../../../shared/modules/conversion.utils';
import { feeParamsAreCustom, getGasFeeEstimate } from './utils';
const isNullOrUndefined = (value) => value === null || value === undefined;
const getMaxPriorityFeePerGasFromTransaction = (
transaction,
gasFeeEstimates,
@ -17,9 +19,8 @@ const getMaxPriorityFeePerGasFromTransaction = (
}
const { maxPriorityFeePerGas, maxFeePerGas, gasPrice } =
transaction?.txParams || {};
return Number(
hexWEIToDecGWEI(maxPriorityFeePerGas || maxFeePerGas || gasPrice),
);
const feeInHexWei = maxPriorityFeePerGas || maxFeePerGas || gasPrice;
return feeInHexWei ? Number(hexWEIToDecGWEI(feeInHexWei)) : null;
};
/**
@ -50,17 +51,20 @@ export function useMaxPriorityFeePerGasInput({
const initialMaxPriorityFeePerGas = supportsEIP1559
? getMaxPriorityFeePerGasFromTransaction(transaction, gasFeeEstimates)
: 0;
: null;
const [maxPriorityFeePerGas, setMaxPriorityFeePerGas] = useState(() => {
if (initialMaxPriorityFeePerGas && feeParamsAreCustom(transaction)) {
if (
!isNullOrUndefined(initialMaxPriorityFeePerGas) &&
feeParamsAreCustom(transaction)
) {
return initialMaxPriorityFeePerGas;
}
return null;
});
useEffect(() => {
if (supportsEIP1559 && initialMaxPriorityFeePerGas) {
if (supportsEIP1559 && !isNullOrUndefined(initialMaxPriorityFeePerGas)) {
setMaxPriorityFeePerGas(initialMaxPriorityFeePerGas);
}
}, [initialMaxPriorityFeePerGas, setMaxPriorityFeePerGas, supportsEIP1559]);

11
ui/hooks/gasFeeInput/useMaxPriorityFeePerGasInput.test.js
View File

@ -102,4 +102,15 @@ describe('useMaxPriorityFeePerGasInput', () => {
});
expect(result.current.maxPriorityFeePerGas).toBe(100);
});
it('returns maxPriorityFeePerGas from transaction if it is 0', () => {
const { result } = renderUseMaxPriorityFeePerGasInputHook({
transaction: {
txParams: {
maxPriorityFeePerGas: '0x0',
},
},
});
expect(result.current.maxPriorityFeePerGas).toBe(0);
});
});