mssql patch & TODO
| From: | Tomas V.V.Cox | Date: | Sat, 14 Apr 2001 14:25:54 +0000 |
| Subject: | mssql patch & TODO | ||
| Groups: | php.pear.dev | ||
| Request: | Send a blank email to pear-dev+get-223@lists.php.net to get a copy of this message | ||
Hi,
1) I nobody have a problem, I'll commit the attached patch to the
mssql.php extension. Chages for it are:
- Remove extra checks in connect
- move fetchrow to use fetchInto
- fix fetchInto to return null on no more rows instead of an error
- Add fetch arbitrary row numbers feature to fetchInto
- spaces
2) in pear/TODO there is:
- "make DB_Error store queries" This is implemented yet, probably needs
to be removed.
- "make regression tests for DB" Because there is some regression test
done, probably is better to say who of them are missing.
3) Can I submit a pear/DB/TODO file, saying what things needs to be
implemented on each extension (things not covered by STATUS)? For
example, in mssql.php I can see:
- implement a better error handling: errorNative, mssqlRaiseError
- in connect, catch initial connection errors with $php_errormsg
(see mysql.php for an example on how to do it)
- add phpdoc comments
- add regression tests
So people with enought spare time can do some needed work.
4) As I changed the default behavoir of ifx::fetch* to ORDERED, can I
update ifx fetchmodes STATUS to "x" instead of "x(t)" ?
Regards,
Tomas V.V.Cox