Joomla! Issue Tracker | Joomla! CMS #4789 - rel nofollow to mailto icon

Closed
19 Oct 2014
Medium
Build: staging
# 4789
Diff
peterlose:pbf-nofollow

Success

Success The Travis CI build passed Details

User tests: Successful: Unsuccessful:

peterlose
17 Oct 2014

This PR adds rel="nofollow" to the mailto link in com_content. This is done in order to stop search engines from following the link.

Votes

# of Users Experiencing Issue

0/1

Average Importance Score

2.00

8b9173b 17 Oct 2014

Added nofollow

losedk - open - 17 Oct 2014

jissues-bot - change - 17 Oct 2014

Labels

Added: ?

allanlhansen - test_item - 17 Oct 2014 - Tested successfully

1apweb - test_item - 17 Oct 2014 - Tested successfully

sandstorm871 - test_item - 17 Oct 2014 - Tested successfully

dgt41 - comment - 17 Oct 2014

@test OK Both of them now have rel=“nofollow”

jcata - test_item - 17 Oct 2014 - Tested successfully

brianteeman - change - 17 Oct 2014

Status

Pending

⇒

Ready to Commit

brianteeman - comment - 17 Oct 2014

Thanks for the code and thanks for testing - setting to RTC

peterlose - comment - 17 Oct 2014

Thanks guys!

zero-24 - comment - 17 Oct 2014

@losedk can you fix the Whitespace error found by travis?

FILE: .../travis/build/joomla/joomla-cms/components/com_content/helpers/icon.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 111 | ERROR | Whitespace found at end of line
--------------------------------------------------------------------------------

zero-24 - comment - 17 Oct 2014

Looks like this line: https://github.com/joomla/joomla-cms/pull/4789/files#diff-0064659e785cdb9b203e3b6e3cef298eR113

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4789.