Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.11.5

Request #4182 mysql blob support
Submitted: 2005-04-19 16:59 UTC
From: therion_5 at hotmail dot com Assigned: alan_k
Status: Closed Package: DB_DataObject
PHP Version: 4.3.10 OS: any
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 43 + 9 = ?

 
 [2005-04-19 16:59 UTC] therion_5 at hotmail dot com
Description: ------------ blob data can be escaped by one of following calls: - mysql_escape_string - mysql_real_escape_string

Comments

 [2005-04-20 03:29 UTC] alan_k
feel free to send a patch (it will get added alot quicker then) Regards Alan
 [2005-04-20 07:13 UTC] therion_5 at hotmail dot com
Here is patch. Sorry can't give you diff, coz i'm still on old version (waiting for multiple db patch) Cast.php // ... function toStringFromString(...) { // ... switch($db) { // ... case 'mysql': return "'".mysql_escape_string($this->value)."'"; // ... } // ... } function toStringFromBlob(...){ // ... switch($db) { // ... case 'mysql': return "'".mysql_escape_string($this->value)."'"; // ... } // ... } // ...
 [2005-04-20 07:35 UTC] alan_k
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.
 [2005-04-22 08:17 UTC] alan_k
This really needs fixing properly.. - the either the db or parent dataobject needs passing to the toString() method.
 [2005-06-04 03:00 UTC] alan_k
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. I think I nailed this one a while back.