Skip to content

Conversation

@JanTvrdik
Copy link
Contributor

@JanTvrdik JanTvrdik commented Jun 26, 2022

  • bug fix: yes
  • BC break: no

This fixes error

ob_end_clean(): Failed to delete buffer. No buffer to delete

but I'm not sure if this is the best way to do it.

It happens with combination of

  • using Engine::renderToString()
  • having configured exception handler $latte->setExceptionHandler(fn(Throwable $e) => throw $e);
  • error happends inside {try}...{/try}

Latte 3.0.2, standalone usage

@dg dg force-pushed the master branch 2 times, most recently from da9a256 to f446bf5 Compare July 9, 2022 23:25
@dg dg force-pushed the master branch 4 times, most recently from 466080e to 13436ee Compare September 12, 2022 22:36
@dg dg force-pushed the master branch 8 times, most recently from 0ef5757 to bdd2655 Compare October 19, 2022 16:06
@dg dg force-pushed the master branch 2 times, most recently from 04c3089 to e3a5b8c Compare October 23, 2022 16:26
@dg dg force-pushed the master branch 3 times, most recently from 9f8324c to 2f638f4 Compare November 6, 2022 18:14
@dg dg force-pushed the master branch 3 times, most recently from 2ff766d to c430568 Compare November 28, 2022 00:21
@dg dg force-pushed the master branch 3 times, most recently from 0415418 to b093a44 Compare December 21, 2022 13:21
@dg dg force-pushed the master branch 4 times, most recently from 8ebabaf to ac219c9 Compare August 29, 2023 12:04
@dg dg force-pushed the master branch 10 times, most recently from 3db2897 to e24c412 Compare September 8, 2023 10:05
@dg dg force-pushed the master branch 2 times, most recently from 46ae380 to 65edbf0 Compare October 2, 2023 21:12
@dg dg force-pushed the master branch 6 times, most recently from 514617f to 794f252 Compare October 11, 2023 13:43
@dg dg closed this in f4742f8 Nov 8, 2023
dg added a commit that referenced this pull request Nov 8, 2023
dg added a commit that referenced this pull request Nov 9, 2023
dg added a commit that referenced this pull request Nov 9, 2023
dg added a commit that referenced this pull request Nov 9, 2023
dg added a commit that referenced this pull request Nov 11, 2023
dg added a commit that referenced this pull request Nov 11, 2023
dg added a commit that referenced this pull request Nov 11, 2023
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