• 
      

    Add a base extension hook for managing multiple items in a registry.

    Review Request #9283 — Created Oct. 18, 2017 and submitted — Latest diff uploaded

    Information

    Djblets
    release-0.10.x
    d33831d...

    Reviewers

    This introduces BaseRegistryMultiItemHook, which is similar to
    BaseRegistryHook but handles registering/unregistering a list of items
    at once. It's useful for extension hooks that otherwise need to manage
    multiple items, helping to consolidate that logic.

    Unlike existing hooks in Review Board that manage multiple items, this
    is also careful to unregister any items that were registered upon
    encountering an error, so that some items don't remain permanently
    registered even if the extension is shut down.

    Unit tests were added for this hook.

    Unit tests pass.

    Moved some extension hooks to this in Review Board and ran tests. Those
    also pass.