[Tickets #2749] NEW: bugfix (and whitespace fixes) for lib/Horde/SessionHandler/pgsql.php

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2749
-----------------------------------------------------------------------
 Ticket             | 2749
 Created By         | cbs at cts.ucla.edu
 Summary            | bugfix (and whitespace fixes) for lib/Horde/SessionHandler/pgsql.php
 Queue              | Horde Framework Packages
 Version            | FRAMEWORK_3
 State              | Unconfirmed
 Priority           | 2. Medium
 Type               | Bug
 Owners             | 
+New Attachment     | pgsql.php.diff
-----------------------------------------------------------------------


cbs at cts.ucla.edu (2005-10-09 03:36) wrote:

The write() handler in pgsql.php is checking the result of the COMMIT, not
the result of the UPDATE.  The current check tests pg_rows_affected(), which
is 0, and always results in true.  The test should (I think) be checking
result of the UPDATE to ensure that 1 row was updated.  See the thread on
horde at lists with the subject "horde 3.0.5, postgresql session handler fails"
from Oct. 8 for more details.