config: Change ConfigException to extend LogicException
authorGergő Tisza <gtisza@wikimedia.org>
Fri, 16 Nov 2018 22:40:28 +0000 (14:40 -0800)
committerKrinkle <krinklemail@gmail.com>
Mon, 1 Jul 2019 20:07:33 +0000 (20:07 +0000)
commitb6a567cf491c10b338bcda9eaa5bac92ca37e491
tree215e25cc452ca1285f2c9bfe09a00f6c59046e21
parentb95a76ae9ecace5772989995a9a6d7e11fb1fdc6
config: Change ConfigException to extend LogicException

This exception is thrown when someone tries to fetch a
configuration entry with an invalid name, or override an already
set entry, both of which are errors in the logic and cannot be
meaningfully handled, so it does not make sense for this exception
to trigger IDE warnings.

(The one exception is the etcd load error, which probably shouldn't
be a ConfigException in the first place. But even that is not
something callers could meaningfully handle.)

Change-Id: I26b5b32f3348965f0cbf607cbb7cb5aaa509ed51
includes/config/ConfigException.php