quicksilver/Quicksilver — pull request #1260 by pjrobertson (2012-11-30 11:44:55+00:00)
Title: Don't add multiple actions with the same ID. Fixes #1259

The code for dealing with duplicates was already in the code, but for some strange reason it just removed the `directTypes` and `directFileTypes` for the *old* action, before adding the new action as well.

This kind of make the old action (the one that was added first) pointless as it's direct types weren't set properly.

I've gone with the view that IDs should be unique. If there's already an action with the same ID that exists, then do nothing. Don't replace it, don't try and do anything fancy :)

I could have fixed #1259 in the services plugin but thought it was better to get it right from the start :)