Merge lp:~mgorven/ibid/config-set-unset-501938 into lp:~ibid-core/ibid/old-trunk-1.6

Proposed by Michael Gorven
Status: Merged
Approved by: Jonathan Hitchcock
Approved revision: not available
Merged at revision: 822
Proposed branch: lp:~mgorven/ibid/config-set-unset-501938
Merge into: lp:~ibid-core/ibid/old-trunk-1.6
Diff against target: 12 lines (+1/-1)
1 file modified
ibid/plugins/config.py (+1/-1)
To merge this branch: bzr merge lp:~mgorven/ibid/config-set-unset-501938
Reviewer Review Type Date Requested Status
Jonathan Hitchcock Approve
Stefano Rivera Approve
Review via email: mp+16700@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Stefano Rivera (stefanor) wrote :

worksforme

review: Approve
Revision history for this message
Jonathan Hitchcock (vhata) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ibid/plugins/config.py'
2--- ibid/plugins/config.py 2009-10-20 15:45:46 +0000
3+++ ibid/plugins/config.py 2009-12-31 17:40:27 +0000
4@@ -44,7 +44,7 @@
5 config = config[part]
6
7 part = key.split('.')[-1]
8- if not isinstance(config, dict) or part not in config:
9+ if not isinstance(config, dict):
10 event.addresponse(u'No such option')
11 return
12 if ',' in value:

Subscribers

People subscribed via source and target branches