Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#6931] - Add email to contacts list

? Success

User tests: Successful: Unsuccessful:

avatar roland-d
roland-d
13 May 2015

Goal

The goal of this pull request is to show the email address of the linked user.

Test instructions

  1. Go to the contact list and make sure there is at least 1 contact with a linked Joomla user
  2. See that only the linked username is displayed
  3. Apply patch
  4. Refresh the contact list, now you will see the linked user email address below the username
  5. Test finished
avatar roland-d roland-d - open - 13 May 2015
avatar vinespie
vinespie - comment - 13 May 2015

@test Succes

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

avatar vinespie vinespie - test_item - 13 May 2015 - Tested successfully
avatar zero-24 zero-24 - change - 13 May 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 13 May 2015
Labels Added: ?
avatar pe7er
pe7er - comment - 14 May 2015

@test success, after the patch the e-mail address is displayed under the users name

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

avatar pe7er pe7er - test_item - 14 May 2015 - Tested successfully
avatar roland-d roland-d - change - 14 May 2015
The description was changed
Status New ⇒ Ready to Commit
avatar sovainfo
sovainfo - comment - 14 May 2015

Suggest to add the mail to line 168 instead of 162.

avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2015
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2015
Labels Added: ?
avatar roland-d
roland-d - comment - 15 May 2015

@sovainfo Good point. I have moved the field and updated the rest of the query to use the query builder.

@pe7er & @vinespie Could you please test again? Thanks.

avatar zero-24 zero-24 - change - 15 May 2015
Status Ready to Commit ⇒ Pending
avatar zero-24 zero-24 - change - 15 May 2015
Category ⇒ Front End
avatar zero-24 zero-24 - alter_testresult - 15 May 2015 - pe7er: Not tested
avatar zero-24 zero-24 - alter_testresult - 15 May 2015 - vinespie: Not tested
avatar zero-24 zero-24 - change - 15 May 2015
Labels Removed: ?
avatar zero-24 zero-24 - change - 15 May 2015
Labels Removed: ?
avatar pe7er
pe7er - comment - 15 May 2015

@test works as advertised :smile:

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

avatar pe7er pe7er - test_item - 15 May 2015 - Tested successfully
avatar zero-24
zero-24 - comment - 15 May 2015

@test works. Great work RTC'ing again :smile:

avatar zero-24 zero-24 - change - 15 May 2015
Status Pending ⇒ Ready to Commit
Easy No ⇒ Yes
avatar zero-24 zero-24 - change - 15 May 2015
Category Front End ⇒ Administration UI/UX
avatar zero-24 zero-24 - test_item - 15 May 2015 - Tested successfully
avatar zero-24 zero-24 - change - 15 May 2015
Labels Added: ?
avatar zero-24 zero-24 - close - 21 May 2015
avatar roland-d roland-d - change - 21 May 2015
Status Ready to Commit ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2015-05-21 09:48:36
Closed_By ⇒ roland-d
avatar roland-d roland-d - close - 21 May 2015
avatar Kubik-Rubik
Kubik-Rubik - comment - 21 May 2015

Merged. Thank you for your contribution @roland-d!

avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment