Doc #74399 [Ver->Csd]: ob_*clean* methods do not respect missing cleanable flag; same for flushing

[email protected] Sun, 07 Jan 2024 14:04:45 +0000
Newsgroups php.doc.bugs
Message-ID <[email protected]>
Edit report at https://bugs.php.net/bug.php?id=74399&edit=1

 ID:                 74399
 Updated by:         [email protected]
 Reported by:        krody dot robi at gmail dot com
 Summary:            ob_*clean* methods do not respect missing cleanable
                     flag; same for flushing
-Status:             Verified
+Status:             Closed
 Type:               Documentation Problem
 Package:            Output Control
 Operating System:   Irrelevant
 PHP Version:        7.1.3
-Assigned To:        
+Assigned To:        girgias
 Block user comment: N
 Private report:     N



Previous Comments:
------------------------------------------------------------------------
[2024-01-07 14:03:53] [email protected]

Closed by https://github.com/php/doc-en/pull/3073

------------------------------------------------------------------------
[2017-04-28 07:50:41] [email protected]

Then this is a doc bug because all four functions claim they need the respective CLEANABLE/FLUSHABLE flag to work.

------------------------------------------------------------------------
[2017-04-28 07:48:47] [email protected]

See also bug #69404

------------------------------------------------------------------------
[2017-04-28 07:43:24] [email protected]

Unfortunately there's no one-to-one correlation between the legacy userland function names and the flags of the new output API.

- ob_end_flush() is actually a finalize & send and not a "flush" in a strict sense
- ob_get_flush() is a finalize & send action, too
- ob_end_clean() is actually a discard & remove and not a "clean" in a strict sense
- ob_get_clean() is a finalize & remove action

I know, the legacy function names are bad and confusing, but were kept for BC, as was their functionality.

------------------------------------------------------------------------
[2017-04-10 08:05:57] [email protected]

Goes back to when the flags were introduced in 5.4: https://3v4l.org/G1Eqn

Indeed, CLEANABLE only affects ob_clean() and FLUSHABLE only affects ob_flush(). Looks like it would be fairly straightforward to add in.

The alternative would be adjusting the docs, but having those functions respect the flags makes sense to me. And in the case of ob_end_clean and ob_end_flush I would expect both the cleanable/flushable and the removable flags to be required.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=74399


--
Edit this bug report at https://bugs.php.net/bug.php?id=74399&edit=1