From 27a557b5c5af1fee23c8be1ffae668b5ed6ba28d Mon Sep 17 00:00:00 2001
From: Jules Laplace <julescarbon@gmail.com>
Date: Thu, 28 Mar 2019 17:10:40 +0100
Subject: fix sorting

---
 scraper/client/paper/citationList.component.js | 36 ++++++++++++++++----------
 1 file changed, 23 insertions(+), 13 deletions(-)

(limited to 'scraper/client/paper/citationList.component.js')

diff --git a/scraper/client/paper/citationList.component.js b/scraper/client/paper/citationList.component.js
index ca4d2482..10e3ba9a 100644
--- a/scraper/client/paper/citationList.component.js
+++ b/scraper/client/paper/citationList.component.js
@@ -9,28 +9,38 @@ import { TableObject, Loader } from '../common'
 import { USES_DATASET } from '../types'
 
 class CitationList extends Component {
+  componentDidMount() {
+    const { citations, api } = this.props
+    const { paperInfo, unknownCitations, verifications } = api
+    const { dataset } = paperInfo
+    if (!dataset || !citations || !verifications[dataset.key]) {
+      this.props.actions.setSortedCitations([])
+      return
+    }
+    let verifiedLookup = verifications[dataset.key] || {}
+    const sortedCitations = citations.map(citation => [
+        citation.title,
+        verifiedLookup[citation.id] ? verifiedLookup[citation.id].uses_dataset : USES_DATASET.NO_DATA,
+        citation.pdf.length,
+        citation
+      ])
+      .sort((a,b) => (b[1] - a[1] || b[2] - a[2] || (a[0].localeCompare(b[0]))))
+      .map(tup => tup[3])
+    this.props.actions.setSortedCitations(sortedCitations)
+  }
+
   render() {
     const { citations, title, api } = this.props
-    const { paperInfo, unknownCitations, verifications } = api
+    const { paperInfo, unknownCitations, verifications, sortedCitations } = api
     const { dataset } = paperInfo
     if (!dataset || !citations || !verifications[dataset.key]) return <Loader />
     let verifiedLookup = verifications[dataset.key] || {}
     // console.log(verifications)
-
     return (
       <div className='citations'>
         <h2>{title}</h2>
         <ul>
-        {citations
-        .map(citation => [
-          citation.title,
-          verifiedLookup[citation.id] ? verifiedLookup[citation.id].uses_dataset : USES_DATASET.NO_DATA,
-          citation.pdf.length,
-          citation
-        ])
-        .sort((a,b) => (b[1] - a[1] || b[2] - a[2] || (a[0].localeCompare(b[0]))))
-        .map((pair, i) => {
-          const [ title, uses_dataset, citation ] = pair
+        {(sortedCitations || []).map((citation, i) => {
           let cite = { ...citation }
           cite.id = {
             _raw: true,
@@ -92,7 +102,7 @@ class CitationList extends Component {
 }
 
 const mapStateToProps = state => ({
-  api: state.api
+  api: state.api,
 })
 const mapDispatchToProps = dispatch => ({
   actions: bindActionCreators({ ...actions }, dispatch),
-- 
cgit v1.2.3-70-g09d2