Use the fallback avatar service if the default service is None.

Review Request #10516 — Created April 4, 2019 and submitted

Information

Djblets
release-1.0.x

Reviewers

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 not None, 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 is None, and that instead we get the fallback
service.

Summary ID
Use the fallback avatar service if the default service is None.
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 not `None`, and always including the fallback service.
c2ca7f0b47063411ce0617b5b840fd6471bcc94e
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-1.0.x (1418b89)
Loading...