Page MenuHomePhabricator

Prevent use of the "quality" constraint in the Badge search API
ClosedPublic

Authored by epriestley on Mar 9 2017, 7:43 PM.
Tags
None
Referenced Files
F18322705: D17487.diff
Sun, Aug 24, 5:56 PM
F18094449: D17487.diff
Thu, Aug 7, 10:08 PM
F18083238: D17487.id42053.diff
Tue, Aug 5, 9:25 AM
F18056369: D17487.id.diff
Mon, Aug 4, 8:18 AM
F18050398: D17487.diff
Sun, Aug 3, 7:33 PM
F17835803: D17487.id.diff
Sat, Jul 26, 3:38 PM
F17826786: D17487.id42053.diff
Sat, Jul 26, 7:37 AM
F17808807: D17487.diff
Jul 25 2025, 4:18 PM
Subscribers
None

Details

Summary

Ref T12270. This just drops the constraint for now, rather than dealing with all the typecasting stuff and putting us in a position which will almost certainly require backward compatibility breaks in the future.

Also renames "badges.*" to "badge.*" for consistency (all other methods are singular: token.*, project.*, differential.revision.*, etc).

Test Plan

Saw "qualities" now "Not Supported", while other constraints continue to work:

Screen Shot 2017-03-09 at 11.42.36 AM.png (70×535 px, 13 KB)

Diff Detail

Repository
rP Phabricator
Branch
badge1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15959
Build 21143: Run Core Tests
Build 21142: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 9 2017, 7:51 PM
This revision was automatically updated to reflect the committed changes.