On July 21, 2014, 6:33 p.m., Miloslav Trmac wrote:
> config/roles/testrole/role.py, lines 109-116
>
<
http://reviewboard-fedoraserver.rhcloud.com/r/17/diff/1/?file=91#file91li...
>
> Silently returns None on an unmatched property
Thomas Woerner wrote:
get_property is not needed if the setting is in _DEFAULTS, see
RoleBase.get_property:
if hasattr(x, "_settings") and prop in x._settings:
return x._settings[prop]
...
elif prop in x._DEFAULTS:
return x._DEFAULTS[prop]
Therefore settings in _settings and in _DEFAULTS are already taken care of. This
means get_property should not be needed in a Role.
Miloslav Trmac wrote:
The concern is about completely unknown setting names; the final do_get_property()
just returns None for them.
Dropped do_get_property from roles completely.
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/17/#review57
-----------------------------------------------------------
On July 22, 2014, 3:35 p.m., Thomas Woerner wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/17/
-----------------------------------------------------------
(Updated July 22, 2014, 3:35 p.m.)
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
Simplifies the role interface
Role cleanup by dropping "failonthis" setting
Diffs
-----
config/roles/testrole/role.py 2f077c62b4a8027e7783a2e08c84bc9c9715393e
src/rolekit/server/rolebase.py 50b5685a038789d02d3f3b0451f5edaecc187964
Diff:
http://reviewboard-fedoraserver.rhcloud.com/r/17/diff/
Testing
-------
Thanks,
Thomas Woerner