Page MenuHomePhabricator

Remove Ferret function aliases and overrides
ClosedPublic

Authored by epriestley on Apr 16 2020, 5:30 PM.
Tags
None
Referenced Files
F15418819: D21130.id50320.diff
Fri, Mar 21, 1:41 AM
F15385503: D21130.id50320.diff
Fri, Mar 14, 10:44 PM
F15341539: D21130.diff
Sun, Mar 9, 8:45 PM
F15291196: D21130.id.diff
Wed, Mar 5, 12:20 AM
Unknown Object (File)
Tue, Feb 25, 12:00 PM
Unknown Object (File)
Sat, Feb 22, 4:30 PM
Unknown Object (File)
Feb 13 2025, 12:35 AM
Unknown Object (File)
Feb 13 2025, 12:32 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.