01tonythomas has uploaded a new change for review.

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

Change subject: Rename br_user column to br_user_email and update comments
......................................................................

Rename br_user column to br_user_email and update comments

column name br_user_email is more apt and is consistent with the
'user' table 'user_email' column.

Bug: 72683
Change-Id: I181a1ad2356e43681b3dbc2abefdd2da2029fe2a
---
M includes/BounceHandlerActions.php
M includes/ProcessBounceEmails.php
M sql/bounce_records.sql
3 files changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BounceHandler 
refs/changes/01/170101/1

diff --git a/includes/BounceHandlerActions.php 
b/includes/BounceHandlerActions.php
index d718658..ffc3c1d 100644
--- a/includes/BounceHandlerActions.php
+++ b/includes/BounceHandlerActions.php
@@ -51,7 +51,7 @@
                $res = $dbr->selectRow( 'bounce_records',
                        array( 'COUNT(*) as total_count' ),
                        array(
-                               'br_user'=> $originalEmail,
+                               'br_user_email'=> $originalEmail,
                                'br_timestamp' >= $bounceValidPeriod
                        ),
                        __METHOD__
diff --git a/includes/ProcessBounceEmails.php b/includes/ProcessBounceEmails.php
index 119d19c..58538e7 100644
--- a/includes/ProcessBounceEmails.php
+++ b/includes/ProcessBounceEmails.php
@@ -62,7 +62,7 @@
                        $dbw = wfGetDB( DB_MASTER, array(), $wikiId );
 
                        $rowData = array(
-                               'br_user' => $originalEmail,
+                               'br_user_email' => $originalEmail,
                                'br_timestamp' => $bounceTimestamp,
                                'br_reason' => $subject
                        );
diff --git a/sql/bounce_records.sql b/sql/bounce_records.sql
index 190db98..ca2e1d8 100644
--- a/sql/bounce_records.sql
+++ b/sql/bounce_records.sql
@@ -4,7 +4,7 @@
 
 CREATE TABLE /*_*/bounce_records (
        br_id           INT unsigned        NOT NULL PRIMARY KEY auto_increment,
-       br_user         VARCHAR(255)    NOT NULL, -- Email Id of failing 
recipient
+       br_user_email           VARCHAR(255)    NOT NULL, -- Email address of 
the failing recipient
        br_timestamp    varbinary(14)   NOT NULL,
        br_reason       VARCHAR(255)    NOT NULL  -- Failure reasons
 )/*$wgDBTableOptions*/;
\ No newline at end of file

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I181a1ad2356e43681b3dbc2abefdd2da2029fe2a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BounceHandler
Gerrit-Branch: master
Gerrit-Owner: 01tonythomas <[email protected]>

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

Reply via email to