- @steven-jones opened merge request.
- ๐ฌ๐งUnited Kingdom steven jones
There's actually a method on
\Drupal\views\ViewExecutable
that tells us if there's a URL, I wonder if that's what we need to check here?@dieterholvoet are you able to test this MR I'm about to open please?
- ๐ฌ๐งUnited Kingdom steven jones
steven jones โ made their first commit to this issueโs fork.
- @dieterholvoet opened merge request.
- ๐ง๐ชBelgium dieterholvoet Brussels
Hitting the same issue. It's happening because
Drupal\views_aggregator\Plugin\views\style\Table::preRender()
creates a duplicate and executes it without saving. Down the line, inDrupal\views_data_export\Plugin\views\display\DataExport::attachTo()
,$view->getUrl()
is called and fails because the attached view doesn't have an ID yet, because it is new and unsaved.I'm able to fix this in Views data export โ by calling
isNew()
before trying to generate an url. I believe that's the way to go, so I'm moving this issue to that project.