From 0fb583e0659847a55d198cc5ecd07594bbc7768d Mon Sep 17 00:00:00 2001
From: Marc Zinnschlag <marc@zpages.de>
Date: Sat, 6 Apr 2013 17:35:36 +0200
Subject: [PATCH] added verifier for race record

---
 apps/opencs/CMakeLists.txt            |  2 +-
 apps/opencs/model/tools/racecheck.cpp | 47 +++++++++++++++++++++++++++
 apps/opencs/model/tools/racecheck.hpp | 29 +++++++++++++++++
 apps/opencs/model/tools/tools.cpp     |  3 ++
 4 files changed, 80 insertions(+), 1 deletion(-)
 create mode 100644 apps/opencs/model/tools/racecheck.cpp
 create mode 100644 apps/opencs/model/tools/racecheck.hpp

diff --git a/apps/opencs/CMakeLists.txt b/apps/opencs/CMakeLists.txt
index 07766507f7..0cf72b24c8 100644
--- a/apps/opencs/CMakeLists.txt
+++ b/apps/opencs/CMakeLists.txt
@@ -35,7 +35,7 @@ opencs_units (model/tools
     )
 
 opencs_units_noqt (model/tools
-    stage verifier mandatoryid skillcheck classcheck factioncheck
+    stage verifier mandatoryid skillcheck classcheck factioncheck racecheck
     )
 
 
diff --git a/apps/opencs/model/tools/racecheck.cpp b/apps/opencs/model/tools/racecheck.cpp
new file mode 100644
index 0000000000..50bb08a7c6
--- /dev/null
+++ b/apps/opencs/model/tools/racecheck.cpp
@@ -0,0 +1,47 @@
+
+#include "racecheck.hpp"
+
+#include <sstream>
+
+#include <components/esm/loadrace.hpp>
+
+#include "../world/universalid.hpp"
+
+CSMTools::RaceCheckStage::RaceCheckStage (const CSMWorld::IdCollection<ESM::Race>& races)
+: mRaces (races)
+{}
+
+int CSMTools::RaceCheckStage::setup()
+{
+    return mRaces.getSize();
+}
+
+void CSMTools::RaceCheckStage::perform (int stage, std::vector<std::string>& messages)
+{
+    const ESM::Race& race = mRaces.getRecord (stage).get();
+
+    CSMWorld::UniversalId id (CSMWorld::UniversalId::Type_Race, race.mId);
+
+    // test for empty name and description
+    if (race.mName.empty())
+        messages.push_back (id.toString() + "|" + race.mId + " has an empty name");
+
+    if (race.mDescription.empty())
+        messages.push_back (id.toString() + "|" + race.mId + " has an empty description");
+
+    // test for positive height
+    if (race.mData.mHeight.mMale<=0)
+        messages.push_back (id.toString() + "|male " + race.mId + " has non-positive height");
+
+    if (race.mData.mHeight.mFemale<=0)
+        messages.push_back (id.toString() + "|female " + race.mId + " has non-positive height");
+
+    // test for non-negative weight
+    if (race.mData.mWeight.mMale<0)
+        messages.push_back (id.toString() + "|male " + race.mId + " has negative weight");
+
+    if (race.mData.mWeight.mFemale<0)
+        messages.push_back (id.toString() + "|female " + race.mId + " has negative weight");
+
+    /// \todo check data members that can't be edited in the table view
+}
\ No newline at end of file
diff --git a/apps/opencs/model/tools/racecheck.hpp b/apps/opencs/model/tools/racecheck.hpp
new file mode 100644
index 0000000000..e3f12599ba
--- /dev/null
+++ b/apps/opencs/model/tools/racecheck.hpp
@@ -0,0 +1,29 @@
+#ifndef CSM_TOOLS_RACECHECK_H
+#define CSM_TOOLS_RACECHECK_H
+
+#include <components/esm/loadrace.hpp>
+
+#include "../world/idcollection.hpp"
+
+#include "stage.hpp"
+
+namespace CSMTools
+{
+    /// \brief VerifyStage: make sure that race records are internally consistent
+    class RaceCheckStage : public Stage
+    {
+            const CSMWorld::IdCollection<ESM::Race>& mRaces;
+
+        public:
+
+            RaceCheckStage (const CSMWorld::IdCollection<ESM::Race>& races);
+
+            virtual int setup();
+            ///< \return number of steps
+
+            virtual void perform (int stage, std::vector<std::string>& messages);
+            ///< Messages resulting from this tage will be appended to \a messages.
+    };
+}
+
+#endif
diff --git a/apps/opencs/model/tools/tools.cpp b/apps/opencs/model/tools/tools.cpp
index db45de43e5..5fb37514f0 100644
--- a/apps/opencs/model/tools/tools.cpp
+++ b/apps/opencs/model/tools/tools.cpp
@@ -15,6 +15,7 @@
 #include "skillcheck.hpp"
 #include "classcheck.hpp"
 #include "factioncheck.hpp"
+#include "racecheck.hpp"
 
 CSMTools::Operation *CSMTools::Tools::get (int type)
 {
@@ -60,6 +61,8 @@ CSMTools::Verifier *CSMTools::Tools::getVerifier()
         mVerifier->appendStage (new ClassCheckStage (mData.getClasses()));
 
         mVerifier->appendStage (new FactionCheckStage (mData.getFactions()));
+
+        mVerifier->appendStage (new RaceCheckStage (mData.getRaces()));
     }
 
     return mVerifier;