Bug #3009 [Ver]: long url's are chopped

From: Date: Mon, 24 Jan 2005 08:25:34 +0000
Subject: Bug #3009 [Ver]: long url's are chopped
References: 1  Groups: php.pear.webmaster 
Request: Send a blank email to pear-webmaster+get-3790@lists.php.net to get a copy of this message
Edit report at http://pear.php.net/bugs/bug.php?id=3009&edit=1 ID: 3009 Updated by: helgi@trance.is Reported By: alan_k at php dot net Status: Verified Type: Bug Package: Bug System Operating System: na PHP Version: Irrelevant Assigned To: dufuz New Comment: btw. using wordwrap($comment, 72, "\n") fixes the problem, instead of having wordwrap($comment, 72, "\n", 1) Think it's just best to use it, we can just move back to the old way, hence the nesting or use what I commit and add this to it, either way works just fine, was just trying to avoid removing the cut mode in my first commit. Previous Comments: ------------------------------------------------------------------------ [2005-01-24 03:18:35] helgi at trance dot is Daniel, first look at pear-cvs, then you see, it's not in functions.inc, it's in bug.php, I just moved addlinks and make_tickets_links outside the nested madness, I actually tested this, on the link alan gave me and some few more, I and Pierre are dicussion this on irc and he's talking about this might be a PHP bug, as you can see in bug report 3149 it wraps on the closing </a> and makes it into </a > So this was just a matter of having the link at right length to be able to reproduce this, which none of my test links did so I didn't find this out until now. ------------------------------------------------------------------------ [2005-01-24 03:02:25] danielc at analysisandsolutions dot com Helgi: What did you modify? I'm looking at bugs/include/functions.inc but don't see anything changed along these lines. http://cvs.php.net/pearweb/public_html/bugs/include/functions.inc The place to fix this is in that file in the addLinks() function. I'm tweaking that regex right now. Let's see what happens. Let alone, for the umpteenth time, test things out before committing them. ------------------------------------------------------------------------ [2005-01-24 02:46:27] helgi at trance dot is No, seems I was wrong/right, pick which ever ;) http://pear.php.net/bugs/bug.php?id=3149 http://pear.php.net/bugs/bug.php?id=3278 Those reports show how things went wrong! :( ------------------------------------------------------------------------ [2005-01-23 19:06:00] helgi at trance dot is No just me trippin', we tho have to fix that http:// stuff, if nothing comes after that it won't be changed into a link, shouldn't be too hard I guess, tho I'm no regex person =) ------------------------------------------------------------------------ [2005-01-23 18:54:09] helgi at trance dot is Or not, will have to test it better. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://pear.php.net/bugs/bug.php?id=3009 -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=3009&edit=1

« previous php.pear.webmaster (#3790) next »