C. Scott Ananian has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/403317 )
Change subject: Don't crash RecordLintJob if the lint info doesn't contain
`params`
......................................................................
Don't crash RecordLintJob if the lint info doesn't contain `params`
We will crash trying to set `templateInfo` if it is present and `params`
is not.
On line 49 of RecordLintJob we're going to use `$errorInfo['params']`,
which will crash if `params` is not present.
Change-Id: I505c676cc0ccd8d54e44e65b04b10c2de03ee37c
---
M includes/ApiRecordLint.php
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Linter
refs/changes/17/403317/1
diff --git a/includes/ApiRecordLint.php b/includes/ApiRecordLint.php
index 2e73282..c7e0ed4 100644
--- a/includes/ApiRecordLint.php
+++ b/includes/ApiRecordLint.php
@@ -74,6 +74,9 @@
continue;
}
$info['location'] = array_slice( $info['dsr'], 0, 2 );
+ if ( !isset( $info['params'] ) ) {
+ $info['params'] = [];
+ }
if ( isset( $info['templateInfo'] ) &&
$info['templateInfo'] ) {
$info['params']['templateInfo'] =
$info['templateInfo'];
}
--
To view, visit https://gerrit.wikimedia.org/r/403317
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I505c676cc0ccd8d54e44e65b04b10c2de03ee37c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Linter
Gerrit-Branch: master
Gerrit-Owner: C. Scott Ananian <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
