From 52234e5fdaeb59ee4c2e25e0d28a753e7012b589 Mon Sep 17 00:00:00 2001 From: Mimi <1119186082@qq.com> Date: Wed, 30 Sep 2020 14:11:23 +0800 Subject: [PATCH] Use `textContent` instead of `innerText` --- layout/_third-party/statistics/firestore.njk | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/layout/_third-party/statistics/firestore.njk b/layout/_third-party/statistics/firestore.njk index 167b807..e80fa0e 100644 --- a/layout/_third-party/statistics/firestore.njk +++ b/layout/_third-party/statistics/firestore.njk @@ -44,8 +44,10 @@ const db = firebase.firestore(); const articles = db.collection('{{ theme.firestore.collection }}'); - if (CONFIG.page.isPost) { // Is article page - const title = document.querySelector('.post-title').innerText.trim(); + if (CONFIG.page.isPost) { + // Fix issue #118 + // https://developer.mozilla.org/en-US/docs/Web/API/Node/textContent + const title = document.querySelector('.post-title').textContent.trim(); const doc = articles.doc(title); let increaseCount = CONFIG.hostname === location.hostname; if (localStorage.getItem(title)) { @@ -55,9 +57,9 @@ localStorage.setItem(title, true); } getCount(doc, increaseCount).then(appendCountTo(document.querySelector('.firestore-visitors-count'))); - } else if (CONFIG.page.isHome) { // Is index page + } else if (CONFIG.page.isHome) { const promises = [...document.querySelectorAll('.post-title')].map(element => { - const title = element.innerText.trim(); + const title = element.textContent.trim(); const doc = articles.doc(title); return getCount(doc); });