search

kugra-al / al-laravel

Status: live
0 stars 0 forks source link

.csv output broken - output all rows instead of just visible ones #23

Closed kugra-al closed 1 year ago

kugra-al commented 1 year ago

.xlsx works

It's trying to output all 170+ fields instead of only the visible ones. Probably hitting URL length. Only visible columns should be exported.

Print also broken (uses all allocated memory) - likely same issue. There is something somewhere about hiding non-visible cols from exports, just need to find it again.

kugra-al commented 1 year ago

https://github.com/yajra/laravel-datatables/issues/1558

https://github.com/yajra/laravel-datatables-buttons/pull/78

https://github.com/yajra/laravel-datatables-buttons/pull/78#issuecomment-639260332

Need to use POST for export methods instead of GET to expose getVisibleColumns. So postExcelVisibleColumns instead of adding buttons in html. Just need to find where that is...

Ed: is here - https://github.com/yajra/laravel-datatables-buttons/blob/8bdd20623bfd21dcb586b916e22132ec9bb2907d/src/resources/assets/buttons.server-side.js#L120 - but where to put? Can't extend, unknown button type