Page MenuHomePhabricator

UnusedUseStatementSniff should not handle @see with url
Open, Needs TriagePublic

Description

The UnusedUseStatementSniff is processing @see annoations as this can contains classes. But there can also linked to a url and in that case there should be no search for classes to recognized them as used

https://www.doxygen.nl/manual/commands.html#cmdsa

The @see in this code avoids that the use Hooks is reported

...
use Hooks;
...

class WikimediaEventsHooks {
...
	/**
...
	 * @see https://www.mediawiki.org/wiki/Manual:Hooks/PageSaveComplete
	 */
	public static function onPageSaveComplete(
...

Event Timeline

Change 916809 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikimediaEvents@master] Removed unused Hooks class import

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

Change 916809 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] Removed unused Hooks class import

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