[MediaWiki-commits] [Gerrit] wikimedia...crm[master]: Revert "Log a stack trace for all dedupe queries"

2017-10-26 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/386706 )

Change subject: Revert "Log a stack trace for all dedupe queries"
..


Revert "Log a stack trace for all dedupe queries"

We don't need it any more, and it's making it impossible to
read test failures, plus cluttering up the production logs.

This reverts commit 1e33d780795a76dc8ce0c2749086cc0586f7b438.

Change-Id: I2b49c7845664ad662506d4448319dfb6878313bb
---
M sites/all/modules/wmf_civicrm/wmf_civicrm.module
1 file changed, 0 insertions(+), 12 deletions(-)

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



diff --git a/sites/all/modules/wmf_civicrm/wmf_civicrm.module 
b/sites/all/modules/wmf_civicrm/wmf_civicrm.module
index 90ca939..caebbc6 100644
--- a/sites/all/modules/wmf_civicrm/wmf_civicrm.module
+++ b/sites/all/modules/wmf_civicrm/wmf_civicrm.module
@@ -3994,18 +3994,6 @@
   }
 }
 
-function wmf_civicrm_civicrm_dupeQuery($ruleGroup, $queryType, &$queries) {
-  $message = "Dedupe queries generated.  Query type $queryType.  ";
-  if ( $ruleGroup instanceof CRM_Dedupe_DAO_RuleGroup ) {
-$message .= "Rule group: {$ruleGroup->name}.  ";
-  }
-  $message .= 'Queries : ' . print_r( $queries, true ) . '.  ';
-  $message .= 'Stack trace: ' . CRM_Core_Error::formatBacktrace( 
debug_backtrace() );
-
-  watchdog( 'wmf_civicrm', $message, array(), WATCHDOG_INFO );
-  return TRUE;
-}
-
 /**
  * Abuse the permissions hook to prevent de-duping without a limit
  *

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2b49c7845664ad662506d4448319dfb6878313bb
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Ejegg 
Gerrit-Reviewer: XenoRyet 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] wikimedia...crm[master]: Revert "Log a stack trace for all dedupe queries"

2017-10-26 Thread Ejegg (Code Review) 
Ejegg has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/386706 )

Change subject: Revert "Log a stack trace for all dedupe queries"
..

Revert "Log a stack trace for all dedupe queries"

We don't need it any more, and it's making it impossible to
read test failures, plus cluttering up the production logs.

This reverts commit 1e33d780795a76dc8ce0c2749086cc0586f7b438.

Change-Id: I2b49c7845664ad662506d4448319dfb6878313bb
---
M sites/all/modules/wmf_civicrm/wmf_civicrm.module
1 file changed, 0 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm 
refs/changes/06/386706/1

diff --git a/sites/all/modules/wmf_civicrm/wmf_civicrm.module 
b/sites/all/modules/wmf_civicrm/wmf_civicrm.module
index 90ca939..caebbc6 100644
--- a/sites/all/modules/wmf_civicrm/wmf_civicrm.module
+++ b/sites/all/modules/wmf_civicrm/wmf_civicrm.module
@@ -3994,18 +3994,6 @@
   }
 }
 
-function wmf_civicrm_civicrm_dupeQuery($ruleGroup, $queryType, &$queries) {
-  $message = "Dedupe queries generated.  Query type $queryType.  ";
-  if ( $ruleGroup instanceof CRM_Dedupe_DAO_RuleGroup ) {
-$message .= "Rule group: {$ruleGroup->name}.  ";
-  }
-  $message .= 'Queries : ' . print_r( $queries, true ) . '.  ';
-  $message .= 'Stack trace: ' . CRM_Core_Error::formatBacktrace( 
debug_backtrace() );
-
-  watchdog( 'wmf_civicrm', $message, array(), WATCHDOG_INFO );
-  return TRUE;
-}
-
 /**
  * Abuse the permissions hook to prevent de-duping without a limit
  *

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2b49c7845664ad662506d4448319dfb6878313bb
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Ejegg 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits