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

Fix missing render context arg #1226

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Conversation

phillmac
Copy link
Contributor

The context param is a mandatory arg to render(). Causes PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function Twig\Template::render(), 0 passed in ... if omitted.

The context param is a mandatory arg to render(). Causes PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function Twig\Template::render(), 0 passed in ... if omitted.
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #1226 (a487dfe) into master (adb574f) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #1226   +/-   ##
=========================================
  Coverage     69.64%   69.64%           
  Complexity     1987     1987           
=========================================
  Files           169      169           
  Lines          7677     7677           
=========================================
  Hits           5347     5347           
  Misses         2330     2330           
Files Changed Coverage Δ
.../Error/Handler/AuthCompromisedExceptionHandler.php 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lcharette lcharette changed the base branch from master to hotfix September 14, 2023 00:22
@lcharette lcharette merged commit 4ebd788 into userfrosting:hotfix Sep 14, 2023
24 of 25 checks passed
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