× Joomla Facebook Connect support forum

Topic-icon Notification mail can't handle my URL:s

Active Subscriptions:

None
14 years 11 months ago #11636 by Bosse
I use Azrul's MyBlog. Some links to the blogs has the name in it, with spaces. When I get a notification mail, it looks like this:
File Attachment:


As you can see, the URL end after, in this case, Mikael. Which of course result in that the URL doesn't take me to page where the comment is.
How do I get JFBConnect to handle these URL:s correctly?
The topic has been locked.
Support Specialist
14 years 11 months ago #11638 by alzander
Looking at the email we send, it's actually just a text email. Whatever email client you're using is seeing the URL and making it a link, and since URLs aren't supposed to have spaces in them, it's simply making everything before the space into the link.

In a future update, we'll probably make these emails into HTML based emails, which will let us mark the whole URL as a link, but until then, there's little we can do immediately.

Sorry for the trouble, and hope that explains the issue. In general, I wouldn't recommend URLs with spaces, but understand that may not be an easy thing to change on your site.

Alex
The topic has been locked.
Active Subscriptions:

None
14 years 11 months ago #11640 by Bosse
Oki,

I don't know if this matter, but just to clarify:
In the URL there's actually no spaces. It looks (ends) like this:
/blogger/Pontus%20Andersson/
which in the notification ends like this:
/blogger/Pontus

So even if it's a text mail, shouldn't the "%20" show?
The topic has been locked.
Support Specialist
14 years 10 months ago #11663 by alzander
Bosse,
Gotcha. You can try the following change, which I believe will fix the issue you're seeing. In the /components/com_jfbconnect/controllers/social.php file, around line 30, you'll see a line like: $href = JRequest::getVar('href'); Directly below that line, add::
$href = urlencode($href);

That will hopefully fix it for Comments. To fix for Like's, look for the same line of code at around line 55 and do the same thing.

Please let us know if that helps, and if so, we'll definitely add it into the next release.

Thanks,
Alex
The topic has been locked.
Active Subscriptions:

None
14 years 10 months ago #11666 by Bosse
Great! I'll come back to you a soon as I've seen if it works.

And speaking of social.php, I've got another problem regarding the language. I've started a new thread about it here:
www.sourcecoast.com/forums/jfbconnect/jf...swedish-in-socialphp
The topic has been locked.
Active Subscriptions:

None
14 years 10 months ago #11672 by Bosse
Nops, didn't work.

Enclosed you've got a pic showing the link, and where I edited in the social.php. Just to make sure that I did it right.
File Attachment:
The topic has been locked.