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

Update all ob_ clean/flush functions #3073

Merged

Conversation

haszi
Copy link
Contributor

@haszi haszi commented Jan 5, 2024

Update all six ob_ clean/flush functions' documentation with the following:

  • describe what each function does in detail
  • differentiate between contents of the output buffer and values returned by the output handler
  • list flags passed to output handlers
  • list flags affecting or, if unexpected, not affecting each function
  • use consistent terminology for output buffer, it's contents and it's handler, and 'turn off' instead of 'destroy' an output buffer
  • add a section on notices issued

Closes the following:
https://bugs.php.net/bug.php?id=64977
https://bugs.php.net/bug.php?id=69404
https://bugs.php.net/bug.php?id=71486
https://bugs.php.net/bug.php?id=74399
https://bugs.php.net/bug.php?id=76563

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks mostly good, although frustrating that there is no way to have a "macro"/"function" in XML as this is a lot of duplicate wording :/

reference/outcontrol/functions/ob-clean.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-end-clean.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-end-clean.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-clean.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-end-flush.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-flush.xml Outdated Show resolved Hide resolved
reference/outcontrol/functions/ob-flush.xml Show resolved Hide resolved
reference/outcontrol/functions/ob-get-clean.xml Outdated Show resolved Hide resolved
Co-authored-by: Gina Peter Banyard <girgias@php.net>
@haszi
Copy link
Contributor Author

haszi commented Jan 7, 2024

I went with the imperative mood and contents.

@Girgias Girgias merged commit 86b976d into php:master Jan 7, 2024
2 checks passed
@haszi haszi deleted the Update-all-ob_-clean-and-flush-function-docs branch January 7, 2024 14:07
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.

None yet

3 participants