Skip to content

fix itemprop containing articleBody#953

Open
AndyTheFactory wants to merge 1 commit into
codelucas:masterfrom
AndyTheFactory:patch-1
Open

fix itemprop containing articleBody#953
AndyTheFactory wants to merge 1 commit into
codelucas:masterfrom
AndyTheFactory:patch-1

Conversation

@AndyTheFactory

Copy link
Copy Markdown

If itemprop is not exactly == "articleBody" the node was "cleaned"

for instance itemprop="description articleBody" would be cleaned. Blogspot / Blogger for instance uses this itemprop

If itemprop is not exactly == "articleBody" the node was "cleaned"

for instance itemprop="description articleBody" would be cleaned. 
Blogspot / Blogger for instance uses this itemprop
dc-harsh added a commit to dc-harsh/newspaper that referenced this pull request May 29, 2026
- requirements: add lxml-html-clean (fixes `import newspaper` breaking on lxml>=5)
- nlp: NLTK punkt_tab fallback in split_sentences (codelucas#1023) + collapse
  intra-sentence newlines (codelucas#873); download_corpora adds punkt_tab (codelucas#1023)
- cleaners: itemprop contains("articleBody") keeps multi-token itemprop nodes (codelucas#953)
- extractors: include <article> in scoring candidate nodes (codelucas#865)
- tests: expect precise meta article:published_time (intended fork behavior)
- bump version to 0.4.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant