Page MenuHomePhabricator

Remove Ferret function aliases and overrides
ClosedPublic

Authored by epriestley on Apr 16 2020, 5:30 PM.
Tags
None
Referenced Files
F17965298: D21130.id50320.diff
Aug 1 2025, 11:30 AM
F17956866: D21130.id50318.diff
Aug 1 2025, 5:47 AM
F17945562: D21130.id.diff
Jul 31 2025, 12:33 PM
F17925771: D21130.diff
Jul 30 2025, 12:03 PM
F17712064: D21130.diff
Jul 17 2025, 9:23 AM
F17702702: D21130.diff
Jul 15 2025, 11:41 PM
Unknown Object (File)
Jun 19 2025, 11:02 AM
Unknown Object (File)
May 13 2025, 10:24 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
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.