NoLog.cz/etherpad
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity

contentcollector: Skip over non-Text, non-Element Nodes

Browse source
This commit is contained in:
Richard Hansen 2021-01-24 02:07:33 -05:00 committed by John McLear
parent 075969aea0
commit 1cb5453aeb
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/static/js/contentcollector.js
View file

@ -39,6 +39,7 @@ const makeContentCollector = (collectStyles, abrowser, apool, className2Author)
const dom = { const dom = {
// .nodeType works with DOM and cheerio 0.22.0. Note: Cheerio 0.22.0 does not provide the // .nodeType works with DOM and cheerio 0.22.0. Note: Cheerio 0.22.0 does not provide the
// Node.*_NODE constants, so they cannot be used here. // Node.*_NODE constants, so they cannot be used here.
isElementNode: (n) => n.nodeType === 1, // Node.ELEMENT_NODE
isTextNode: (n) => n.nodeType === 3, // Node.TEXT_NODE isTextNode: (n) => n.nodeType === 3, // Node.TEXT_NODE
// .tagName works with DOM and cheerio 0.22.0, but: // .tagName works with DOM and cheerio 0.22.0, but:
// * With DOM, .tagName is an uppercase string. // * With DOM, .tagName is an uppercase string.
@ -395,7 +396,7 @@ const makeContentCollector = (collectStyles, abrowser, apool, className2Author)
cc.startNewLine(state); cc.startNewLine(state);
} }
} }
} else { } else if (dom.isElementNode(node)) {
const tname = dom.tagName(node) || ''; const tname = dom.tagName(node) || '';
if (tname === 'img') { if (tname === 'img') {