From 70e248145c2313d4495cb786827b6a7b857c9cb3 Mon Sep 17 00:00:00 2001 From: umherirrender Date: Sat, 30 May 2015 19:38:04 +0200 Subject: [PATCH] list=categorymembers: Do not log feature usage when using default value list=categorymembers&cmstartsortkey was logged even the default value was set Bug: T100878 Change-Id: Idc52fe0c208216bc3e475d0e4b2e398cb73a6b80 --- includes/api/ApiQueryCategoryMembers.php | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/includes/api/ApiQueryCategoryMembers.php b/includes/api/ApiQueryCategoryMembers.php index ec0c1d146b..4042bd74bd 100644 --- a/includes/api/ApiQueryCategoryMembers.php +++ b/includes/api/ApiQueryCategoryMembers.php @@ -158,7 +158,9 @@ class ApiQueryCategoryMembers extends ApiQueryGeneratorBase { } $startsortkey = pack( 'H*', $params['starthexsortkey'] ); } else { - $this->logFeatureUsage( 'list=categorymembers&cmstartsortkey' ); + if ( $params['startsortkey'] !== null ) { + $this->logFeatureUsage( 'list=categorymembers&cmstartsortkey' ); + } $startsortkey = $params['startsortkey']; } if ( $params['endsortkeyprefix'] !== null ) { @@ -169,7 +171,9 @@ class ApiQueryCategoryMembers extends ApiQueryGeneratorBase { } $endsortkey = pack( 'H*', $params['endhexsortkey'] ); } else { - $this->logFeatureUsage( 'list=categorymembers&cmendsortkey' ); + if ( $params['endsortkey'] !== null ) { + $this->logFeatureUsage( 'list=categorymembers&cmendsortkey' ); + } $endsortkey = $params['endsortkey']; } -- 2.20.1