Page MenuHomePhabricator

Replace usages of Linker::link() and Linker::linkKnown() in extension PageForms
Closed, ResolvedPublic

Description

Linker::link() and Linker::linkKnown() functions are deprecated as of MediaWiki 1.28, and the new LinkRenderer is preferred instead.
https://doc.wikimedia.org/mediawiki-core/master/php/classLinker.html

./PageForms/includes/PF_Hooks.php:		$text = Linker::linkKnown( $sp->getPageTitle(), $editMsg, [], $linkParams );

Event Timeline

hi @Yaron_Koren, I would like to work on it. Would you please tell me where can I find this repository :)

@Palak199: Hi, please see/click the code project "Tag" in the sidebar; then go to the project tag overview (first entry in the left sidebar).
Many project tag overview pages link to their source repository: https://phabricator.wikimedia.org/project/profile/272/ Thanks! :)

Change 677109 had a related patch set uploaded (by Palak199; author: Palak199):

[mediawiki/extensions/PageForms@master] Replace:: Linker::linkKnown with linkRenderer

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

Change 677109 abandoned by Yaron Koren:

[mediawiki/extensions/PageForms@master] Replace:: Linker::linkKnown with linkRenderer

Reason:

It's never a good idea to submit a patch with no research or testing.

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

Change 677109 abandoned by Yaron Koren:

My bad, @Yaron_Koren
I read this documentation
https://doc.wikimedia.org/mediawiki-core/master/php/classLinker.html
and found that information about use of linkKnown and link is given but only one line has been quoted about linkRenderer. Please guide me on where should I find about it.

This site is helpful in finding code examples, as well as underlying classes: https://codesearch.wmcloud.org/search/

Change 677109 restored by Yaron Koren:

[mediawiki/extensions/PageForms@master] Replace:: Linker::linkKnown with linkRenderer

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

This site is helpful in finding code examples, as well as underlying classes: https://codesearch.wmcloud.org/search/

Sure, I got it now! thankyou :)

Change 677109 merged by jenkins-bot:

[mediawiki/extensions/PageForms@master] Replace:: Linker::linkKnown with linkRenderer

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

Yaron_Koren claimed this task.