Tickets :: Comment on [#10805] Add stream support to Horde

* Your Email Address
* Spam protection	Enter the letters below: . .__.. ..__ .__ \| \| \|\|_/ [__)[__) \|___\|__\|\| \[__)\| \
Comment	> Changes have been made in Git (master): > > commit ea32d50494dcadacd5173f8059f6f49f8b3dfbf1 > Author: Jan Schneider <jan@horde.org> > Date: Fri Feb 3 17:36:53 2017 +0100 > > Make the $value a string-only property again. > > It's a BC break that this might sometimes be a stream now. > Instead make $value a setter/getter and add $stream property too. > Convert between these two on the fly. > Also make Horde_Db_Value_Text extend Horde_Db_Value_Binary > because they share most of the code. This requires to change some > class tests though, because "instanceof Horde_Db_Value_Binary" is > true now for Value_Text objects too. > > Request: 10805 > > framework/Db/lib/Horde/Db/Adapter/Base.php \| 2 +- > framework/Db/lib/Horde/Db/Adapter/Oci8.php \| 8 +-- > framework/Db/lib/Horde/Db/Adapter/Pdo/Base.php \| 8 +-- > framework/Db/lib/Horde/Db/Value/Binary.php \| 89 > ++++++++++++++++++++------ > framework/Db/lib/Horde/Db/Value/Text.php \| 25 +------- > 5 files changed, 80 insertions(+), 52 deletions(-) > > http://github.com/horde/horde/commit/ea32d50494dcadacd5173f8059f6f49f8b3dfbf1
Attachment
Watch this ticket