Bsitu has uploaded a new change for review.

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


Change subject: header editing is broken for headers created by anon users
......................................................................

header editing is broken for headers created by anon users

Fatal error: Call to a member function getName() on a non-object
in /Flow/includes/Block/Topic.php on line 533

This is mainly because User::newFromName() would fail on IPs by default

Change-Id: I8df80b0a959cd7acfafa091ca89d98ad6cf952af
---
M includes/Block/Topic.php
M includes/Model/PostRevision.php
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/68/93868/1

diff --git a/includes/Block/Topic.php b/includes/Block/Topic.php
index c7b7f44..3ca66f4 100644
--- a/includes/Block/Topic.php
+++ b/includes/Block/Topic.php
@@ -530,7 +530,7 @@
                                '*' => $post->getContent( null, $contentFormat 
),
                                'format' => $contentFormat
                        );
-                       $output['user'] = $post->getCreator()->getName();
+                       $output['user'] = $post->getCreatorName();
                }
 
                if ( ! isset( $options['no-children'] ) ) {
diff --git a/includes/Model/PostRevision.php b/includes/Model/PostRevision.php
index 7772838..c436759 100644
--- a/includes/Model/PostRevision.php
+++ b/includes/Model/PostRevision.php
@@ -130,7 +130,7 @@
                        if ( $this->getCreatorIdRaw() !== 0 ) {
                                $user = User::newFromId( 
$this->getCreatorIdRaw() );
                        } else {
-                               $user = User::newFromName( 
$this->getCreatorNameRaw() );
+                               $user = User::newFromName( 
$this->getCreatorNameRaw(), false );
                        }
 
                        return $user;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8df80b0a959cd7acfafa091ca89d98ad6cf952af
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>

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

Reply via email to