Hi Adam, On 28/11/2016 16:29, Adam Baratz wrote: >> Based on my second thoughts on my initial approach, I created an RFC to >> supplant it: >> https://wiki.php.net/rfc/debugging_pdo_prepared_statement_emulation_v2 >> > > I'd appreciate any feedback on this RFC, especially from any of the people > who voted against the original[1].
v2 seems more reasonable to me. I would not use "Parsed SQL" though, as it's not just the result of parsing. In fact it could effectively be what has been sent to the backend, regardless of statement emulation... however, nothing better than "Sent SQL" comes to mind atm. I.e. in case of pdo_pgsql: SQL: "SELECT * FROM tbl WHERE x = ?" Sent SQL: "SELECT * FROM tbl WHERE x = $1" (could be available before execute) or w/ emulated prepares: Sent SQL: "SELECT * FROM tbl WHERE x = 'foo'" (only available after execute) For consistency, I would prefer to only add the information after execute() has been called. Cheers -- Matteo Beccati Development & Consulting - http://www.beccati.com/ -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
