Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Wrong function name in "077-supplement-for-088.ts" #29907

Open
Teng-Web3 opened this issue Jan 25, 2025 · 1 comment
Open

[Bug]: Wrong function name in "077-supplement-for-088.ts" #29907

Teng-Web3 opened this issue Jan 25, 2025 · 1 comment
Labels
external-contributor regression-prod-12.10.1 Regression bug that was found in production in release 12.10.1 type-bug

Comments

@Teng-Web3
Copy link

Describe the bug

I'm reading the metamask source code recently and found a bug:
The function name is currently named "transformState077For086" in "077-supplement-for-088.ts"
export default function transformState077For086
It called in file "077.js"
import transformState077For088 from './077-supplements/077-supplement-for-088'; newState = transformState077For088(newState);
Although this does not affect normal calls, because the imported function name and the default exported function name can be inconsistent.
But this is still a issue that should be fixed because:
1.Incorrect function names affect code readability
2.Wrong function names will be displayed during debugging
3.It violates the principle of code consistency

Expected behavior

Rename named "transformState077For086" to "transformState077For088" in "077-supplement-for-088.ts"

Screenshots/Recordings

No response

Steps to reproduce

No need.

Error messages or log output

Detection stage

In production (default)

Version

12.10.1

Build type

None

Browser

Chrome

Operating system

Windows

Hardware wallet

No response

Additional context

No response

Severity

No response

@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by team Jan 25, 2025
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by severity Jan 25, 2025
@metamaskbot metamaskbot added external-contributor regression-prod-12.10.1 Regression bug that was found in production in release 12.10.1 labels Jan 25, 2025
@Teng-Web3
Copy link
Author

Please assign it to me if it need to fix

@Teng-Web3 Teng-Web3 changed the title [Bug]: [Bug]: Wrong function name in "077-supplement-for-088.ts" Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contributor regression-prod-12.10.1 Regression bug that was found in production in release 12.10.1 type-bug
Projects
Status: To be fixed
Status: To be fixed
Development

No branches or pull requests

2 participants