? Success

User tests: Successful: Unsuccessful:

avatar aardgoose
aardgoose
24 Mar 2016

Issue

Email cloaking in contact category component view does not work,

Steps to reproduce the issue

Set up a contact category menu item displaying a contact category containing contacts with email addresses. Make sure show email address is enabled for this menu item and the email cloaking plugin is enabled.

Expected result

The email address should be cloaked and if required prefixed with mailto:

Actual result

A plain uncloaked email address is shown.
Incorrect parameter name checked to determine if a contact's email address should be pre-processed before display.

Summary of Changes

"show_email_headings" checked instead of incorrect "show_email" parameter.

Email cloaking now works correctly

avatar aardgoose aardgoose - open - 24 Mar 2016
avatar aardgoose aardgoose - change - 24 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Mar 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 24 Mar 2016
Category Plugins
avatar Ruchiranga
Ruchiranga - comment - 24 Mar 2016

Hi,
I'm trying to test this fix but I cannot reproduce the issue. I disabled javascript and opened the contact and the email address was not shown(i.e it was cloaked).

P.S : is this issue there only in the new 3.5 version? I did the testing on the previous version.

avatar aardgoose
aardgoose - comment - 24 Mar 2016

Found in 3.4 but the fix equally applies to 3.5 (the code error is still there). If you look in default layout file category/tmpl/default_items.php it checks "show_email_headings", but the original view.html.php file checks "show_email" which doesn't work.

Are you looking at the contact email as displayed on the category listing page or a contact details. The problem is on the category listing page. This is with a standard installation.

avatar Ruchiranga
Ruchiranga - comment - 25 Mar 2016

I'm not clear what you are referring to by category listing page.
Here is a screen shot of what I'm looking at.
image


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

avatar aardgoose
aardgoose - comment - 25 Mar 2016

Hi, it looks like you are looking at the single contact display.

The Category list is referred to as "list contacts in a category" when you set up a new menu item.

The attached shows the correct behavior (the email addresses are highlighted because the mailto link has been added, and inspecting the page source shows that the cloaking java script is used for the display. Without the fix it is just a bare email address, no cloaking, no mailto link,

Thanks for looking at this.

capture

avatar infograf768 infograf768 - test_item - 28 Mar 2016 - Tested successfully
avatar infograf768
infograf768 - comment - 28 Mar 2016

I have tested this item :white_check_mark: successfully on 3bf4908

This is indeed a bug as show_email is used when displaying a unique contact and should not be the criteria to cloak emails in a contact category list.

Good find!


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

avatar mikeveeckmans mikeveeckmans - test_item - 30 Mar 2016 - Tested successfully
avatar mikeveeckmans
mikeveeckmans - comment - 30 Mar 2016

I have tested this item :white_check_mark: successfully on 3bf4908


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

avatar brianteeman brianteeman - change - 30 Mar 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 30 Mar 2016

RTC - thanks


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

avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 30 Mar 2016
Milestone Added:
avatar rdeutz rdeutz - change - 12 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-12 19:48:14
Closed_By rdeutz
avatar rdeutz rdeutz - close - 12 Apr 2016
avatar rdeutz rdeutz - merge - 12 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 12 Apr 2016
avatar rdeutz rdeutz - reference | 19a888d - 12 Apr 16
avatar rdeutz rdeutz - merge - 12 Apr 2016
avatar rdeutz rdeutz - close - 12 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - change - 12 Apr 2016
Labels Removed: ?
avatar aardgoose aardgoose - head_ref_deleted - 17 Apr 2016
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment