Feeling Lucky
?
[#3915] - *Mailcloak broken on images
- Closed
- 17 Jul 2014
- Medium
- Build: staging
- # 3915
- Diff
- infograf768:mailcloak_image
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Inset an image in an article content.
Then add a mail adress to the image.
You will get this
What happens is that the < and > are converted to html entities.
This patch should solve it
Kubik-Rubik
- comment
- 17 Jul 2014
PS: We have the same issue in the HTML library (libraries/html/email.php).
The convertEncoding function was moved to a wrong position, so that image links are also encoded which leads to the same broken output as described in this PR.
// Convert text - here is the right place
$text = static::convertEncoding($text);
This code section should be placed back in the if($email) statement. PR will follow!
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-07-17 14:45:55 |
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33957&start=0