jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/350118 )
Change subject: Fix test error.
......................................................................
Fix test error.
This incorrect prefix was causing api fails in weird cases in the test suite.
They
were not causing test failures until I made some other changes but were uncool.
Note that I have a cunning plan to remove this hack altogether
Change-Id: I08bee8712cefe8e928950b72457fe1917fc7e7a7
---
M packages/DB/common.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Ejegg: Looks good to me, approved
diff --git a/packages/DB/common.php b/packages/DB/common.php
index abbee0f..fe9a4f3 100644
--- a/packages/DB/common.php
+++ b/packages/DB/common.php
@@ -1162,7 +1162,7 @@
else {
global $user;
if (empty($user)) {
- $prefix = 'phpunit-test';
+ $prefix = '';
}
else {
$prefix = "/* https://civicrm.wikimedia.org/user/{$user->uid}
*/ ";
--
To view, visit https://gerrit.wikimedia.org/r/350118
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I08bee8712cefe8e928950b72457fe1917fc7e7a7
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm/civicrm
Gerrit-Branch: master
Gerrit-Owner: Eileen <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
