jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/365981 )
Change subject: Emoticons: Fixed "colored 0 error"
......................................................................
Emoticons: Fixed "colored 0 error"
Any colored 0 (<span style="color:red;">0</span>) would be removed from
the content. The reason is an misused empty function. PHP empty is true
on 0, so the content was replaced with an empty string. Now checking for
str_len instead.
ERM#6926
Change-Id: Iccb6f7ed107a3a52f35abe05ba2803f5fbd7c554
---
M Emoticons/Emoticons.class.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Pwirth: Looks good to me, approved
jenkins-bot: Verified
diff --git a/Emoticons/Emoticons.class.php b/Emoticons/Emoticons.class.php
index d56880d..701b14c 100644
--- a/Emoticons/Emoticons.class.php
+++ b/Emoticons/Emoticons.class.php
@@ -130,7 +130,7 @@
}
$fCallable = function( $aMatches ) use( $aMapping ) {
- return empty( $aMatches[0] ) ? '' : str_replace(
+ return strlen( $aMatches[0] ) === 0 ? '' : str_replace(
$aMapping['emoticons'],
$aMapping['replacements'],
$aMatches[0]
--
To view, visit https://gerrit.wikimedia.org/r/365981
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iccb6f7ed107a3a52f35abe05ba2803f5fbd7c554
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: master
Gerrit-Owner: Mglaser <[email protected]>
Gerrit-Reviewer: Ljonka <[email protected]>
Gerrit-Reviewer: Pwirth <[email protected]>
Gerrit-Reviewer: Robert Vogel <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
