From 244b40bfbf0a5b138eb045919bde4281b25803af Mon Sep 17 00:00:00 2001
From: auria <auria@178a84e3-b1eb-0310-8ba1-8eac791a3b58>
Date: Wed, 8 Sep 2010 22:40:54 +0000
Subject: [PATCH] Added support for deleting added keyboard configurations

git-svn-id: svn+ssh://svn.code.sf.net/p/supertuxkart/code/main/trunk@5929 178a84e3-b1eb-0310-8ba1-8eac791a3b58
---
 data/gui/options_device.stkgui               |  9 +++--
 src/input/device_manager.cpp                 | 36 ++++++++++++++++++++
 src/input/device_manager.hpp                 |  5 ++-
 src/states_screens/options_screen_input2.cpp | 25 ++++++++++++++
 src/utils/ptr_vector.hpp                     | 11 +++---
 5 files changed, 78 insertions(+), 8 deletions(-)

diff --git a/data/gui/options_device.stkgui b/data/gui/options_device.stkgui
index 8e929cccb..d3802f725 100644
--- a/data/gui/options_device.stkgui
+++ b/data/gui/options_device.stkgui
@@ -18,14 +18,19 @@
     
     <box proportion="1" width="100%" layout="vertical-row">
     
+        <!-- Configuration name -->
         <spacer height="6" width="10"/>
-        <label id="title" width="100%" text_align="center" text="[none]" />
+        <label id="title" width="100%" text_align="center" />
         <spacer height="16" width="10"/>
         
+        <!-- List of key bindings -->
         <list id="actions" proportion="5" width="75%" align="center"/>
          
+         <!-- Bottom buttons -->
         <spacer width="50" height="20" /> 
- 
+        <button id="delete"
+                I18N="To delete a keyboard configuration"  text="Delete Configuration"/>
+        <spacer width="50" height="10" /> 
         <button id="back_to_device_list" I18N="In the input configuration screen"  text="Back to device list"/>
         
         <spacer width="50" height="10" /> 
diff --git a/src/input/device_manager.cpp b/src/input/device_manager.cpp
index b0ce6f6ab..b0a318594 100644
--- a/src/input/device_manager.cpp
+++ b/src/input/device_manager.cpp
@@ -191,10 +191,46 @@ void DeviceManager::addEmptyKeyboard()
 }
 
 // -----------------------------------------------------------------------------
+
 void DeviceManager::addGamepad(GamePadDevice* d)
 {
     m_gamepads.push_back(d);
 }
+
+// -----------------------------------------------------------------------------
+
+bool DeviceManager::deleteConfig(DeviceConfig* config)
+{
+    for (int n=0; n<m_keyboards.size(); n++)
+    {
+        if (m_keyboards[n].getConfiguration() == config)
+        {
+            m_keyboards.erase(n);
+            n--;
+        }
+    }
+    for (int n=0; n<m_gamepads.size(); n++)
+    {
+        if (m_gamepads[n].getConfiguration() == config)
+        {
+            m_gamepads.erase(n);
+            n--;
+        }
+    }
+    
+    if (m_keyboard_configs.erase(config))
+    {
+        return true;
+    }
+    
+    if (m_gamepad_configs.erase(config))
+    {
+        return true;
+    }
+    
+    return false;
+}
+
 // -----------------------------------------------------------------------------
 
 InputDevice* DeviceManager::mapKeyboardInput( int btnID, InputManager::InputDriverMode mode,
diff --git a/src/input/device_manager.hpp b/src/input/device_manager.hpp
index f958ae0e1..36f46d860 100644
--- a/src/input/device_manager.hpp
+++ b/src/input/device_manager.hpp
@@ -102,7 +102,10 @@ public:
     KeyboardDevice*     getKeyboardFromBtnID(const int btnID);
 
 
-
+    /**
+      * \brief Delete the given config and removes DeviceManager references to it.
+      */
+    bool deleteConfig(DeviceConfig* config);
        
     /** Given some input, finds to which device it belongs and, using the corresponding device object,
       * maps this input to the corresponding player and game action. 
diff --git a/src/states_screens/options_screen_input2.cpp b/src/states_screens/options_screen_input2.cpp
index 69240510a..9842c3269 100644
--- a/src/states_screens/options_screen_input2.cpp
+++ b/src/states_screens/options_screen_input2.cpp
@@ -65,6 +65,21 @@ void OptionsScreenInput2::init()
     RibbonWidget* tabBar = this->getWidget<RibbonWidget>("options_choice");
     if (tabBar != NULL)  tabBar->select( "tab_controls", PLAYER_ID_GAME_MASTER );
     
+    ButtonWidget* deleteBtn = this->getWidget<ButtonWidget>("delete");
+    if (m_config->getType() != DEVICE_CONFIG_TYPE_KEYBOARD)
+    {
+        deleteBtn->setDeactivated();
+    }
+    else if (input_manager->getDeviceList()->getKeyboardAmount() < 2)
+    {
+        // don't allow deleting the last config
+        deleteBtn->setDeactivated();
+    }
+    else
+    {
+        deleteBtn->setActivated();
+    }
+
     
     LabelWidget* label = this->getWidget<LabelWidget>("title");
     label->setText( m_config->getName().c_str() );
@@ -373,6 +388,16 @@ void OptionsScreenInput2::eventCallback(Widget* widget, const std::string& name,
             }
         }
     }
+    else if (name == "delete")
+    {
+        // TODO: ask for confirmation before deleting
+        const bool success = input_manager->getDeviceList()->deleteConfig(m_config);
+        assert(success);
+        m_config = NULL;
+        input_manager->getDeviceList()->serialize();
+        StateManager::get()->replaceTopMostScreen(OptionsScreenInput::getInstance());
+    }
+    
 }   // eventCallback
 
 // -----------------------------------------------------------------------------
diff --git a/src/utils/ptr_vector.hpp b/src/utils/ptr_vector.hpp
index 96596d097..fcb6a9ba7 100644
--- a/src/utils/ptr_vector.hpp
+++ b/src/utils/ptr_vector.hpp
@@ -179,21 +179,22 @@ void remove(TYPE* obj)
 }
 
 /**
-  * Removes and deletes
+  * \brief Removes and deletes the given object.
+  * \return whether this object was found in the vector and deleted
   */
-void erase(TYPE* obj)
+bool erase(void* obj)
 {
     for(unsigned int n=0; n<contentsVector.size(); n++)
     {
         TYPE * pointer = contentsVector[n];
-        if(pointer == obj)
+        if((void*)pointer == obj)
         {
             contentsVector.erase(contentsVector.begin()+n);
             delete pointer;
-            return;
+            return true;
         }
     }
-    
+    return false;
 }
 
 };