Linter: Fix action=record-lint getAllowedParams()
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Legoktm
	Nov 22 2016, 3:28 AM

Description

From T148583#2813012:

The action=record-lint has the field types mixed up with their defaults (i.e. If you don't specify a page, it defaults to the page named "string"). They should also probably be marked mandatory (This obviously doesn't really matter since its internal only).

Oops.

Details

	Subject	Repo	Branch	Lines +/-
	API: Fix up action=record-lint's getAllowedParams()	mediawiki/extensions/Linter	master	+12 -3

Customize query in gerrit

Related Objects

Mentioned In: rELINT2f39ab7fff9f: API: Fix up action=record-lint's getAllowedParams()
rELINT9c9af5611034: API: Fix up action=record-lint's getAllowedParams()
T148583: Security review for Linter extension
Mentioned Here: T148583: Security review for Linter extension

Event Timeline

Legoktm created this task.Nov 22 2016, 3:28 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 22 2016, 3:28 AM

Legoktm mentioned this in T148583: Security review for Linter extension.Nov 22 2016, 3:33 AM

Change 322844 had a related patch set uploaded (by Legoktm):
API: Fix up action=record-lint's getAllowedParams()

https://gerrit.wikimedia.org/r/322844

gerritbot added a project: Patch-For-Review.Nov 22 2016, 9:28 AM

Legoktm mentioned this in rELINT9c9af5611034: API: Fix up action=record-lint's getAllowedParams().Nov 22 2016, 9:28 AM

Legoktm claimed this task.Nov 22 2016, 9:30 AM

Legoktm mentioned this in rELINT2f39ab7fff9f: API: Fix up action=record-lint's getAllowedParams().Nov 23 2016, 2:34 AM

Change 322844 merged by jenkins-bot:
API: Fix up action=record-lint's getAllowedParams()