Skip to content

Fix TemplateAliasProcessor#template_name arity#2010

Open
viralpraxis wants to merge 2 commits intopresidentbeef:mainfrom
viralpraxis:fix-template-alias-processor-template-name-arity
Open

Fix TemplateAliasProcessor#template_name arity#2010
viralpraxis wants to merge 2 commits intopresidentbeef:mainfrom
viralpraxis:fix-template-alias-processor-template-name-arity

Conversation

@viralpraxis
Copy link
Copy Markdown
Contributor

<snip>/gems/brakeman-8.0.2/lib/brakeman/processors/lib/render_helper.rb:26 - wrong number of arguments (given 0, expected 1)

```shell
<snip>/gems/brakeman-8.0.2/lib/brakeman/processors/lib/render_helper.rb:26 - wrong number of arguments (given 0, expected 1)
```
@presidentbeef
Copy link
Copy Markdown
Owner

I don't think this is the right fix. I think the call in render_helper is... wrong? Or something else is going on.

@viralpraxis
Copy link
Copy Markdown
Contributor Author

viralpraxis commented Feb 26, 2026

I don't think this is the right fix. I think the call in render_helper is... wrong? Or something else is going on.

Yep, the call was wrong. We should always pass the action name as an argument. Fixed

@viralpraxis
Copy link
Copy Markdown
Contributor Author

@presidentbeef can we proceed with this one?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants