Merge pull request #234 from tomaszrondio/patch-5

SWP-1904 List articles with missing article_statistics correctly
superdesk · Jan 29, 2020 · 08fc93f · 08fc93f
2 parents 8a95e02 + 7ba2221
commit 08fc93f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/client/__tests__/components/ContentLists/Automatic/__snapshots__/ArticleItem.js.snap b/client/__tests__/components/ContentLists/Automatic/__snapshots__/ArticleItem.js.snap
@@ -22,7 +22,7 @@ exports[`ContentLists/Automatic/ArticleItem renders correctly 1`] = `
       <span
         class="sd-overflow-ellipsis sd-list-item--element-grow"
       >
-        2 years ago
+        3 years ago
       </span>
       <span
         class="label"

diff --git a/client/services/PublisherHelpers.js b/client/services/PublisherHelpers.js
@@ -56,7 +56,9 @@ export function PublisherHelpersFactory() {
       let count = 0;
 
       articles.forEach(art => {
-        count += parseInt(art.article_statistics.page_views_number);
+        if(art.article_statistics && art.article_statistics.page_views_number) {
+          count += parseInt(art.article_statistics.page_views_number);
+        }
       });
 
       return count;
@@ -72,7 +74,9 @@ export function PublisherHelpersFactory() {
       let count = 0;
 
       articles.forEach(art => {
-        count += parseInt(art.comments_count);
+        if(art.comments_count) {
+          count += parseInt(art.comments_count);
+        }
       });
 
       return count;