• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-1.0.x (1418b89)