From 3a92ff96ba5068aff0417219eed8cf759b55b9f7 Mon Sep 17 00:00:00 2001
From: Mikaël Capelle <capelle.mikael@gmail.com>
Date: Sat, 2 Jan 2021 15:08:29 +0100
Subject: Remove setName and remove from mod interface.

---
 src/modlist.cpp      | 29 ++++++++++++++++++++++++++++-
 src/modlist.h        |  3 +++
 src/modlistproxy.cpp |  5 +++++
 src/modlistproxy.h   |  1 +
 4 files changed, 37 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/modlist.cpp b/src/modlist.cpp
index 237f65fa..c8056140 100644
--- a/src/modlist.cpp
+++ b/src/modlist.cpp
@@ -940,7 +940,12 @@ bool ModList::removeMod(MOBase::IModInterface* mod)
 {
   unsigned int index = ModInfo::getIndex(mod->name());
   if (index == UINT_MAX) {
-    return mod->remove();
+    if (auto* p = dynamic_cast<ModInfo*>(mod)) {
+      return p->remove();
+    }
+    else {
+      return false;
+    }
   }
   else {
     return ModInfo::removeMod(index);
@@ -948,6 +953,28 @@ bool ModList::removeMod(MOBase::IModInterface* mod)
   notifyModRemoved(mod->name());
 }
 
+MOBase::IModInterface* ModList::renameMod(MOBase::IModInterface* mod, const QString& name)
+{
+  unsigned int index = ModInfo::getIndex(mod->name());
+  if (index == UINT_MAX) {
+    if (auto* p = dynamic_cast<ModInfo*>(mod)) {
+      p->setName(name);
+      return p;
+    }
+    else {
+      return nullptr;
+    }
+  }
+  else {
+    if (renameMod(index, name)) {
+      return ModInfo::getByName(name).get();
+    }
+    else {
+      return nullptr;
+    }
+  }
+}
+
 IModList::ModStates ModList::state(const QString &name) const
 {
   unsigned int modIndex = ModInfo::getIndex(name);
diff --git a/src/modlist.h b/src/modlist.h
index fd26bb91..21a19fab 100644
--- a/src/modlist.h
+++ b/src/modlist.h
@@ -167,6 +167,9 @@ public:
   // \copydoc MOBase::IModList::remove
   bool removeMod(MOBase::IModInterface* mod);
 
+  // \copydoc MOBase::IModList::renameMod
+  MOBase::IModInterface* renameMod(MOBase::IModInterface* mod, const QString& name);
+
   /// \copydoc MOBase::IModList::state
   MOBase::IModList::ModStates state(const QString &name) const;
 
diff --git a/src/modlistproxy.cpp b/src/modlistproxy.cpp
index 6165fc3f..cae5b1f7 100644
--- a/src/modlistproxy.cpp
+++ b/src/modlistproxy.cpp
@@ -55,6 +55,11 @@ bool ModListProxy::removeMod(MOBase::IModInterface* mod)
   return m_Proxied->removeMod(mod);
 }
 
+MOBase::IModInterface* ModListProxy::renameMod(MOBase::IModInterface* mod, const QString& name)
+{
+  return m_Proxied->renameMod(mod, name);
+}
+
 IModList::ModStates ModListProxy::state(const QString& name) const
 {
   return m_Proxied->state(name);
diff --git a/src/modlistproxy.h b/src/modlistproxy.h
index 162b0b43..d886d8b5 100644
--- a/src/modlistproxy.h
+++ b/src/modlistproxy.h
@@ -19,6 +19,7 @@ public:
   QStringList allModsByProfilePriority(MOBase::IProfile* profile = nullptr) const override;
   MOBase::IModInterface* getMod(const QString& name) const override;
   bool removeMod(MOBase::IModInterface* mod) override;
+  MOBase::IModInterface* renameMod(MOBase::IModInterface* mod, const QString& name) override;
   ModStates state(const QString& name) const override;
   bool setActive(const QString& name, bool active) override;
   int setActive(const QStringList& names, bool active) override;
-- 
cgit v1.3.1