Siebrand 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: Siebrand: Looks good to me, approved jenkins-bot: Verified -- To view, visit https://gerrit.wikimedia.org/r/36058 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: REL1_18 Gerrit-Owner: CSteipp <[email protected]> Gerrit-Reviewer: PleaseStand <[email protected]> Gerrit-Reviewer: Siebrand <[email protected]> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
