Katie Horn has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/167985

Change subject: Modify fredge QC to explicitly use UTC dates
......................................................................

Modify fredge QC to explicitly use UTC dates

Take advantage of the fact that we already solved this problem
in wmf_common. No need to add a new explicit dependency, as q2c
eventually depends on that module already.

Change-Id: I672c009c11eb2d672d9e9c498e89d1480cd5c7de
---
M sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm 
refs/changes/85/167985/1

diff --git a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module 
b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
index ed72fe3..3b75921 100644
--- a/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
+++ b/sites/all/modules/queue2civicrm/fredge/wmf_fredge_qc.module
@@ -139,7 +139,7 @@
        * Which is okay for now: We force IPv4 on payments.
        * @TODO eventually: Actually handle IPv6 here.
        */
-      
+
 //    $body['user_ip'] = inet_pton($body['user_ip']);
 
       watchdog('fredge', "Weird. Somehow an ipv6 address got through on 
payments. Caught in antifraud consumer. $txnid");
@@ -147,7 +147,7 @@
     } else {
       $body['user_ip'] = ip2long($body['user_ip']);
     }
-    
+
   }
 
   $id = fredge_insert_data($body, 'payments_fraud', $txnid, true);
@@ -209,7 +209,7 @@
     } else {
       //the field exists. Woot.
       if (array_key_exists('mysql_type', $definition) && 
$definition['mysql_type'] === 'DATETIME') {
-        $msg[$field] = date('Y-m-d H:i:s', $msg[$field]);
+        $msg[$field] = wmf_common_date_unix_to_sql($msg[$field]);
       }
       $insert[$field] = $msg[$field];
     }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I672c009c11eb2d672d9e9c498e89d1480cd5c7de
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Katie Horn <[email protected]>

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

Reply via email to