jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/391888 )

Change subject: Introduce the UserIdentity interface.
......................................................................


Introduce the UserIdentity interface.

This provides a narrow view on a user identity, providing access to
ID and name.

This has been extracted from I140f43a6fb443b for re-use with Actors,
on Anomie's request.

Change-Id: Ief00db5ce382537c5bf992159eae6baf096ae4be
---
M autoload.php
M includes/user/User.php
A includes/user/UserIdentity.php
3 files changed, 54 insertions(+), 1 deletion(-)

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



diff --git a/autoload.php b/autoload.php
index aef7e5e..5005534 100644
--- a/autoload.php
+++ b/autoload.php
@@ -957,6 +957,7 @@
        'MediaWiki\\Tidy\\RemexDriver' => __DIR__ . 
'/includes/tidy/RemexDriver.php',
        'MediaWiki\\Tidy\\RemexMungerData' => __DIR__ . 
'/includes/tidy/RemexMungerData.php',
        'MediaWiki\\Tidy\\TidyDriverBase' => __DIR__ . 
'/includes/tidy/TidyDriverBase.php',
+       'MediaWiki\\User\\UserIdentity' => __DIR__ . 
'/includes/user/UserIdentity.php',
        'MediaWiki\\Widget\\ComplexNamespaceInputWidget' => __DIR__ . 
'/includes/widget/ComplexNamespaceInputWidget.php',
        'MediaWiki\\Widget\\ComplexTitleInputWidget' => __DIR__ . 
'/includes/widget/ComplexTitleInputWidget.php',
        'MediaWiki\\Widget\\DateInputWidget' => __DIR__ . 
'/includes/widget/DateInputWidget.php',
diff --git a/includes/user/User.php b/includes/user/User.php
index 854ebbd..37a80f2 100644
--- a/includes/user/User.php
+++ b/includes/user/User.php
@@ -27,6 +27,7 @@
 use MediaWiki\Auth\AuthManager;
 use MediaWiki\Auth\AuthenticationResponse;
 use MediaWiki\Auth\AuthenticationRequest;
+use MediaWiki\User\UserIdentity;
 use Wikimedia\ScopedCallback;
 use Wikimedia\Rdbms\Database;
 use Wikimedia\Rdbms\DBExpectedError;
@@ -48,7 +49,7 @@
  * for rendering normal pages are set in the cookie to minimize use
  * of the database.
  */
-class User implements IDBAccessObject {
+class User implements IDBAccessObject, UserIdentity {
        /**
         * @const int Number of characters in user_token field.
         */
diff --git a/includes/user/UserIdentity.php b/includes/user/UserIdentity.php
new file mode 100644
index 0000000..57a0408
--- /dev/null
+++ b/includes/user/UserIdentity.php
@@ -0,0 +1,51 @@
+<?php
+/**
+ * Interface for objects representing user identity.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
+ * @file
+ */
+
+namespace MediaWiki\User;
+
+/**
+ * Interface for objects representing user identity.
+ *
+ * This represents the identity of a user in the context of page revisions and 
log entries.
+ *
+ * @since 1.31
+ */
+interface UserIdentity {
+
+       /**
+        * @since 1.31
+        *
+        * @return int The user ID. May be 0 for anonymous users or for users 
with no local account.
+        */
+       public function getId();
+
+       /**
+        * @since 1.31
+        *
+        * @return string The user's logical name. May be an IPv4 or IPv6 
address for anonymous users.
+        */
+       public function getName();
+
+       // TODO: in the future, we should also provide access to the actor ID 
here.
+       // TODO: we may want to (optionally?) provide a global ID, see 
CentralIdLookup.
+
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ief00db5ce382537c5bf992159eae6baf096ae4be
Gerrit-PatchSet: 6
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Anomie <[email protected]>
Gerrit-Reviewer: Brion VIBBER <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to