? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
18 Jan 2019

Pull Request for Issue #23569

Summary of Changes

Deleting unused contact fields:
banned_email
banned_subject
banned_text

Keeping them in the Component Options as, since 2.5, it is the only place where they work.

Deprecating some ini strings
Corrected sql files
Modified some unit test files EXCEPT /tests/unit/stubs/database/jos_finder_links.csv which I have no idea how to clean.
For that one, @mbabker @wilsonge @Hackwar please make PR against mine

Testing Instructions

Edit a contact, a contact category, all contacts menu items and make sure the 3 fields do not appear.
Check the sql for possible errors.

avatar infograf768 infograf768 - open - 18 Jan 2019
avatar infograf768 infograf768 - change - 18 Jan 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jan 2019
Category Administration com_contact Language & Strings Front End SQL Installation Postgresql MS SQL Unit Tests
avatar ChristineWk
ChristineWk - comment - 18 Jan 2019

I have tested this item successfully on 4f51528

- Contact fields under: components > contacts > contacts > mycontact > Form:
banned_email
banned_subject
banned_text
are deleted.

  • In com_contacts Global Options it works - still working :-)

  • sql > sorry, don't know


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23585.
avatar ChristineWk
ChristineWk - comment - 18 Jan 2019

I have tested this item successfully on 4f51528

- Contact fields under: components > contacts > contacts > mycontact:
banned_email
banned_subject
banned_text
are deleted.

  • In com_contacts Global Options it works - still working :-)

  • sql > sorry, don't know


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23585.
avatar ChristineWk ChristineWk - test_item - 18 Jan 2019 - Tested successfully
avatar viocassel
viocassel - comment - 20 Jan 2019

I have tested this item successfully on 4f51528


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

avatar viocassel viocassel - test_item - 20 Jan 2019 - Tested successfully
avatar infograf768 infograf768 - change - 20 Jan 2019
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 20 Jan 2019

Rtc
@maintainers please look at the file I did not correct, if it is still used.


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

avatar Quy
Quy - comment - 20 Jan 2019

RTC


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

avatar HLeithner HLeithner - change - 1 Feb 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-01 10:59:35
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 1 Feb 2019
avatar HLeithner HLeithner - merge - 1 Feb 2019
avatar HLeithner
HLeithner - comment - 1 Feb 2019

thx

Add a Comment

Login with GitHub to post a comment