• 
      

    Conditionally return total_results from list resources.

    Review Request #13274 — Created Sept. 15, 2023 and submitted

    Information

    RBTools
    release-4.x

    Reviewers

    We have some resources that behave like lists but don't contain a
    total_results field. When we access this, we crash, because we assume
    this will always be present.

    This change makes that field optional, setting it to None if not
    found.

    Unit tests pass.

    Tested this with the watched review request resource, which exhibits
    this failure.

    Summary ID
    Conditionally return total_results from list resources.
    We have some resources that behave like lists but don't contain a `total_results` field. When we access this, we crash, because we assume this will always be present. This change makes that field optional, setting it to `None` if not found.
    562e55f3f6f0a490dc12b4752f70cd8addc61bc2
    david
    1. Ship It!
    2. 
        
    maubin
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-4.x (739844a)