Fixed bug regarding two plugins matching the same one-shot trigger/alias

Author: nickgammon

ProcessPreviousLine.cpp

@@ -894,6 +894,7 @@ assemble the full text of the original line.
 
   int iDeletedCount = 0;
   int iDeletedNonTemporaryCount = 0;
+  set<CPlugin *> pluginsWithDeletions;
 
   for (OneShotItemMap::const_iterator one_shot_it = mapOneShotItems.begin ();
        one_shot_it != mapOneShotItems.end ();
@@ -922,11 +923,18 @@ assemble the full text of the original line.
     // now delete its entry
     GetTriggerMap ().RemoveKey (strTriggerName);
 
+    pluginsWithDeletions.insert (m_CurrentPlugin);
+
    }  // end of deleting one-shot items
 
    if (iDeletedCount > 0)
      {
-     SortTriggers ();
+     // make sure we sort the correct plugin(s)
+     for ( set<CPlugin *>::iterator i = pluginsWithDeletions.begin (); i != pluginsWithDeletions.end (); i++)
+       {
+       m_CurrentPlugin = *i;
+       SortTriggers ();
+       }
 
      if (iDeletedNonTemporaryCount > 0) // plugin mods don't really count
        SetModifiedFlag (TRUE);   // document has changed

evaluate.cpp

@@ -177,6 +177,7 @@ CAliasList AliasList;
 
   int iDeletedCount = 0;
   int iDeletedNonTemporaryCount = 0;
+  set<CPlugin *> pluginsWithDeletions;
 
   for (OneShotItemMap::const_iterator one_shot_it = mapOneShotItems.begin ();
        one_shot_it != mapOneShotItems.end ();
@@ -205,11 +206,18 @@ CAliasList AliasList;
     // now delete its entry
     GetAliasMap ().RemoveKey (strAliasName);
 
-   }  // end of deleting one-shot items
+    pluginsWithDeletions.insert (m_CurrentPlugin);
+
+    }  // end of deleting one-shot items
 
    if (iDeletedCount > 0)
      {
-     SortAliases ();
+     // make sure we sort the correct plugin(s)
+     for ( set<CPlugin *>::iterator i = pluginsWithDeletions.begin (); i != pluginsWithDeletions.end (); i++)
+       {
+       m_CurrentPlugin = *i;
+       SortAliases ();
+       }
 
      if (iDeletedNonTemporaryCount > 0) // plugin mods don't really count
        SetModifiedFlag (TRUE);   // document has changed