Merge lp:~dcaro/clicompanion/fix-909545 into lp:clicompanion

Proposed by David Caro
Status: Merged
Approved by: Marek Bardoński
Approved revision: 95
Merged at revision: 99
Proposed branch: lp:~dcaro/clicompanion/fix-909545
Merge into: lp:clicompanion
Diff against target: 14 lines (+4/-0)
1 file modified
clicompanionlib/controller.py (+4/-0)
To merge this branch: bzr merge lp:~dcaro/clicompanion/fix-909545
Reviewer Review Type Date Requested Status
Marek Bardoński Approve
Review via email: mp+87102@code.launchpad.net

Description of the change

Just added a little if when executting the command to avoid acessing an empty array.

To post a comment you must log in.
Revision history for this message
Marek Bardoński (bdfhjk) wrote :

Look good. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'clicompanion' (properties changed: -x to +x)
2=== modified file 'clicompanionlib/controller.py'
3--- clicompanionlib/controller.py 2011-11-18 23:56:17 +0000
4+++ clicompanionlib/controller.py 2011-12-29 19:10:28 +0000
5@@ -362,6 +362,10 @@
6 ## send the command to the terminal
7 def run_command(self, widget, notebook, liststore):
8
9+ ## if called without selecting a command from the list return
10+ if not view.ROW:
11+ return
12+
13 text = ""
14 row_int = int(view.ROW[0][0]) ## removes everything but number from [5,]
15

Subscribers

People subscribed via source and target branches