Nikerabbit has uploaded a new change for review.

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

Change subject: TranslateSandbox: also delete log entries for deleted users.
......................................................................

TranslateSandbox: also delete log entries for deleted users.

I could swear the log_user_text was displayed previously, but now
it clearly tries to load users which do not exist, and displays the
ip of the current user instead. A different kind of log entry is
needed if we want to log deleted users.

Bug: T142995
Change-Id: I611f7a26377d3809aaadee6dd7b0643d908031a6
---
M utils/TranslateSandbox.php
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Translate 
refs/changes/32/304832/1

diff --git a/utils/TranslateSandbox.php b/utils/TranslateSandbox.php
index b19ef64..a82b280 100644
--- a/utils/TranslateSandbox.php
+++ b/utils/TranslateSandbox.php
@@ -97,6 +97,7 @@
                $dbw->delete( 'user', array( 'user_id' => $uid ), __METHOD__ );
                $dbw->delete( 'user_groups', array( 'ug_user' => $uid ), 
__METHOD__ );
                $dbw->delete( 'user_properties', array( 'up_user' => $uid ), 
__METHOD__ );
+               $dbw->delete( 'logging', array( 'log_user' => $uid ), 
__METHOD__ );
 
                // If someone tries to access still object still, they will get 
anon user
                // data.

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I611f7a26377d3809aaadee6dd7b0643d908031a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <niklas.laxst...@gmail.com>

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

Reply via email to