Legoktm has uploaded a new change for review.
https://gerrit.wikimedia.org/r/315641
Change subject: Re-implement linter logging for a MediaWiki extension
......................................................................
Re-implement linter logging for a MediaWiki extension
This re-implements the linter logging for a MediaWiki extension that
creates an API module named 'record-lint', and takes the logs as a JSON
POST parameter.
Additionally, also send requests for changes that don't have any issues
so that the extension could remove fixed issues.
Change-Id: I45600afed59027479c37be99c39ca5b2ad50661d
---
M lib/logger/linter.js
1 file changed, 22 insertions(+), 17 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/parsoid
refs/changes/41/315641/1
diff --git a/lib/logger/linter.js b/lib/logger/linter.js
index bb584ec..2a3c960 100644
--- a/lib/logger/linter.js
+++ b/lib/logger/linter.js
@@ -16,25 +16,30 @@
Linter.prototype.logLintOutput = function(logData, cb) {
try {
- if (this.buffer.length > 0) {
- if (!this._env.conf.parsoid.linterAPI) {
- console.log(this.buffer);
- } else {
- request.post(
- this._env.conf.parsoid.linterAPI,
- { json: this.buffer },
- function(error, response, body) {
- if (!error &&
response.statusCode === 200) {
- console.log(body);
- }
- }
- );
- }
- this.buffer = [];
- return;
+ if (!this._env.conf.parsoid.linterAPI) {
+ console.log(this.buffer);
} else {
- console.log("No Issues found");
+ request.post(
+ this._env.conf.parsoid.linterAPI,
+ { form: {
+ data: JSON.stringify(this.buffer),
+ page: this._env.page.name,
+ revision:
this._env.page.meta.revision.revid,
+ action: 'record-lint',
+ format: 'json',
+ formatversion: 2,
+ } },
+ function(error, response, body) {
+ if (!error && response.statusCode ===
200) {
+ console.log(body);
+ } else {
+ console.log(body);
+ }
+ }
+ );
}
+ this.buffer = [];
+ return;
} catch (e) {
console.error("Error in logLintOutput: " + e);
return;
--
To view, visit https://gerrit.wikimedia.org/r/315641
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I45600afed59027479c37be99c39ca5b2ad50661d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/parsoid
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
