Page MenuHomePhabricator

Remove Ferret function aliases and overrides
ClosedPublic

Authored by epriestley on Apr 16 2020, 5:30 PM.
Tags
None
Referenced Files
F13993127: D21130.id50320.diff
Tue, Oct 22, 8:51 PM
F13977837: D21130.id50318.diff
Fri, Oct 18, 8:18 PM
F13963326: D21130.diff
Tue, Oct 15, 3:16 PM
Unknown Object (File)
Wed, Oct 9, 1:21 PM
Unknown Object (File)
Sep 9 2024, 2:38 PM
Unknown Object (File)
Sep 6 2024, 7:22 AM
Unknown Object (File)
Sep 3 2024, 2:02 PM
Unknown Object (File)
Sep 1 2024, 1:35 PM
Subscribers
None

Details

Summary

Ref T13511. Ferret functions currently define "aliases", and some applications override the default aliases.

This probably isn't really the right model, since it means the available function aliases in global search depend on the types of documents you're searching for. This isn't fundamentally unworkable but is kind of weird.

Regardless, these don't actually work. Searching for "description:x" is a syntax error.

Since they don't work, it's a good bet no one is relying on them. Just get rid of them until there's a clearer argument for the feature.

Test Plan

Grepped for "getFunctionMap", got no other hits. Ran some queries with the alias functions, got syntax errors.

Diff Detail

Repository
rP Phabricator
Branch
field1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24136
Build 33235: Run Core Tests
Build 33234: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Apr 16 2020, 8:40 PM
This revision was automatically updated to reflect the committed changes.