CiviCRM Community Forums (archive)

So it all happened when the end user copied one link on the page to another place on the page.
Then for every test email and email after that the second link did not work.
It would show up like this
h ttp://nationalpriorities.org/cms/sites/all/modules/civicrm/extern/url.php?u=25543&qid=47158
Notice the space in the http.
We fixed it, removed it, renamed it, edited the html, and still 2 hours of testing came down to us just canceling and starting over.
Now I looked in the database and all was well.
Even the preview window looked okay when I hovered my mouse over it.
He was using Firefox on Windows XP
We just junked it and started over and all worked fine.
Should I put this in as a bug.

The line was this actually the same as two lines above that worked fine.
And worked in the second email without a hitch.

Not sure who's sources you grepped 
Here is the code from the errored email

<br>Dear Friend,<br><br>The National Priorities Project (NPP) has made a resolution for the New Year. NPP will expose the cost of defending US oil supply and build the case for investing in clean energy instead.<br><br>But we cannot do it alone, so we are asking you to join our efforts and give <a href="http://nationalpriorities.org/cms/sites/all/modules/civicrm/extern/url.php?u=25430&qid=47139">now</a>.&nbsp; We have already raised over $50,000 and are halfway to our goal.<br><br>NPP has led the way in bringing the cost of our current national security policies to the public. From the Iraq war to nuclear weapons, you can find out the cost to your community by clicking <a href="http://nationalpriorities.org/cms/sites/all/modules/civicrm/extern/url.php?u=25450&qid=47139">here</a>. <br><br>Now we are prepared to tackle the failed energy policies that cost us precious dollars and an even more precious planet.<br><br>You can help NPP make this change possible by giving <a href="h
 ttp://nationalpriorities.org/cms/sites/all/modules/civicrm/extern/url.php?u=25525&qid=47139" target="_blank">now</a>.<br><br>In this holiday season, what better gift for our children and grand children than a safer and sustainable world?<br><br>Thank you,<br>Greg Speeter, Executive Director<br><br>P.S. Whether it is $50 or $5,000, your gift is essential. Every dollar given is tax deductible and will be spent on uncovering the cost of oil. Please donate at <a href="http://nationalpriorities.org/cms/sites/all/modules/civicrm/extern/url.php?u=25525&qid=47139">www.nationalpriorities.org/support</a>.<br>

I will try but what I do not understand is this.
We tok that same email that did not work.
Copied pasted it into a new email.
Removed the space between the h
and it worked.
?

I see. Okay I will look at this in the morning

I see you’ve filed CRM-2477 – I commented there what should be (IMHO) our solution for this.

One Why it works when I upload the file instead.

Maybe the file got wrapped (properly, i.e., on a space) somewhere along the way?

Two What it worked (same line same everything with no extra line breaks ie even longer) when I add the link after it to see if it is a link number problem?

Because after the mailserver breaks the line hard in the middle of the previous link, this second link is near the beginning of a new line, so there’s no point in breaking it.

Personally, I believe this kind of line-wrapping should be done as late in the code chain as possible; I think it should be done by the proper PEAR class, but we don’t want to hack third party code unless strictly necessary, so we should do this in our wrapper send() function.

I don’t really want to dig too much into where exactly the difference lies at the moment (it’s a bit moot if send() is to be patched accordingly).