[com_fields] Undefined property: stdClass::$assigned_cat_ids, com_fields

[anibalsanchez]

Steps to reproduce the issue

1. Define a field in Users
2. Navigate to create or edit a user

Expected result

No notice

Actual result

Notice: Undefined property: stdClass::$assigned_cat_ids, com_fields/helpers/fields.php on line 453

System information (as much as possible)

Additional comments

[Bakual]

@laoneo That's because com_users doesn't use categories, right? The parameter is thus missing in the field definition as well.
I think I can make a PR tomorrow to deal with that.

[laoneo]

This issue reveals a problem when a component doesn't use categories for grouping. Correctly implemented, we need to offer for the component a way to define the hardcoded category_id more dynamically

• https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_fields/models/field.php#L152
• https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_fields/helpers/fields.php#L118.

I guess this would mean that we can't use the category form field here https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_fields/models/forms/field.xml#L35. Instead of we need to make our own generic one. But I don't know if this is doable in a stable way?

[Bakual]

The first one isn't an issue as $tupel->category_id refers to the #__fields_categories table. That will always be that column name 😄

The other one may be an issue if an extension doesn't use catid as property. However it may also be possible for them to just copy the property to catid before it goes through fields. Otherwise we would need a way to alias the category property of an item. But it isn't the cause of the issue here.

The category field however is fine. That should work with all extensions using com_categories.

[Bakual]

See PR #13570

Closing this.

[laoneo]

The category field however is fine. That should work with all extensions using com_categories.

Like it is now, it ONLY works for categories. It is not possible to assign fields to user groups or other component specific grouping items.

[Bakual]

Ah sure, that isn't possible. But it's also not something which we tried to support so far.

[laoneo]

That's what I wanted to say 🙊