search

lagotto / alm-report

ALM Reports
http://almreports.plos.org/
MIT License
8 stars 3 forks source link

Testing - Preview page display issues #66

Closed jenniferlin15 closed 9 years ago

jenniferlin15 commented 9 years ago

The display of articles on the Preview page has issues: 1) I add all articles published by date: Oct 6. 2) I add them all page by page (since Select All is broken). 3) I click on Preview report. The page has missing article metadata (see screenshot) for a subset of articles listed. Also, the page displays more than 25 results (should only be 25).

This also occurs when I upload a bulk file of DOIs. The tool takes you to the Preview page and the same issue occurs.

preview bug

jure commented 9 years ago

This bug is related to the new caching system and the current "Super Select All" functionality. The above Preview probably occurred sometime after the "Super Select All" functionality was used (or was tried). This added a bunch of really sparse documents into the cache, as this "Super Select All" requested only id and publication_date fields.

One fix for this is in https://github.com/articlemetrics/alm-report/commit/2e1135056512e593beb7b374e75c899375584954 but I'll need to take just a bit of a better look tomorrow.

jure commented 9 years ago

@jenniferlin15 Would it be possible for you to get me a file which causes these kinds of issues (i.e. the CSV you upload using bulk upload)?

jenniferlin15 commented 9 years ago

deinitely. i'm at work now. when i get home, i'll pull the file from my personal computer (originally used)

jure commented 9 years ago

Fixed by #73.