? ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
5 Jun 2021

The form for an individual contact defines fields for all the icon images so that you can override them at the contact level as well as the component level.

However the fields are never displayed so they are never used. I have gone back over 8 years in the history and I cant see them being used at any point in that time period.

This pr removes the fields from the xml. No point in maintaining code that is never used

avatar brianteeman brianteeman - open - 5 Jun 2021
avatar brianteeman brianteeman - change - 5 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jun 2021
Category Administration com_contact
avatar chmst
chmst - comment - 5 Jun 2021

The icons are used in single contacts. Here for example I have added an image as icon-address

grafik

avatar brianteeman
brianteeman - comment - 5 Jun 2021

how? By using the form in the component options? That hasnt changed.

avatar chmst
chmst - comment - 5 Jun 2021

Clobal confiuration - options. sry .. will be back later

avatar brianteeman
brianteeman - comment - 5 Jun 2021

Thought so. I spent a stupid amount of time checking this

avatar chmst
chmst - comment - 5 Jun 2021

Don't blame me ;) I spent an insane amout of time to find this: it is the marker-class
https://github.com/joomla/joomla-cms/blob/4.0-dev/components/com_contact/tmpl/contact/default_address.php#L27

avatar brianteeman
brianteeman - comment - 5 Jun 2021

They're still global params aren't they?. Unless you have also found out where you can set the params for an individual contact

avatar chmst
chmst - comment - 5 Jun 2021

There is no chance for individual contacts. I think that this was long time before my "active time".
The default icons were somehow knuffy - 1990 style. And web site admins should have own icons for all their contacts (for all contacts).

avatar brianteeman
brianteeman - comment - 5 Jun 2021

There is no chance for individual contacts. I think that this was long time before my "active time".

As I said I went back 8 years without finding it

The default icons were somehow knuffy - 1990 style. And web site admins should have own icons for all their contacts (for all contacts).

Agreed and they still can. This PR doesnt change that. The only thing this pr does is to remove a bunch of fields that are defined in xml and nowhere else

avatar chmst
chmst - comment - 5 Jun 2021

Removing these fields would be a B/C break. If websites have adde here own icons. If they are removed, the lyouts can be made much easier.

avatar brianteeman
brianteeman - comment - 5 Jun 2021

Removing these fields would be a B/C break.

How? they are NOT used. The ones that are used are in the global component options

avatar brianteeman
brianteeman - comment - 5 Jun 2021

Or am I missing something

avatar chmst
chmst - comment - 5 Jun 2021

Params are used here:
https://github.com/joomla/joomla-cms/blob/4.0-dev/components/com_contact/src/View/Contact/HtmlView.php#L266

Maybe I am missing something? I never use contact without own override, so am not expert here.

avatar brianteeman
brianteeman - comment - 5 Jun 2021

Those icons are defined in the global options for the contact component aren't they?

Thats not the fields this PR is removing.

Please open joomla in your browser. You can set custom icons in the component options. There is nowhere you can set custom icons for an individual contact.

Now apply this PR. Nothing has changed. Therefore the fields in this xml are not used and can be deleted.

avatar Quy
Quy - comment - 6 Jun 2021

See if #31435 confirms this.

avatar brianteeman
brianteeman - comment - 6 Jun 2021

bingo

avatar chmst
chmst - comment - 6 Jun 2021

My bad - I mixed up the xml files. You are right.

avatar Fedik
Fedik - comment - 6 Jun 2021

the language strings also need to be removed or?

ah I see, it also in component config, still in use

avatar brianteeman
brianteeman - comment - 6 Jun 2021

Now we know the fields that are being removed can we get some tests please

avatar Quy
Quy - comment - 7 Jun 2021

Should the info be updated/removed as this is the only place that these settings can be configured?

Should this be fixed in J3 too?

34418

avatar chmst
chmst - comment - 7 Jun 2021

@Quy yes, the message is wrong.

avatar brianteeman
brianteeman - comment - 7 Jun 2021

And it was always bad english. ;)

I could remove it but then there is a consistency issue with the other tabs. How about simply

These settings apply to a single contact

avatar chmst
chmst - comment - 7 Jun 2021

I don't use this feature, so cannot say where is the advantage if every contact has own icons. You could change the message and say "these settings are for all contacts and cannot be set individually" (in correct english)?

avatar brianteeman brianteeman - change - 7 Jun 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jun 2021
Category Administration com_contact Administration com_contact Language & Strings
avatar chmst chmst - test_item - 7 Jun 2021 - Tested successfully
avatar chmst
chmst - comment - 7 Jun 2021

I have tested this item successfully on 5941306


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

avatar ChristineWk ChristineWk - test_item - 7 Jun 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 7 Jun 2021

I have tested this item successfully on 5941306


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

avatar ChristineWk
ChristineWk - comment - 7 Jun 2021

with Patch:

screen shot 2021-06-07 at 11 56 34


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

avatar chmst chmst - change - 7 Jun 2021
Status Pending Ready to Commit
avatar chmst
chmst - comment - 7 Jun 2021

RTC


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

avatar rdeutz rdeutz - close - 7 Jun 2021
avatar rdeutz rdeutz - merge - 7 Jun 2021
avatar rdeutz rdeutz - change - 7 Jun 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-07 21:01:53
Closed_By rdeutz
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 7 Jun 2021

thank you

Add a Comment

Login with GitHub to post a comment