openstack/deb-python-taskflow
Code Issues Proposed changes

Use a name property setter instead of a set_name method

Browse Source 
Change-Id: I4a41204e16e636e12d649592ae909d45fdb33f9f
This commit is contained in:
Joshua Harlow
2014-05-28 14:05:07 -07:00
parent ccd26c58fa
commit d099f82de7
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
taskflow/flow.py
View File

@@ -46,7 +46,7 @@ class Flow(object):
# NOTE(akarpinska): if retry doesn't have a name, # NOTE(akarpinska): if retry doesn't have a name,
# the name of its owner will be assigned # the name of its owner will be assigned
if self._retry and self._retry.name is None: if self._retry and self._retry.name is None:
self._retry.set_name(self.name + "_retry") self._retry.name = self.name + "_retry"
@property @property
def name(self): def name(self):

3
taskflow/retry.py
View File

@@ -54,7 +54,8 @@ class Retry(atom.Atom):
def name(self): def name(self):
return self._name return self._name
def set_name(self, name): @name.setter
def name(self, name):
self._name = name self._name = name
@abc.abstractmethod @abc.abstractmethod