Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There is a bug in client side CSV export, function $scope.getAllData() #51

Open
dariuspranskus opened this issue Aug 5, 2019 · 3 comments

Comments

@dariuspranskus
Copy link

dariuspranskus commented Aug 5, 2019

Hello

Description

There is a bug in client side CSV export code, specifically in $scope.getAllData() function. This causes some lines being omitted from the export for each 500 lines or so.

The incorrect line is line 153. I believe index should not be added while calculating qTop:

var page = {
    qTop: (pageHeight * index) + index, // +index should not be here
    qLeft: 0,
    qWidth: columns,
    qHeight: pageHeight,
    index: index
};

Steps to Reproduce

  1. Take any application which has more that 500 records
  2. Set the extension object up. Choose Comma separated CSV - Client side option
  3. Export the data using the extension
  4. Export the data using QlikSense native functionality
  5. Compare the results, specifically number of rows.
  6. See some records missing from the extension export.

Proposed solution

Remove + index from qTop value calculation.

Thanks

@dariuspranskus dariuspranskus changed the title There is a bug in client side CSV export, function getAllData() There is a bug in client side CSV export, function $scope.getAllData() Aug 5, 2019
@eagaratez
Copy link

I also noticed the missing entries however I was suspecting that it was related to a 'for loop' gathering 499 values instead of 500 due to a < instead of a <=. I'm going to try your fix, thanks.

@gepascual
Copy link

Hi, I had the same problem. Is there a solution?

@jbaberdc
Copy link

jbaberdc commented Nov 3, 2022

Same here! Is there any resolution in the works for this? I see that this is a pretty old issue thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants