Use the fallback avatar service if the default service is None.
Review Request #10516 — Created April 4, 2019 and submitted — Latest diff uploaded
The introduction of the fallback avatar service was intended to always
provide a useful service if the user had opted out of any other
configured services, but it didn't catch the case where the user didn't
have an explicit service configured and the administrator had configured
the default to be no service.This change covers this case, only including the default as a viable
option if it's notNone
, and always including the fallback service.
Unit tests passed.
Verified manually that we no longer get errors with a
None
avatar
service if the default isNone
, and that instead we get the fallback
service.