Page MenuHomePhabricator

Remove "Contains Words" constraint from Maniphest
ClosedPublic

Authored by epriestley on Sep 11 2017, 11:21 PM.
Tags
None
Referenced Files
F18387035: D18586.id44637.diff
Fri, Aug 29, 2:47 AM
F18348321: D18586.diff
Tue, Aug 26, 4:33 PM
F18253120: D18586.diff
Thu, Aug 21, 9:14 PM
F18097096: D18586.id44637.diff
Fri, Aug 8, 11:07 AM
F18090199: D18586.id.diff
Wed, Aug 6, 2:47 PM
F18082702: D18586.diff
Tue, Aug 5, 7:08 AM
F18081987: D18586.diff
Tue, Aug 5, 3:23 AM
F18032207: D18586.diff
Sun, Aug 3, 12:18 AM
Subscribers
None

Details

Summary

Ref T12819. Obsoleted by the Ferret engine, which is unprototyping shortly.

This breaks compatibility in two ways:

  • maniphest.query no longer supports "fullText" (now throws an explicit exception).
  • Existing saved searches with a "Contains Words" constraint will no longer have that constraint.

It seems unlikely (?) that either of these are seeing too much use, and they should be easy to fix. I'll note them in the changelog.

Test Plan

Viewed Maniphest, no more "Contains Words" field. Called maniphest.query with "fullText", got explicit exception.

Diff Detail

Repository
rP Phabricator
Branch
ferret1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18414
Build 24793: Run Core Tests
Build 24792: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Sep 12 2017, 12:13 AM
This revision was automatically updated to reflect the committed changes.