TypeError when adding a launcher and nothing is selected in the directory view

[My Karlsson]

Summary:

An error is emitted if a launcher is added when no row is selected in the directory tree view.

Steps to reproduce:

1. Deselect any selected entries in the directory tree view.
2. Click on the "+" button and select "Add Launcher" from the drop-drown menu.

Expected result:

Nothing.

Actual result:

The following traceback is emitted.

Traceback (most recent call last):
  File "/home/mk/Downloads/menulibre/menulibre/MenulibreApplication.py", line 1633, in on_add_launcher_cb
    self.add_launcher()
  File "/home/mk/Downloads/menulibre/menulibre/MenulibreApplication.py", line 1422, in add_launcher
    if row_data[2] == MenuItemTypes.DIRECTORY:
TypeError: 'NoneType' object is not subscriptable

Workaround:

I have attached a patch which makes the add_launcher routine exit early if no row is selected in the directory tree view.

[Sean Davis (bluesabre)]

Thanks for the patch! Confirmed and applied at the below commit.

http://bazaar.launchpad.net/~menulibre-dev/menulibre/master/revision/461

[My Karlsson (mykarlsson-deactivatedaccount)]

Thanks!