ID: 14698
Updated by: yohgaki
Reported By: [EMAIL PROTECTED]
Status: Analyzed
Bug Type: Documentation problem
Operating System: Linux
PHP Version: 4.1.0
Assigned To: yohgaki
New Comment:
Thanks for checking the change :)
I'm the new module maintainer, so I'm asking previous module maintainers about it.
Since MySQL seems it does the same, I'll keep this behavior probably, but it is not a
promise yet. ;)
Previous Comments:
------------------------------------------------------------------------
[2001-12-26 02:19:28] [EMAIL PROTECTED]
Implemented 6 months ago by jon (1.112) pgsql.c.
Looks reasonable.
------------------------------------------------------------------------
[2001-12-26 02:13:51] [EMAIL PROTECTED]
Works wonderfully without the row. That's certainly a win convienence wise, especially
for folks coming over from mysql where the loop syntax is trivial
(while $row = fetch())
{}
I'd prefer to see the docs change then the functionality. I'll take a look at the code
itself but it is hard to see how this is a performance loss, most users end up keeping
a counter anyways.
Why do you prefer making the row required? It seems like a silly added restriction,
especially when I bet 80% of the cases where the function is called don't need it.
------------------------------------------------------------------------
[2001-12-26 02:09:44] [EMAIL PROTECTED]
Additional comment:
All pg_fetch_* works as reporter mentioned, but
DO NOT take advantage of this.
I may change behavior after I check php3 and php4 CVS log :)
------------------------------------------------------------------------
[2001-12-26 02:06:55] [EMAIL PROTECTED]
I've checked current code. Reporter is right.
I need to check when this has been changed and I think row should be required
parameter. I'll keep this behavior if and only if this was the implemented from the
beginning :)
(Current code internally advance row in result structure)
So DO NOT take advantage of this.
------------------------------------------------------------------------
[2001-12-26 01:57:46] [EMAIL PROTECTED]
array pg_fetch_array (resource result, int row [, int result_type])
makes int row appear required, when it is actually optional.
------------------------------------------------------------------------
Edit this bug report at http://bugs.php.net/?id=14698&edit=1
