MaxSem has submitted this change and it was merged. Change subject: Remove the throw from Message::extractParam() ......................................................................
Remove the throw from Message::extractParam() Per Tim Starling's comment on bug 41400, I replaced the throw (when a message parameter is invalid) with a call to trigger_error(). The string [INVALID] is used as a placeholder to make it clear from the output that an error has occurred. Change-Id: I974d55d44d392c956e7de6d243da9d8dc07d8833 --- M includes/Message.php 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: MaxSem: Verified; Looks good to me, approved jenkins-bot: Looks good to me, but someone else must approve -- To view, visit https://gerrit.wikimedia.org/r/38822 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I974d55d44d392c956e7de6d243da9d8dc07d8833 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: wmf/1.21wmf5 Gerrit-Owner: MaxSem <[email protected]> Gerrit-Reviewer: MaxSem <[email protected]> Gerrit-Reviewer: PleaseStand <[email protected]> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
