From: umherirrender <umherirrender_de.wp@web.de>
Date: Sat, 30 May 2015 17:38:04 +0000 (+0200)
Subject: list=categorymembers: Do not log feature usage when using default value
X-Git-Tag: 1.31.0-rc.0~11248^2
X-Git-Url: http://git.cyclocoop.org/%24self?a=commitdiff_plain;h=70e248145c2313d4495cb786827b6a7b857c9cb3;p=lhc%2Fweb%2Fwiklou.git

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
---

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'];
 				}