update callback subscriptions of object methods to use decorator
Bug #1668262 reported by
Kevin Benton
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Fix Released
|
Low
|
Kevin Benton |
Bug Description
We can now use @registry.
This improves readability for two purposes:
1. Using the decorator right above the method makes it easy to see when looking at the code when the method is going to be called instead of having to search for the component that subscribes it.
2. We can stop overriding __new__ in class definitions just for registry subscriptions, which makes the class definitions a little cleaner.
See https:/
Changed in neutron: | |
status: | New → Confirmed |
importance: | Undecided → Low |
tags: | added: low-hanging-fruit |
Changed in neutron: | |
assignee: | nobody → Cedric Brandily (cbrandily) |
Changed in neutron: | |
assignee: | Cedric Brandily (cbrandily) → Kevin Benton (kevinbenton) |
description: | updated |
Changed in neutron: | |
assignee: | Cedric Brandily (cbrandily) → Ihar Hrachyshka (ihar-hrachyshka) |
Changed in neutron: | |
assignee: | Ihar Hrachyshka (ihar-hrachyshka) → Kevin Benton (kevinbenton) |
Changed in neutron: | |
status: | In Progress → Fix Released |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/438718
Review: https:/