HomePhabricator

Split AdvancedOptionsBuilder
18e15a579584Unpublished

Authored by gabriel-wmde on Apr 6 2018, 2:53 PM.

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Split AdvancedOptionsBuilder

Split configuration and widget building into two classes and uncouple
the configuration from the SearchModel.

Clean up dependencies - init does not need the UI classes for search
fields, those are best placed in AdvancedOptionsConfig where they are
used.

This refactoring is in preparation for adding field defaults to the
options which will be used in SearchModel.

Bug: T190681
Change-Id: I3e57bd2225cfe3e76d1c6df73360b9ba2ecfb67b

Details

Committed
gabriel-wmdeApr 9 2018, 2:31 PM
ChangeId
I3e57bd2225cfe3e76d1c6df73360b9ba2ecfb67b

Commit No Longer Exists

This commit no longer exists in the repository.