For eg. Wikidata items, it doesn't make sense to show this link because items don't have wikitext. Also it doesn't make sense to show the link if the user cannot edit the page.
This can be checked using [[ https://phabricator.wikimedia.org/source/mediawiki/browse/master/includes/Title.php$2165 | Title::quickUserCan ]] and another call.
The relevant method is [[ https://phabricator.wikimedia.org/diffusion/EPMI/browse/master/includes/Hooks.php$25 | Hooks::onBaseTemplateToolbox ]].
This is addressed in the patch for T167722: ParserMigration fails on pages with a content model that is not supported by EditPage.php.
Right. Except for the quickUserCan part. So that patch can be amended or another patch can be created on top of that one.
That is a good point. Actually, I could adjust the patch not to include the change to Hooks.php, to allow the proposed patch to handle that as well. It would make the existing patch a little more focussed too. Does that sound good?
A note that popped up during the review of 358425: ParserMigration aims to migrate Wiki text content formats, so not just the Wiki text content model. Any check should ensure that it does check against the content format, as extensions such as ProofreadPage use the same content format, but a different content model, and benefit from this extension.
I'd like to work on this. I'll start by setting up the environment and investigating the code
Change #1233790 had a related patch set uploaded (by Suraj Seth; author: Suraj Seth):
[mediawiki/extensions/EditSubpages@master] Hide EditSubpages link for non-editable or non-wikitext pages (T205798)
Change #1233790 abandoned by Suraj Seth:
[mediawiki/extensions/EditSubpages@master] Hide EditSubpages link for non-editable or non-wikitext pages (T205798)