Route a method whose name is a reserved word.

[amvanbaren]

Pull Request Checklist

• Have you read How to write the perfect pull request?
• Have you read through the contributor guidelines?
• Have you referenced any issues you're fixing using commit message keywords?
• Have you added copyright headers to new files?
• Have you checked that both Scala and Java APIs are updated?
• Have you updated the documentation for both Scala and Java sections?
• Have you added tests for any changed functionality?

Helpful things

Fixes

Fixes #11439

Purpose

What does this PR do?

• It escapes method names that are a reserved word with backticks in files generated from the routes file.
• It parses routes with backticks around the method name.

Background Context

Why did you take this approach?

@mkurz commented #11439 (comment) that we might be able to escape all the method we generate with backticks. While I was working on the issue I discovered there already is a safeKeyword method. I added a safeMethod method that is almost identical in functionality.

@proudust created a project that reproduces the issue. A couple of routes in the project have backticks around the method name. That's why I added functionality to the RoutesFileParser to parse routes with backticks around the method name.
I don't know if this is desirable. By allowing backticks around the method name a route like GET /space controllers.FooController.` `() should also be valid. @mkurz What do you think?

References

Are there any relevant issues / PRs / mailing lists discussions?

[mkurz]

@amvanbaren Can you please resolve the conflicts by rebasing on current main branch? Thanks!

[mkurz]

Tagging this for 2.9.1

[amvanbaren]

@mkurz I've rebased on the current main branch.

[mkurz]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased