metasfresh / metasfresh-webui-api-legacy

metasfresh webui API server
GNU General Public License v3.0
30 stars 32 forks source link

Support Export of massive records to Excel #744

Open metasnw opened 6 years ago

metasnw commented 6 years ago

Is this a bug or feature request?

FR

What is the current behavior?

when you start process "download selected " with many records (15.000+) then nothing happens because the feature is designed for single and selective downloads (IDs are passed as parameters) which does not scale but supports multi select records.

Which are the steps to reproduce?

  1. have any window with 15K records
  2. have a role with a limit of >15K
  3. http://docs.metasfresh.org/webui_collection/EN/Export_data.html
  4. NOK: nothing happens
  5. NOK: server log throws expection ( de.metas.ui.web.view.ViewRestController.exportToExcel)
  6. NOK: when you try it again you get a server error in browser
  7. NOK: export does not work anymore

What is the expected or desired behavior?

  1. if more records are selected by the user than we support we should display an error message
  2. Allow user to export more than a few hundred records 1. if no records are selected we export all allowed by the role limit
metas-mk commented 6 years ago

@metasnw

if no records are selected we export all allowed by the role limit

we don't want this behavior pattern anymore because it is confusing for the user. the user shall do "select all" and then run an action. if nothing selected then nothing shall happen. ok to adjust the requirement?

metasnw commented 6 years ago

@metas-mk yes, sure! what ever solves it :-)

metasnw commented 6 years ago

trace: https://pastebin.com/5LfFSAiF

metasnw commented 6 years ago

tested on master 5.43.1-7854+master

  1. open window invoices with 4.700 records
  2. select all of page
  3. export
  4. OK: file downloaded

  1. open window invoices with 4.700 records
  2. select all of page, select allpages
  3. export
  4. OK: waited 11 min. then file was downloaded