- 🇳🇿New Zealand RoSk0 Wellington
Spent several hours debugging 🐛 _access tag can leading to fatal error Closed: won't fix that lead to creating 🐛 Query metadata get lost when $view->query->query() is called multiple times during page build Active .
I was going to propose a similar change to #12, but surprised people reporting it as problematic as my understanding was quite firm on that being the correct approach. It would be awesome if people can share details about what that approach breaks.
I completely share the concerns of #23 around performance , but mostly afraid of the failing tests . Quick patch test however succeeded, but that is only a quick test full test suite is running and I will report back here later.
I don't quite understand the purpose of modifying cache configuration of the view, so proposing a variation of #10 without changing cache settings , but with restoration of the original configuration as I believe this is crucial part that was missing in the original implementation - without it modified view is passed further for batch processing.
- 🇮🇷Iran sgarsivaz
I use views_data_export:^1.2 and have more than 10K row and 5 column to export. three of this 5 column is views_field_view which fetch data from different views.
without any patch I can not export any thing. by using #25 patch problem solved.
but I have to use batch size = 100. any batch size more than that result in an error. by this batch size it takes me 30 to 40 minutes. - 🇬🇧United Kingdom jonathanshaw Stroud, UK
#25 looks reasonable.
This seems tricky to debug and reason about, so a test would be really good. This would also establish that it was fine to follow #25 in not unsetting the cache.
- heddn Nicaragua
#25 fixed the issue for me too. It wasn't clear if that causes a re-execution of the query or not. Can someone confirm?
- 🇮🇳India vishal.kadam Mumbai
Those who are experiencing data inconsistency issue in batch as compared to standard mode.
I've identified the source of the batch export limit problem. It has to do with the sequence of query results.
In order to prevent changes in the query result sequence, add the unique fields to the sort.
- 🇸🇮Slovenia alesbencina
I've tried patch #25 and worked well. The issue I had was that the view exposed filters weren't taken in countQuery() and therefore it always resulted into max number of rows (I have more than 100k in a table).
Acctually $view->build(); helped a lot which added the filters from the request.
- last update
5 months ago 3 pass - Status changed to RTBC
2 months ago 10:11am 23 August 2024 - 🇺🇦Ukraine rollins
I can confirm that patch #25 resolved the bug
Thank you @rosk0 - 🇺🇸United States maskedjellybean Portland, OR
#25 worked for me as well. Thanks!