Everything
php
Learn Community Releases The PHP Project
Home » Project » Newsgroups » php.internals » [patch] PDO_FETCH_KEYS »

[patch] PDO_FETCH_KEYS

php.internals

Hans-Peter Oeri

18 years ago
Hi! I discovered that PDO::FETCH_KEY_PAIR does not work as documented. Documented is: 'Fetch into an array where the 1st column is a key and all subsequent columns are values' Actually, all but two column result sets throw an error... Then I thought about generalizing the fetch mode. Have 1..n 'key' columns. If only one value is left, assign it as a scalar. If multiple columns are left over, have it taken as FETCH_ASSOC. Please find attached a humble suggestion to generalize FETCH_KEY_PAIR to FETCH_KEYS, including an attribute to define the number of 'key' columns (defaults to one). The result should be fully backwards compatible. Please give me your suggestions. HPO PS: My diff includes my 'old' FETCH_2D patch.

Wez Furlong

18 years ago
What purpose does this serve exactly? Why do we need to allocate an empty string for every database connection? Why do we need this in the core? There's a lot of manipulation being done here that is better suited to happen in PHP script land where it's safer and easier to maintain. --Wez. On Nov 24, 2007, at 4:31 PM, Hans-Peter Oeri wrote:

Hans-Peter Oeri

18 years ago
Hi! Wez Furlong wrote:
> What purpose does this serve exactly?
PDO::FETCH_KEYS is supposed to be a "generalized" form of the existing PDO::FETCH_KEY_PAIR. The latter is limited to two column results and organises it into key=>value. PDO::FETCH_2D is an IMHO better strategy than the existing ATTR_FETCH_TABLE_NAMES. The latter includes two "information tokens" into one datum (the array index).
> Why do we need to allocate an empty string for every database connection?
The empty string is the default value for the attribute. NULL actually has a different meaning. Of course, the actual implementation of a default value is open for change, of course.
> Why do we need this in the core? > There's a lot of manipulation being done here that is better suited to > happen in PHP script land where it's safer and easier to maintain.
PDO::FETCH_2D is *NOT* easily implemented in PHP as it needs information unavailable before. PDO::FETCH_KEYS is just a generalization of the existing PDO::FETCH_KEY_PAIR. What do you mean by "manipulation"? I only intended to implement fetch modes I need. HPO PS: Please notice the updated patch.