Change MethodAccess to MethodCall in query example #20979

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.

Sign up for GitHub

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

Jump to bottom

Closed

myvyang wants to merge 2 commits into github:main from myvyang:patch-1

+17 −13

myvyang commented Dec 8, 2025

MethodAccess has been deprecated.


          Change MethodAccess to MethodCall in query example

21fc06c

MethodAccess has been deprecated.

myvyang requested a review from a team as a code owner

December 8, 2025 07:27

github-actions bot added the documentation label

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved

owen-mc reviewed

View reviewed changes

docs/codeql/codeql-language-guides/basic-query-for-java-code.rst Outdated Show resolved Hide resolved


          Also rename variable from ma to mc

cb31f82

This is slightly less confusing, given the type name is now `MethodCall`.

owen-mc reviewed

View reviewed changes

Contributor

owen-mc left a comment

Thanks for this contribution. I tried to change the variable name from ma to mc as well, but something went wrong with the UI and it applied the patches wrongly. Please delete the commit that I added, and add another commit changing the variable name.

Author

myvyang commented Dec 9, 2025

Thanks for this contribution. I tried to change the variable name from ma to mc as well, but something went wrong with the UI and it applied the patches wrongly. Please delete the commit that I added, and add another commit changing the variable name.

The UI bug was caused by a global find-and-replace operation that accidentally changed "image" to "imcge" in image URLs. I've created a new PR that also fixes the incorrect "MethodAccess" references in other files. #20992

myvyang closed this

myvyang mentioned this pull request

Change MethodAccess to MethodCall in query example. #20992

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels