Doc #76500 [Com]: fputcsv() is ignoring the value FALSE
| From: | thomas at landauer dot at | Date: | Tue, 19 Jun 2018 15:52:32 +0000 |
| Subject: | Doc #76500 [Com]: fputcsv() is ignoring the value FALSE | ||
| References: | 1 | Groups: | php.doc.bugs |
| Request: | Send a blank email to doc-bugs+get-15803@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76500&edit=1
ID: 76500
Comment by: thomas at landauer dot at
Reported by: thomas at landauer dot at
Summary: fputcsv() is ignoring the value FALSE
Status: Closed
Type: Documentation Problem
Package: Filesystem function related
Operating System: Linux Mint
PHP Version: 7.1.18
Assigned To: cmb
Block user comment: N
Private report: N
New Comment:
> And you won't be able to distinguish either from an empty string.
Well, an empty string is not possible in a boolean field ;-)
> In my opinion, adding yet another optional argument to fputcsv()
does more harm than good.
Why? Which harm?
Previous Comments:
------------------------------------------------------------------------
[2018-06-19 15:44:14] cmb@php.net
> So if you have a nullable boolean field in your database, this
> function becomes completely useless, since you can't tell W“á‚we
> \#‚ìW6S)null
> from
false.
And you won't be able to distinguish either from an empty string.
CSV knows nothing about NULLs and booleans â it's all about
strings. You have to cater to that yourself in any way that suits
your use-case.
> But I'd suggest to add an optional argument to configure how
> false is treated - [â¦]
In my opinion, adding yet another optional argument to fputcsv()
does more harm than good. However, feel free to start the RFC
process[1]. :)
[1] <https://wiki.php.net/rfc/howto>
------------------------------------------------------------------------
[2018-06-19 15:19:45] thomas at landauer dot at
Yeah, sure (string)false === ''. But (string)null ===
'' too! So if you have a nullable boolean field in your database, this function
becomes completely useless, since you can't tell null from false.
Fixing the documentation is certainly better than nothing (haven't seen it live, though). But
I'd suggest to add an optional argument to configure how false is treated -
something like the $options constants in json_encode().
------------------------------------------------------------------------
[2018-06-19 15:03:01] cmb@php.net
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.
Thank you for the report, and for helping us make our documentation better.
------------------------------------------------------------------------
[2018-06-19 15:02:37] cmb@php.net
Automatic comment from SVN on behalf of cmb
Revision: http://svn.php.net/viewvc/?view=revision&revision=345171
Log: Fix #76500: fputcsv() is ignoring the value FALSE
------------------------------------------------------------------------
[2018-06-19 14:57:12] cmb@php.net
Indeed, this is not a bug, since (string)false === ''.
The docs could be improved in this regard, though.
------------------------------------------------------------------------
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=76500
--
Edit this bug report at https://bugs.php.net/bug.php?id=76500&edit=1