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

Bug #3572 Use of addslashes() adds too much backslashes
Submitted: 2005-02-24 09:59 UTC
From: bluem at gmx dot de Assigned: aidan
Status: Closed Package: PHP_Compat
PHP Version: Irrelevant OS: Any (reprod.: Linux, Mac OS X)
Roadmaps: 1.6.0a1    
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:
Solve the problem : 41 - 15 = ?

 [2005-02-24 09:59 UTC] bluem at gmx dot de
Description: ------------ This bug affects the implementation of var_export() in the latest PHP_Compat version (4.3.1). Problem: Strings are quoted using $stringdelim, but nevertheless escaping is done with addslashes(). This results in double quotes being escaped with a backslash, although no backslash would be necessary. Solution: It should suffice to exchange the two occurrences of addslashes() with the following code: str_replace($stringdelim, '\\'.$stringdelim, $value) Reproduce code: --------------- echo "<pre>"; print_r(var_export('Hello "Foo" World')); echo "</pre>\n"; Expected result: ---------------- 'Hello "Foo" World' Actual result: -------------- 'Hello \"Foo\" World'


 [2005-02-28 08:47 UTC] aidan
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 In case this was a 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 found a number of other problems, they're all fixed now. Diff: