Parsoid doesn't properly handle double-underscore magic words
Open, Needs TriagePublic
Actions

Assigned To

Authored By

	cscott
	Aug 26 2024, 6:51 PM

Description

For the double-underscore magic words __HIDDENCAT__, __NOINDEX__, and __INDEX__, Parsoid is missing the tracking category corresponding to this magic word. This seems to indicate that Parsoid is not actually invoking the legacy parser to handle these magic words, and so the resulting ParserOutput is likely to be omitting the proper index policy values as well.

The __NOGALLERY__ and __NOTOC__ magic words are handled in the same place of the legacy parser, and might also be missing some necessary behavior flags.

Event Timeline

cscott created this task.Aug 26 2024, 6:51 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 26 2024, 6:51 PM

MSantos assigned this task to cscott.Aug 29 2024, 2:28 PM

MSantos added a project: Content-Transform-Team-WIP.

MSantos moved this task from Uncategorized to Phase 1 - DiscussionTools support on the Parsoid-Read-Views board.

MSantos edited projects, added Parsoid-Read-Views (Phase 1 - DiscussionTools support); removed Parsoid-Read-Views.

MSantos moved this task from Needs Triage to Missing Functionality on the Parsoid board.Thu, Sep 19, 2:25 PM

We have Env::{get,set}BehaviorSwitch in Parsoid, and this potentially violates fragment independence as well when we interrogate the state of a number of these behavior switches in src/Wt2Html/DOM/Processors/WrapSectionsState.

We should probably be invoking some code in SiteConfig to handle the switches to ensure that we don't drift from the core implementation, although we can probably handle it in Parsoid in src/Wt2Html/TT/BehaviorSwitchHandler.

Parsoid doesn't properly handle double-underscore magic wordsOpen, Needs TriagePublicActions

Description

Event Timeline

Parsoid doesn't properly handle double-underscore magic words
Open, Needs TriagePublic
Actions