From 7740980fccbbd2d4eb250511182260bcb5d26037 Mon Sep 17 00:00:00 2001
From: jacob1 <jfu614@gmail.com>
Date: Mon, 11 Aug 2014 00:56:26 -0400
Subject: [PATCH] fix crash when selecting PROP from element search, fixes #209

---
 src/gui/elementsearch/ElementSearchActivity.cpp | 17 ++++++++++++-----
 src/gui/elementsearch/ElementSearchActivity.h   |  2 ++
 2 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/src/gui/elementsearch/ElementSearchActivity.cpp b/src/gui/elementsearch/ElementSearchActivity.cpp
index 92a5c72be..f36aee85e 100644
--- a/src/gui/elementsearch/ElementSearchActivity.cpp
+++ b/src/gui/elementsearch/ElementSearchActivity.cpp
@@ -25,7 +25,8 @@ ElementSearchActivity::ElementSearchActivity(GameController * gameController, st
 	WindowActivity(ui::Point(-1, -1), ui::Point(236, 302)),
 	gameController(gameController),
 	tools(tools),
-	firstResult(NULL)
+	firstResult(NULL),
+	exit(false)
 {
 	ui::Label * title = new ui::Label(ui::Point(4, 5), ui::Point(Size.X-8, 15), "Element Search");
 	title->SetTextColour(style::Colour::InformationTitle);
@@ -56,7 +57,7 @@ ElementSearchActivity::ElementSearchActivity(GameController * gameController, st
 			CloseAction(ElementSearchActivity * a) : a(a) {  }
 			void ActionCallback(ui::Button * sender_)
 			{
-				a->Exit();
+				a->exit = true;
 			}
 		};
 
@@ -167,7 +168,7 @@ void ElementSearchActivity::searchTools(std::string query)
 void ElementSearchActivity::SetActiveTool(int selectionState, Tool * tool)
 {
 	gameController->SetActiveTool(selectionState, tool);
-	Exit();
+	exit = true;
 }
 
 void ElementSearchActivity::OnDraw()
@@ -179,17 +180,23 @@ void ElementSearchActivity::OnDraw()
 	g->drawrect(Position.X+searchField->Position.X, Position.Y+searchField->Position.Y+searchField->Size.Y+8, searchField->Size.X, Size.Y-(searchField->Position.Y+searchField->Size.Y+8)-23, 255, 255, 255, 180);
 }
 
+void ElementSearchActivity::OnTick(float dt)
+{
+	if (exit)
+		Exit();
+}
+
 void ElementSearchActivity::OnKeyPress(int key, Uint16 character, bool shift, bool ctrl, bool alt)
 {
 	if(key == KEY_ENTER || key == KEY_RETURN)
 	{
 		if(firstResult)
 			gameController->SetActiveTool(0, firstResult);
-		Exit();
+		exit = true;
 	}
 	if(key == KEY_ESCAPE)
 	{
-		Exit();
+		exit = true;
 	}
 }
 
diff --git a/src/gui/elementsearch/ElementSearchActivity.h b/src/gui/elementsearch/ElementSearchActivity.h
index 74bb898d8..76f81ec60 100644
--- a/src/gui/elementsearch/ElementSearchActivity.h
+++ b/src/gui/elementsearch/ElementSearchActivity.h
@@ -21,11 +21,13 @@ class ElementSearchActivity: public WindowActivity {
 	void searchTools(std::string query);
 public:
 	class ToolAction;
+	bool exit;
 	Tool * GetFirstResult() { return firstResult; }
 	ElementSearchActivity(GameController * gameController, std::vector<Tool*> tools);
 	void SetActiveTool(int selectionState, Tool * tool);
 	virtual ~ElementSearchActivity();
 	virtual void OnDraw();
+	virtual void OnTick(float dt);
 	virtual void OnKeyPress(int key, Uint16 character, bool shift, bool ctrl, bool alt);
 };