From 7e691ddf38171879fa3d44907b713cae6dc0fc02 Mon Sep 17 00:00:00 2001
From: James Crook <james.k.crook@gmail.com>
Date: Thu, 1 Jun 2017 23:42:54 +0100
Subject: [PATCH] Fix crash bug (regression) in Automation

Somewhere along the line, GetEffectIdentifier got changed to GetEffectByIdentifier, leading to abuse of an iterator when attempting to invoke an effect by name.
---
 src/commands/CommandManager.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/commands/CommandManager.cpp b/src/commands/CommandManager.cpp
index 6f39cc12c..87d4d63dc 100644
--- a/src/commands/CommandManager.cpp
+++ b/src/commands/CommandManager.cpp
@@ -1341,7 +1341,7 @@ bool CommandManager::HandleTextualCommand(wxString & Str, CommandFlag flags, Com
    const PluginDescriptor *plug = pm.GetFirstPlugin(PluginTypeEffect);
    while (plug)
    {
-      if (em.GetEffectByIdentifier(plug->GetID()).IsSameAs(Str))
+      if (em.GetEffectIdentifier(plug->GetID()).IsSameAs(Str))
       {
          return proj->OnEffect(plug->GetID(), AudacityProject::OnEffectFlags::kConfigured);
       }