? ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
7 Mar 2017

Pull Request for Issue #14392.

Summary of Changes

Don't default to all user groups to show on contact fields. Like it is now it is not possible to prevent to show the user custom fields on the contact details. This pr removes the hardcoded default value "All".

Testing Instructions

Clear the "Show User Custom Fields" option in the contact options.
image

Expected result

Keep empty.

Actual result

Defaults the value to "All".

Documentation Changes Required

avatar laoneo laoneo - open - 7 Mar 2017
avatar laoneo laoneo - change - 7 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Mar 2017
Category Administration com_contact
avatar laoneo laoneo - change - 7 Mar 2017
Title
Don't default to all user fields to show
[com_fields] Don't default to all user fields to show
avatar laoneo laoneo - edited - 7 Mar 2017
avatar gwsdesk gwsdesk - test_item - 8 Mar 2017 - Tested unsuccessfully
avatar gwsdesk
gwsdesk - comment - 8 Mar 2017

I have tested this item ? unsuccessfully on d140e0f

I have tested this unsuccessfully See https://www.screencast.com/t/dDPWttlRH


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

avatar gwsdesk
gwsdesk - comment - 8 Mar 2017

See the reason and what is happening https://www.screencast.com/t/dDPWttlRH


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

avatar laoneo
laoneo - comment - 8 Mar 2017

I don't have flash ?

avatar gwsdesk
gwsdesk - comment - 8 Mar 2017

Not sure @laoneo
If I empty the field and save I get the broken cut text. If I load the options initially the "All"is given in the field entry. Changing it to empty and successively saving and reloading gives that behavior. So not sure actually what should be shown after empty and save.....should it show "all" or empty (with correct text)

avatar laoneo
laoneo - comment - 8 Mar 2017

It should be an empty select box as in your screencast. The broken text is a "bug" in the template as discussed in #13594. So I guess we should handle the cut text problem in the issue. Is the rest working as expected?

avatar gwsdesk
gwsdesk - comment - 8 Mar 2017

Correct works as expected for the prevention so will change the test result Thanks Allon

avatar gwsdesk gwsdesk - test_item - 8 Mar 2017 - Tested successfully
avatar gwsdesk
gwsdesk - comment - 8 Mar 2017

I have tested this item successfully on d140e0f


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

avatar coolcat-creations coolcat-creations - test_item - 8 Mar 2017 - Tested unsuccessfully
avatar coolcat-creations
coolcat-creations - comment - 8 Mar 2017

I have tested this item ? unsuccessfully on d140e0f

The userfields are still show, although nothing is selected


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

avatar laoneo
laoneo - comment - 8 Mar 2017

Is it empty in the menu configuration and the global contact options?

avatar coolcat-creations
coolcat-creations - comment - 8 Mar 2017

Yes both are empty, also created a completely new one as the existing failed, but also no luck

avatar brianteeman brianteeman - test_item - 25 Mar 2017 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 25 Mar 2017

I have tested this item ? unsuccessfully on d140e0f

I created a user field group called testgroup
I create a user text field called textfield
I created a contsct and linked it to a user
I created a menu item to display a single contact
I checked that the option field did not have a category set
The contact is displayed on the front end with both the group and its fields

Just a guess but instead of having it blank would it not be better to have a NONE setting


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14407.
avatar laoneo laoneo - change - 27 Mar 2017
Labels Added: ?
avatar laoneo
laoneo - comment - 27 Mar 2017

The important point is that the setting in the global contact options is not defaulted to all. So make sure that you can save it with no value selected and it stay like that:
image

As soon as I select a group or All, then the fields do show up, but when none is selected, then no fields should be shown.

avatar brianteeman
brianteeman - comment - 27 Mar 2017

As stated above in my report it is not enough for the options page to show no value. it actually has to work in that way which it doesnt

avatar laoneo
laoneo - comment - 27 Mar 2017

I'v just tested it with your instructions and when the field is empty, then it shows no fields on the contact page. So for me it is working as expected. I'v just tested it with the latest staging.

avatar brianteeman
brianteeman - comment - 27 Mar 2017

ok let me test again with latest staging

avatar brianteeman
brianteeman - comment - 27 Mar 2017

Ignore me I was being an idiot. All good

avatar brianteeman brianteeman - close - 27 Mar 2017
avatar brianteeman brianteeman - change - 27 Mar 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-03-27 11:17:55
Closed_By brianteeman
avatar brianteeman brianteeman - change - 27 Mar 2017
Status Closed New
Closed_Date 2017-03-27 11:17:55
Closed_By brianteeman
avatar brianteeman brianteeman - change - 27 Mar 2017
Status New Pending
avatar brianteeman brianteeman - reopen - 27 Mar 2017
avatar brianteeman brianteeman - test_item - 27 Mar 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 27 Mar 2017

I have tested this item successfully on 691b8cf


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

avatar laoneo
laoneo - comment - 27 Mar 2017

Glad to hear.

avatar laoneo
laoneo - comment - 28 Mar 2017

@gwsdesk can you test it again as it doesn't count your test because of the upstream merge. Thanks.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

I have tested this item successfully on 691b8cf


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Mar 2017

RTC after two successful testes.

avatar rdeutz rdeutz - change - 30 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-30 21:22:21
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 30 Mar 2017
avatar rdeutz rdeutz - merge - 30 Mar 2017

Add a Comment

Login with GitHub to post a comment