From bf206185aace074b8d06262b21a5e99fef890f25 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Mon, 9 Sep 2019 19:00:28 +0100 Subject: [PATCH] Remove spurious `@class` tags in two files This was only used in two PHP files in all of core. It is not needed (the block above the class keyword is naturally for that class) and only emits a warning from Doxygen: > PHPVersionCheck.php:35: warning: missing argument after \class. > SpecialPageData.php:28: warning: missing argument after \class. This is unlike JSDuck, where it is used to differentiate a function from a constructor/class. In Doxygen, `@class` is only used to create a virtual class within the docs that isn't explicitly in an indexed file, and thus requires a name. Bug: T232104 Change-Id: Ie504a8e2b72be8f370dc0c90da6a03f64d93df2d --- includes/PHPVersionCheck.php | 2 -- includes/specials/SpecialPageData.php | 1 - 2 files changed, 3 deletions(-) diff --git a/includes/PHPVersionCheck.php b/includes/PHPVersionCheck.php index b63a84d807..bf138c4b1e 100644 --- a/includes/PHPVersionCheck.php +++ b/includes/PHPVersionCheck.php @@ -31,8 +31,6 @@ * * @note This class uses setter methods instead of a constructor so that * it can be compatible with PHP 4, PHP 5 and PHP 7 (without warnings). - * - * @class */ class PHPVersionCheck { /* @var string The number of the MediaWiki version used. */ diff --git a/includes/specials/SpecialPageData.php b/includes/specials/SpecialPageData.php index 978efa7f12..df8bce147b 100644 --- a/includes/specials/SpecialPageData.php +++ b/includes/specials/SpecialPageData.php @@ -25,7 +25,6 @@ * The web server should generally be configured to make this accessible via a canonical URL/URI, * such as . * - * @class * @ingroup SpecialPage */ class SpecialPageData extends SpecialPage { -- 2.20.1