6.0.0-beta4
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
3/10/26
History
Attachments
Comment
Watch
Download
Comment on [#10602] Broken Postgresql value encoding
*
Your Email Address
*
Spam protection
Enter the letters below:
.__ .___..__ ._.. . [__) | | \ | | | | | |__/_|_|/\|
Comment
> This fixes on 9.1, and *should* fix on 8.1 also. > > However, it doesn't use quotedStringPrefix() anymore. That is > because this was failing to add the 'E' prefix, since > Horde_Db_Adapter_Postgresql_Schema is extending > Horde_Db_Adapter_Base_Schema which will always return the empty > string. The dynamic determination of whether to add 'E' is instead > in Horde_Db_Adapter_Pdo_Pgsql. > > So #1 - quoteBinary() doesn't have any access to this adapter object. > And #2, why do we even need this anyway? Once we escape the string > by converting characters to octets, we necessarily need to report > this to the postgres as an escaped string, which means we must ALWAYS > use the 'E' prefix. > > So I'm going to go ahead and remove the quotedStringPrefix() code. > It sort of looks like an artifact given the new Horde_Db_Value_Binary > code, and it isn't used anymore after this fix.
Attachment
Watch this ticket
New Ticket
My Tickets
Search
Query Builder
Reports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers