In multi-content/template-block scenarios, Linter displays "--" in the "Through a template"? column
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	ssastry
	Apr 13 2017, 4:46 PM

Description

As far as I can tell, when template info for a linter error is not a single template but actually a multi-content block, linter is displaying a "--" in the "Through a template"? column. This is misleading. Something else should be displayed there. Unsure if this is a Parsoid issue or a Linter UI issue.

Details

	Subject	Repo	Branch	Lines +/-
	Linter: Add a flag to templateInfo for multi-part templated blocks	mediawiki/services/parsoid	master	+9 -11
	Handle multi-part-template-block output from Parsoid's linter	mediawiki/extensions/Linter	master	+12 -5

Customize query in gerrit

Related Objects

Mentioned In: T165538: Need a mechanism / process to purge invalid / stale entries from the Linter database
T163549: Only lint pages that have wikitext contentmodel
T161151: Parsoid should resolve template paths before providing them to Linter
T164006: Suggestion: API for fetching lint errors for a specific revision
T163091: Parsoid: Add API endpoint to get lint errors for arbitrary wikitext
T153107: Parsoid is generating [[Foo|Foo]] instead of [[Foo]] for some VE edits
T37247: content-holding <div> should only contain the page text
T164792: Add class mw-parser-output to Parsoid's output
T141226: Missing data-mw content in wikitext leads to html2wt exceptions
rELINT3eea8ed824e6: Handle multi-part-template-block output from Parsoid's linter
rELINT80120b0b1fbd: Handle multi-part-template-block output from Parsoid's linter
T163092: Linter does not handle "Multi-part-template" well
T163045: Linter extension indicating a "missing end" or/stripped tag when no such condition exists.
T163003: List exact point where Linter found the malformation.
Mentioned Here: rGPARa182c227f09d: T141226: Fix logic when unpacking dom fragments
T37247: content-holding <div> should only contain the page text
T141226: Missing data-mw content in wikitext leads to html2wt exceptions
T153107: Parsoid is generating [[Foo|Foo]] instead of [[Foo]] for some VE edits
T161151: Parsoid should resolve template paths before providing them to Linter
T163091: Parsoid: Add API endpoint to get lint errors for arbitrary wikitext
T163549: Only lint pages that have wikitext contentmodel
T164006: Suggestion: API for fetching lint errors for a specific revision
T164792: Add class mw-parser-output to Parsoid's output

Event Timeline

ssastry created this task.Apr 13 2017, 4:46 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 13 2017, 4:46 PM

ssastry added projects: Parsoid, MediaWiki-extensions-Linter.Apr 13 2017, 4:47 PM

ssastry removed subscribers: Parsoid, MediaWiki-extensions-Linter.

ssastry triaged this task as High priority.Apr 14 2017, 6:23 PM

ssastry moved this task from Needs Triage to Next Up on the Parsoid board.

ssastry mentioned this in T163003: List exact point where Linter found the malformation..Apr 14 2017, 9:37 PM

ssastry merged a task: T163003: List exact point where Linter found the malformation..

ssastry added a subscriber: ShakespeareFan00.

ssastry mentioned this in T163045: Linter extension indicating a "missing end" or/stripped tag when no such condition exists..Apr 16 2017, 1:59 PM

ssastry mentioned this in T163092: Linter does not handle "Multi-part-template" well.Apr 17 2017, 3:51 AM

Legoktm merged a task: T163092: Linter does not handle "Multi-part-template" well.Apr 17 2017, 4:21 AM

Legoktm subscribed.

ssastry moved this task from Next Up to Non-Parsing-Team Tasks on the Parsoid board.Apr 17 2017, 7:06 PM

Change 352615 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Linter: Emit a diagnostic message for multi template content blocks

https://gerrit.wikimedia.org/r/352615

gerritbot added a project: Patch-For-Review.May 8 2017, 4:47 PM

Change 352714 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/extensions/Linter@master] Handle multi-part-template-block output from Parsoid's linter

https://gerrit.wikimedia.org/r/352714

ssastry mentioned this in rELINT80120b0b1fbd: Handle multi-part-template-block output from Parsoid's linter.May 8 2017, 9:52 PM

ssastry mentioned this in rELINT3eea8ed824e6: Handle multi-part-template-block output from Parsoid's linter.May 9 2017, 4:03 AM

Change 352714 merged by jenkins-bot:
[mediawiki/extensions/Linter@master] Handle multi-part-template-block output from Parsoid's linter

https://gerrit.wikimedia.org/r/352714

ReleaseTaggerBot added a project: MW-1.30-release-notes (WMF-deploy-2017-05-23_(1.30.0-wmf.2)).May 10 2017, 3:01 PM

Change 352615 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Linter: Add a flag to templateInfo for multi-part templated blocks

https://gerrit.wikimedia.org/r/352615