Skip to content

Template: fix "Failed to delete buffer" error#307

Closed
JanTvrdik wants to merge 5 commits intonette:masterfrom
JanTvrdik:patch-1
Closed

Template: fix "Failed to delete buffer" error#307
JanTvrdik wants to merge 5 commits intonette:masterfrom
JanTvrdik:patch-1

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 5 times, most recently from 48ae1fe to 20b2769 Compare January 3, 2023 01:51
@dg dg force-pushed the master branch 16 times, most recently from 038b1aa to 8976643 Compare January 17, 2023 21:18
@dg dg force-pushed the master branch 9 times, most recently from 5cb9f21 to 68ff96c Compare January 26, 2023 23:41
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