- Issue created by @Turneight
- Open on Drupal.org โEnvironment: PHP 8.1 & MySQL 8
55:37 55:10 Queued - Status changed to Postponed: needs info
about 1 year ago 2:03am 6 September 2023 - last update
about 1 year ago 29,471 pass - Status changed to Needs review
about 1 year ago 6:05pm 6 September 2023 - Status changed to Needs work
about 1 year ago 7:30pm 6 September 2023 - ๐บ๐ธUnited States smustgrave
Thank you for reporting. Tagging for a test case to be added
- ๐ฎ๐นItaly Turneight
Not only the count aggregation type causes the error but all but the default "group" one.
All types of aggregation produce the same error.
The error is also present in drupal 9.5.x and has already been reported.
The basic problem is that by setting a grouping type other than the default "group" the form for selecting the columns does not appear upon subsequent modifications.
The proposed patch does not solve the main problem, but it allows you to work on your vision and not lose all the work done.
I haven't found any problems that the patch can cause.
I'm looking for the cause of the main problem but I don't know the module enough, it could also depend on the views_ui module - ๐ฎ๐นItaly Turneight
I found the main problem!
The EntityField class which extends FieldPluginBase which extends HandlerBase is used to submit the form.
The NumericField class which extends FieldPluginBase which extends HandlerBase is used in the construction of the form.
The definition of the group_column and group_columns fields occurs in the EntityField class which is not called when constructing the form.
Proposed solution:
In the ConfigHandlerGroup->buildForm class of the views_ui module, ensure that in this case EntityField is called as handler instead of NumericField.
I ask for confirmation from those who know these two modules better. - ๐ฎ๐นItaly Turneight
The path ensures that the same handler is used both for building and sending the form.
- last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago 29,471 pass - Status changed to Closed: duplicate
about 1 year ago 6:41am 12 September 2023 - ๐ณ๐ฑNetherlands Lendude Amsterdam
As pointed out in #10, this is a duplicate of ๐ Ajax error on views with aggregation Postponed: needs info , so closing this one so we can focus our efforts to fix this in one spot.