Bug #3009 [Ver]: long url's are chopped
| From: | helgi at trance dot is | 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