? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
13 Jun 2017

Pull Request for Issue #16574.

Summary of Changes

When a "Contact - Mail" custom field is assigned to a category, then on the front only the fields in the contact form should be shown where the actual contact belongs to.

Testing Instructions

  • Create a contact category A
  • Create a contact category B
  • Create a contact A for category A
  • Create a contact B for category B
  • Create a Mail custom field with the title A and assign it to category A
  • Create a Mail custom field with the title B and assign it to category B
  • Create a menu item for contact A
  • Create a menu item for contact B

Expected result

  • When opening contact A on the front only the custom field A should be shown below the contact form.
  • When opening contact B on the front only the custom field B should be shown below the contact form.

Actual result

  • When opening contact A on the front the custom field A and B are shown below the contact form.
  • When opening contact B on the front the custom field A and B are shown below the contact form.
avatar laoneo laoneo - open - 13 Jun 2017
avatar laoneo laoneo - change - 13 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jun 2017
Category Front End com_contact
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Jun 2017

I have tested this item successfully on fc597d5


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 13 Jun 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Jun 2017

@federone @jnk22 please test.

avatar sandstorm871
sandstorm871 - comment - 14 Jun 2017

Looking to test this & hopefully not a daft question ;)
Tested in 3.7.3b1 & 3.8dev

How do you add "Create a Mail custom field with the title A "
I see custom fields as expected for text, textarea, calenadra, etc but none more mail?


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 Jun 2017
avatar sandstorm871
sandstorm871 - comment - 14 Jun 2017

I have tested this item successfully on fc597d5

Set up as per test instructions & saw the issue as per described.
After applying patch #16666 saw the expected result.


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

avatar sandstorm871 sandstorm871 - test_item - 14 Jun 2017 - Tested successfully
avatar sandstorm871
sandstorm871 - comment - 14 Jun 2017

Ooh didn't know about that @franz-wohlkoenig Thanks.
Now successfully tested with patch #16666


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 14 Jun 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 Jun 2017

RTC after two successful tests.

avatar rdeutz rdeutz - change - 15 Jun 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-15 06:54:41
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 15 Jun 2017
avatar rdeutz rdeutz - merge - 15 Jun 2017
avatar federone
federone - comment - 18 Jun 2017

@franz-wohlkoenig thank you very much, have tested and works well.

Best Wishes,
Federico

Add a Comment

Login with GitHub to post a comment