On Aug. 26, 2015, 2:09 p.m., Stephen Gallagher wrote:
> src/rolekit/server/io/rolesettings.py, lines 68-69
>
<
http://reviewboard-fedoraserver.rhcloud.com/r/196/diff/1/?file=874#file87...
>
> Similarly, isn't it more pythonic for this to be:
>
> ```
> try:
> cb = self._callbacks["changed"]
> except KeyError:
> # Log the lack of a changed callback
> return
> cb_args = [ key, value ]
> ...
> ```
This will never fail, because self._callbacks = { "changed": None } is set in
__init__. cb will be None if no handler is set.
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/196/#review481
-----------------------------------------------------------
On Aug. 26, 2015, 1:21 p.m., Thomas Woerner wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/196/
-----------------------------------------------------------
(Updated Aug. 26, 2015, 1:21 p.m.)
Review request for RoleKit Mailing List, Miloslav Trmac, Nils Philippsen, Stephen
Gallagher, and Thomas Woerner.
Repository: rolekit
Description
-------
Only the changed signal is supported right now. As soon as a settings is
modified, the connected handler will be called for this setting.
Diffs
-----
src/rolekit/server/io/rolesettings.py 18f4871faf87ba23d21a3f9bc2c13b14463a8ccf
Diff:
http://reviewboard-fedoraserver.rhcloud.com/r/196/diff/
Testing
-------
Thanks,
Thomas Woerner