Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2005-04-13 Thread Jani Taskinen
Don't forget the NEWS entries for PHP_5_0 and PHP_4_3 branches.. --Jani On Tue, 12 Apr 2005, Frank M. Kromann wrote: fmk Tue Apr 12 13:45:33 2005 EDT Modified files: /php-src/ext/mssql php_mssql.c Log: Fix for #32682. Error on module shutdown when called from activescript

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2004-12-02 Thread Frank M. Kromann
Hi Ilia, That fix does not do anuthing. The data length is checked above this code and a NULL value is returned on an empty string. - Frank > iliaa Thu Dec 2 21:26:04 2004 EDT > > Modified files: > /php-src/ext/mssqlphp_mssql.c > Log: > Fixed bug #30962

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2004-12-02 Thread Andi Gutmans
Do we want to return empty string or NULL? Andi At 02:26 AM 12/3/2004 +, Ilia Alshanetsky wrote: iliaa Thu Dec 2 21:26:04 2004 EDT Modified files: /php-src/ext/mssql php_mssql.c Log: Fixed bug #30962 (mssql returns space for NULL columns). http://cvs.php.net/diff.php/php-s

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2003-10-16 Thread Derick Rethans
On Thu, 16 Oct 2003, Jani Taskinen wrote: > On Wed, 15 Oct 2003, Ilia Alshanetsky wrote: > > >If in fact code will be broken then yes, INI directive may be the only route. > >However, I do not see why trailing spaces were being stripped in the first > >place, as the original report indicates it

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2003-10-15 Thread Frank M. Kromann
Trailing spaces are returned by the database for a char() column, if the inserted data is shorter than the fixed length. This is not the case with a varchar or a text column. Lets fix the Sybase extension as well and see how manu bugs we are getting from this. - Frank > On Wed, 15 Oct 2003, Ili

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2003-10-15 Thread Jani Taskinen
On Wed, 15 Oct 2003, Ilia Alshanetsky wrote: >If in fact code will be broken then yes, INI directive may be the only route. >However, I do not see why trailing spaces were being stripped in the first >place, as the original report indicates it does technically 'corrupt' the >data? I'm -100

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2003-10-15 Thread Ilia Alshanetsky
If in fact code will be broken then yes, INI directive may be the only route. However, I do not see why trailing spaces were being stripped in the first place, as the original report indicates it does technically 'corrupt' the data? Ilia On October 15, 2003 05:09 pm, Frank M. Kromann wrote: >

Re: [PHP-CVS] cvs: php-src /ext/mssql php_mssql.c

2003-10-15 Thread Frank M. Kromann
Hi Ilia, I think this will break code else where. And now the mssql, sybase and sybase_ct code handles trailing spaces differently. Would it make sence to create a flagin PHP.INI for this ? - Frank > iliaa Tue Oct 14 23:31:30 2003 EDT > > Modified files: > /php-src/