[staging] Contact Banned parameters work only from component Options

[infograf768]

Pull Request for Issue #23546

Summary of Changes

Concerns Banned Email, Banned Subject, Banned Text parameter fields.

The validation of these, before this patch, is only done depending on the parameters defined in the component Options Form tab.
Whatever has been entered in the Contact menu item or in the Contact itself are ignored.

Testing Instructions

Tests have to be done when one sends mail to a contact in frontend.

The priorities for these parameters depend if the contact is displayed from a single contact menu item or not.

Set the parameters in the various places below and test priorities. Creating the params and taking them off between each test.

Case1: From a single contact menu item

Priority1: menu item parameters
Priority2: Contact parameters
Priority3: Component parameters

Case2: No single contact menu item
For example for another type of contacts menu item or from no menu item at all when clicking on the author link in an article.

Priority1: Contact parameters
Priority2: Component parameters

Test is successful when one gets a message of the type depending on the field where the banned item is used.
Error:
Invalid field: Email
Invalid field: Subject
Invalid field: Message

Before patch

Only the Component parameters are used.

After patch

Parameters used depend on the priorities.

[ChristineWk]

I have tested this item ✅ successfully on 80bfdde

Thank you @infograf768 - will inform forum accordingly

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23569.}

[viocassel]

I have tested this item ✅ successfully on 80bfdde

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23569.}

[Quy]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23569.}

[infograf768]

To tell the truth, as these fields (as present for a contact or single contact menu item) never worked, even in 2.5 where we do have exactly the same code, another solution is to get rid of them and just keep the component Options ones.
It looks like this has not created issues for years and it did only when someone tried to use them.

@joomla/cms-maintainers @ChristineWk @viocassel
What do you think?

EDIT: This means deleting them in 6 .xml, 12 .sql, and some .csv and one .xml test files.

[Hackwar]

+1 for deleting

[wilsonge]

Delete them

[ChristineWk]

@infograf768 Personally, I agree with you, but I do not know what other users or the thread creator means about it. Anyway Thanks for your investigation!

[infograf768]

OK, will work on this and when ready will close this PR.

[infograf768]

Please see #23585
There is one .csv which needs love and care.
Closing this PR.