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
=== modified file 'clicompanion' (properties changed: -x to +x)
=== modified file 'clicompanionlib/controller.py'
--- clicompanionlib/controller.py 2011-11-18 23:56:17 +0000
+++ clicompanionlib/controller.py 2011-12-29 19:10:28 +0000
@@ -362,6 +362,10 @@
362 ## send the command to the terminal362 ## send the command to the terminal
363 def run_command(self, widget, notebook, liststore):363 def run_command(self, widget, notebook, liststore):
364364
365 ## if called without selecting a command from the list return
366 if not view.ROW:
367 return
368
365 text = ""369 text = ""
366 row_int = int(view.ROW[0][0]) ## removes everything but number from [5,]370 row_int = int(view.ROW[0][0]) ## removes everything but number from [5,]
367371

Subscribers

People subscribed via source and target branches