Dzahn has submitted this change and it was merged.

Change subject: Weekly Bugzilla Report mail: Fix wrong SQL query on bug 
resolutions
......................................................................


Weekly Bugzilla Report mail: Fix wrong SQL query on bug resolutions

Remove accidential use of double quotes that made some queries
return "0" values.

This improves https://gerrit.wikimedia.org/r/#/c/62220/

Bug: 48072
Change-Id: Ic645a3c1e81ee094efbe8624f22b81b87e3d6902
---
M templates/misc/bugzilla_report.php
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/templates/misc/bugzilla_report.php 
b/templates/misc/bugzilla_report.php
index 1c09a07..b07e07d 100755
--- a/templates/misc/bugzilla_report.php
+++ b/templates/misc/bugzilla_report.php
@@ -90,9 +90,9 @@
 FROM
         bugs, bugs_activity
 WHERE
-        bugs.resolution = "$resolution"
+        bugs.resolution = $resolution
 AND
-        bugs_activity.added = "$resolution"
+        bugs_activity.added = $resolution
 AND
         bugs_activity.bug_when
 BETWEEN
@@ -114,9 +114,9 @@
 FROM
         bugs, bugs_activity
 WHERE
-        bugs.bug_status = "$state"
+        bugs.bug_status = $state
 AND
-        bugs_activity.added = "$state"
+        bugs_activity.added = $state
 AND
         bugs_activity.bug_when
 BETWEEN

-- 
To view, visit https://gerrit.wikimedia.org/r/63998
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic645a3c1e81ee094efbe8624f22b81b87e3d6902
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Aklapper <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to