Bsitu has uploaded a new change for review.
https://gerrit.wikimedia.org/r/86738
Change subject: Typo fix + conditional fix
......................................................................
Typo fix + conditional fix
bundleTitle['message'] defaults to empty string, it would never
be null, should just check if the string is empty
Change-Id: I6b9dd8b8a94fed114c0799004848daed472daf3a
---
M formatters/BasicFormatter.php
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo
refs/changes/38/86738/1
diff --git a/formatters/BasicFormatter.php b/formatters/BasicFormatter.php
index e3c226c..e4ee214 100644
--- a/formatters/BasicFormatter.php
+++ b/formatters/BasicFormatter.php
@@ -134,7 +134,7 @@
}
/**
- * Apply some custom change beforing formatting, child class
overwriting this method
+ * Apply some custom change before formatting, child class overwriting
this method
* should always invoke a call to the parent method unless child class
wants to overwrite
* the default completely
*
@@ -145,7 +145,7 @@
protected function applyChangeBeforeFormatting( EchoEvent $event, User
$user, $type ) {
// Use the bundle message if use-bundle is true and there is a
bundle message
$this->generateBundleData( $event, $user, $type );
- if ( $this->bundleData['use-bundle'] && isset(
$this->bundleTitle['message'] ) ) {
+ if ( $this->bundleData['use-bundle'] &&
$this->bundleTitle['message'] ) {
$this->title = $this->flyoutTitle = $this->bundleTitle;
}
}
--
To view, visit https://gerrit.wikimedia.org/r/86738
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b9dd8b8a94fed114c0799004848daed472daf3a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
