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

Request #13470 make_link should make it easy to be xhtml friendly
Submitted: 2008-03-23 07:06 UTC
From: doconnor Assigned: dufuz
Status: Closed Package: pearweb (version CVS)
PHP Version: 5.2.5 OS:
Roadmaps: 1.18.0    
Subscription  


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 : 48 + 28 = ?

 
 [2008-03-23 07:06 UTC] doconnor (Daniel O'Connor)
Description: ------------ make_link('?a&b'); //should out ?a&b make_link('?a&b'); //should out ?a&b This helps prevent easy mistakes with URLs and makes us xhtml friendly.

Comments

 [2008-03-23 07:13 UTC] doconnor (Daniel O'Connor)
ah %#$#^... htmlspecialchars_decode is 5.1.0+
 [2008-03-30 20:53 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Please do not add a doctype via patches and given pearweb has a php 5.2.3 dep then you can use htmlspecialchars_decode if you so desire tho it sounds like you want to be using the forth param of htmlspecialchars instead of decoding then encoding again.
 [2008-03-30 21:00 UTC] dufuz (Helgi Þormar Þorbjörnsson)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. Committed with htmlspecialchars and forth param to false, check out if it works for us
 [2008-03-30 21:01 UTC] doconnor (Daniel O'Connor)
Oops! doctype was meant to be local only. I take it we're switching away from the current pear.php.net xhtml strict
 [2008-03-30 21:03 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Yeah, I mentioned to you that we'd go for HTML5 hence the empty doctype; It looks more promising than XHTML2 anyway