- ๐ณ๐ฟNew Zealand quietone
Steps to reproduce with Drupal core were asked for 2 years ago and haven't been supplied. Also, a view maintainer has not been able to reproduce the problem. Steps to reproduce were again asked for 9 months ago.
Since that hasn't been supplied I am closing this issue.
If you are experiencing this problem on a supported version of Drupal reopen the issue, by setting the status to 'Active', and provide complete steps to reproduce the issue โ (starting from "Install Drupal core").
- Status changed to Needs review
about 1 year ago 1:33pm 23 October 2023 - Status changed to Needs work
about 1 year ago 1:38pm 23 October 2023 - ๐บ๐ธUnited States smustgrave
Still canโt replicate. If you are going to reopen could you update the steps to reproduce please.
- ๐ต๐ฑPoland lamp5 Rzeszow
Steps to reproduce has been update. Please check first post or https://www.drupal.org/node/3263208/revisions โ
- Assigned to john.oltman
- Merge request !10352Issue #3263208: Missing schema on views with aggregation โ (Open) created by john.oltman
- ๐บ๐ธUnited States john.oltman
Added MR, as expected the "test only" pipeline failed due to schema errors, otherwise the fix passes.
This is holding up a recipe for Drupal CMS that cannot pass tests until this is fixed, so hoping this can be reviewed ASAP.
- ๐ฎ๐ณIndia sagarmohite0031
Hello,
Still canโt replicate.
Need clear steps to reproduce the issue. - ๐บ๐ธUnited States john.oltman
The steps work to replicate, but since there is an MR with a "test only" fail, they are less important now. If you really want to see it, import this view into a test site. https://git.drupalcode.org/project/drupal/-/blob/2bd05246e8bba0ff72332b0...
Aggregating with a COUNT on any entity ID (a user ID, in that example) will create a missing schema error. Look at the artifacts from the MR tests. It's right there.
- ๐บ๐ธUnited States john.oltman
Here is the "test only" run showing the missing schema errors when the test view (a simple view of users with a COUNT on user ID) is installed into a site without the accompanying schema fixes: https://git.drupalcode.org/issue/drupal-3263208/-/jobs/3498941
- ๐บ๐ธUnited States john.oltman
Updated the MR with a post update hook and accompanying test to ensure the hook works. Back to ready for review.
- ๐บ๐ธUnited States smustgrave
Believe feedback on this one has been addressed! Nice!