Maniphest T235801

Rework ParamValidator to use MessageValue
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Anomie
	Oct 17 2019, 8:06 PM

Tags

Referenced Files

None

Subscribers

Description

It was a mistake trying to do ParamValidator without MessageValue. Now that MessageValue exists, let's use it.

This should make it easier to use in the Action API, and possibly the REST API as well.

Details

	Subject	Repo	Branch	Lines +/-
	ParamValidator: Use MessageValue!	mediawiki/core	master	+2 K -581

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Open		None	T235568 Implement Param Validation for Existence of Title and Access Rights
		Resolved		Anomie	T142080 HTTP API typechecking method is very confusing and intimidating
		Resolved		Anomie	T235801 Rework ParamValidator to use MessageValue

Event Timeline

Anomie created this task.Oct 17 2019, 8:06 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 17 2019, 8:06 PM

Anomie added parent tasks: T235568: Implement Param Validation for Existence of Title and Access Rights, T142080: HTTP API typechecking method is very confusing and intimidating.Oct 17 2019, 8:06 PM

Anomie moved this task from Inbox to Waiting for Review on the Platform Team Workboards (Clinic Duty Team) board.

Change 543952 had a related patch set uploaded (by Anomie; owner: Anomie):
[mediawiki/core@master] ParamValidator: Use MessageValue!

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

gerritbot added a project: Patch-For-Review.Oct 17 2019, 8:07 PM

Change 543952 merged by jenkins-bot:
[mediawiki/core@master] ParamValidator: Use MessageValue!

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

Maintenance_bot removed a project: Patch-For-Review.Nov 13 2019, 7:10 AM

Anomie closed this task as Resolved.Nov 13 2019, 8:47 PM

Is this ticket done?
Should it be in the done column?
Should it be unassigned?

Aklapper removed a subscriber: Anomie.Oct 16 2020, 5:38 PM