Check the options tab of a custom field and try the various parameters.
All having an effect
Most don't have any effect
From what I see we don't need the following parameters in the field form:
title
isn't used as far as I see except for the fields manager. In the forms and item views the label
is used instead. One of it is useless imho.alias
isn't used anywhere and doesn't make sense for a field.image
and image alternate text
is unused as far as I saw.From what I see we don't need the following columns in the database table #__fields:
title
or label
(see above)alias
(see above)options
- couldn't find a way to populate thatattributes
- couldn't find a way to populate thatcreated_by_alias
- doesn't make sense for a fieldversion
- fields doesn't seem to support versioning anywaypublish_up
and publish_down
- currently unused.hits
- doesn't make sense for a fieldLabels |
Added:
?
|
Concerning Title and Label, I think we should keep them as LABEL can be translatable via a new language string and, in this case, it has to obey to our usual rules for string constants.
This may not be really readable vs Title.
Category | ⇒ | com_fields |
Title |
|
Images can be used in a template override, but this is not implemented yet, just one more possibility for the site admin when he is doing an override.
Images can be used in a template override, but this is not implemented yet, just one more possibility for the site admin when he is doing an override.
Yes sure, in overrides everything is possible. But I don't see the usecase for it to store data which eventually could be used in an override. If we have the options, it should be used in core or removed. Otherwise users will be confused and telling us core is broken.
Found out the alias is used as the field id in the HTML code. So maybe it should be named differently?
Many users of DPFields did use the alias also for a more human readable lookup.
Found out the alias is used as the field id in the HTML code. So maybe it should be named differently?
Many users of DPFields did use the alias also for a more human readable lookup.
I have a similar thing to 'alias' called 'name' of the field,
I would suggest we have an advanced config option in com_fields which is used if doing template overrides which shows the additional input values that the user can then use in the front end in an override. Displaying inputs which have no effect in the default output would lead to a very confused user.
There is (currently) no configuration at all for com_fields.
Imho it makes no sense to have additional database columns which may only be used (or not) by templates overrides.
Most of the mentioned fields even don't make sense in an override. For example "hits", "created_by_alias", "publish_up" and "publish_down". They would need code to deal with it.
The image may be used in an override, but really what is the usecase? Why would you want to attach an image to a field? If you need a template override anyway to show it, you could as well just add some code there hardcoded to show that image you want. Or even better use CSS and show the image based on a class on the field.
I think it's also for the fact the DPFields will have to provide a migration for it's current users.
We should probably look at adding ACL for com_fields, what ACL is controlling who can add fields?
ACL should be working with PR #13019.
As for the users of DPFields, I have no clue. Does that one use the same component name (com_fields)? If so then it's an issue, otherwise not. I'd rather rename com_fields component then to something else to avoid that issue. DPFields can add a migration path to its extension, not a core issue to deal with.
DPfields is "com_dpfields"
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-01-17 15:41:28 |
Closed_By | ⇒ | Bakual |
Version and hits has been already removed can't remember the pr number