Doc #70118 [Csd->Nab]: PDOStatement::execute manual says "values are treated as PDO::PARAM_STR"

Edit report at https://bugs.php.net/bug.php?id=70118&edit=1 ID: 70118 Updated by: tpunt@php.net Reported by: chealer at gmail dot com Summary: PDOStatement::execute manual says "values are treated as PDO::PARAM_STR" -Status: Closed +Status: Not a bug Type: Documentation Problem Package: Documentation problem PHP Version: Irrelevant -Assigned To: tpunt +Assigned To: Block user comment: N Private report: N Previous Comments: ------------------------------------------------------------------------ [2015-12-24 20:27:28] chealer at gmail dot com tpunt: technically, PDO::PARAM_STR *is* an integer. There are strings in systems other than PHP, so the suggestion is valid, but feel free to be more specific. In any case, please explain why this was closed. ------------------------------------------------------------------------ [2015-12-19 18:00:07] tpunt@php.net PDO::PARAM_STR does not *represent* an integer. It is *represented by* an integer, but it itself represents "the SQL CHAR, VARCHAR, or other string data type"[1]. Given the aforementioned, and that the default binding type is PDO::PARAM_STR, I still don't see the sentence as being incorrect. If anything, I'd say it is more technically accurate than mentioning "string", since we're not actually talking about strings in PHP, but rather strings at the database level (like PDO::PARAM_NULL representing the SQL NULL data type, rather than PHP's null). [1]: http://uk1.php.net/manual/en/pdo.constants.php ------------------------------------------------------------------------ [2015-12-19 16:43:00] chealer at gmail dot com tpunt, why was this closed? Note that this is not about ambiguity, nor just about clarity. The sentence is simply wrong. PDO::PARAM_STR represents an integer which I do not know, so the current form would translate to something like "All values are treated as 1234.", which is at best the opposite of what the sentence should claim. ------------------------------------------------------------------------ [2015-12-18 21:05:51] tpunt@php.net Given that it's in the context of parameter binding, I think the current description is quite unambiguous. Whilst it could be changed to read "as strings (PDO::PARAM_STR)", I personally don't feel it adds any extra clarity to the sentence. ------------------------------------------------------------------------ [2015-07-23 14:53:06] chealer at gmail dot com Description: ------------ input_parameters An array of values with as many elements as there are bound parameters in the SQL statement being executed. All values are treated as PDO::PARAM_STR. --- From manual page: http://www.php.net/pdostatement.execute --- PDO::PARAM_STR indicates string parameters. "as PDO::PARAM_STR" could read "as strings (PDO::PARAM_STR)", or simply "as strings". ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=70118&edit=1