FileImporter constructs EditPage for a special page
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• Pchelolo
	Feb 9 2021, 4:49 PM

Description

FileImporter extension has this code:

https://github.com/wikimedia/mediawiki-extensions-FileImporter/blob/master/src/Html/WikitextEditor.php#L40-L52

it constructs an EditPage just to pass it into a EditPage::showEditForm:initial hook to generate the editing sidebar or something and so needs to construct Article, WikiPage, Title on a special page. This is now deprecated and will definitely break.

Details

	Subject	Repo	Branch	Lines +/-
	Fix wrong context title in WikitextEditor	mediawiki/extensions/FileImporter	master	+25 -20

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Declined		dchen	T118706 Conduct heuristic evaluation of image upload and insert flow in VisualEditor
Open		None	T115858 Design improvements for mw.ForeignStructuredUpload.BookletLayout
Open		None	T115865 Insert image in content immediately after it's uploaded, skipping the "General settings" step
Duplicate		None	T115864 Figure out if the description of the image can be used as the caption on-wiki
Open	Feature	None	T53032 When inserting an image, set its caption by default to be the Commons image description
Open	BUG REPORT	None	T12863 Links on commons upload summaries do not link to commons
Open		None	T5498 Image history is confusing
Stalled		None	T96384 Integrate file revisions with description page history
Open	Feature	None	T39534 Wikimedia Commons should support searching by color
Duplicate		None	T39535 Wikimedia Commons should support filtering by color
Resolved		None	T19503 Provide metadata support on Wikimedia Commons
Open		None	T122038 Moving/Deleting a video file does not move/delete subtitles files, nor does it inform the file mover of their existence
Open		None	T135221 Make TimedText content an integral part of the File page
Open	Feature	None	T2167 Use a dedicated interface for adding meta-data like interwiki links, rather than wikitext
Open		None	T132072 Integrate page meta-data as a new content model revision slot for consistency and atomicity
Resolved		None	T51662 VisualEditor: Use Multimedia/Wikidata's proposed rich structured meta-data in the image insertion dialog
Resolved		None	T68108 [Epic] Store media information for files on Wikimedia Commons as structured data
Open	Feature	None	T14963 allow per-page exceptions to spam blacklist
Open		None	T203157 Make the spam whitelist its own slot
Open	Feature	None	T56140 Move TemplateData to its own JSON-content namespace and associate with Template-namespace, or to its own TemplateData content model and revision slot
Stalled		None	T174043 Deploy Multi-Content Revisions
Open		None	T174022 Implement multi-content revisions
Open		daniel	T196087 Refactored implementation of MCR page update interface
Open		pmiazga	T302623 FY2022-2023: Improve Backend Pageview Timing
Resolved		None	T206498 MediaWiki needlessly queries page data twice on page views
Invalid		None	T273514 MW expedition work [high effort]
Open		None	T273542 Migrate ZObjectStore from WikiPage to PageStore when it exists
Resolved		None	T195069 Factor PageStore and PageRecord out of WikiPage
Resolved		daniel	T272421 Make WikiPage a PageRecord
Resolved		daniel	T272424 Make WikiPage a ProperPageIdentity
Resolved		daniel	T272518 Watch for extensions broken by WikiPage now deprecating construction on bad titles
Resolved		daniel	T274274 FileImporter constructs EditPage for a special page

Event Timeline

• Pchelolo created this task.Feb 9 2021, 4:49 PM

daniel triaged this task as High priority.Feb 9 2021, 5:43 PM

daniel moved this task from Unsorted pile to PageStore pile on the Platform Team Workboards (MW Expedition) board.

Change 663038 had a related patch set uploaded (by Daniel Kinzler; owner: Daniel Kinzler):
[mediawiki/extensions/FileImporter@master] Don't construct Article on a SpecialPage

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

gerritbot added a project: Patch-For-Review.Feb 9 2021, 9:00 PM

daniel moved this task from PageStore pile to Waiting for Review on the Platform Team Workboards (MW Expedition) board.Feb 9 2021, 9:07 PM

thiemowmde mentioned this in T274340: CodeMirror syntax highlighting doesn't work when importing a file.Feb 10 2021, 9:07 AM

Aklapper added a project: Move-Files-To-Commons.Feb 10 2021, 9:21 AM

daniel claimed this task.Feb 10 2021, 10:16 AM

Change 663038 merged by jenkins-bot:
[mediawiki/extensions/FileImporter@master] Fix wrong context title in WikitextEditor

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

thiemowmde mentioned this in rEFLI529f821d91d6: Fix wrong context title in WikitextEditor.Feb 10 2021, 5:00 PM

Maintenance_bot removed a project: Patch-For-Review.Feb 10 2021, 5:11 PM

• Pchelolo moved this task from Waiting for Review to On the train on the Platform Team Workboards (MW Expedition) board.Feb 22 2021, 8:07 PM

• Pchelolo closed this task as Resolved.Feb 23 2021, 9:01 PM