check the presence of OnActivate using the SuppressActivate flag instead of looking for keywords

apps/openmw/mwbase/scriptmanager.hpp

@@ -42,10 +42,6 @@ namespace MWBase
             ///< Compile script with the given namen
             /// \return Success?
 
-            virtual bool hasOnActivate (const std::string& name) = 0;
-            ///< Determine if a script with the given name contains OnActivate
-            /// \return Contains OnActivate?
-
             virtual std::pair<int, int> compileAll() = 0;
             ///< Compile all scripts
             /// \return count, success

apps/openmw/mwgui/inventorywindow.cpp

@@ -663,8 +663,7 @@ namespace MWGui
         if (object.getClass().getName(object) == "") // objects without name presented to user can never be picked up
             return;
 
-        std::string scriptName = object.getClass().getScript(object); // Objects that have OnActivte in their script cannot be picked up through inventory
+        if (object.getRefData().hasSuppressActivate()) // if Flag_SuppressActivate is set, script that contains OnActivate is attached to the item
-        if (!scriptName.empty() && MWBase::Environment::get().getScriptManager()->hasOnActivate(scriptName))
         {
             if (std::find(mItemsToActivate.begin(), mItemsToActivate.end(), object) == mItemsToActivate.end())
             {

apps/openmw/mwscript/scriptmanagerimp.cpp

@@ -88,12 +88,6 @@ namespace MWScript
         return false;
     }
 
-    bool ScriptManager::hasOnActivate(const std::string& name)
-    {
-        const ESM::Script *script = mStore.get<ESM::Script>().find(name);
-        return script->mScriptText.find("OnActivate");
-    }
-
     void ScriptManager::run (const std::string& name, Interpreter::Context& interpreterContext)
     {
         // compile script

apps/openmw/mwscript/scriptmanagerimp.hpp

@@ -62,10 +62,6 @@ namespace MWScript
             ///< Compile script with the given namen
             /// \return Success?
 
-            virtual bool hasOnActivate(const std::string& name);
-            ///< Determine if a script with the given name contains OnActivate
-            /// \return Contains OnActivate?
-
             virtual std::pair<int, int> compileAll();
             ///< Compile all scripts
             /// \return count, success

apps/openmw/mwworld/refdata.cpp

@@ -192,6 +192,11 @@ namespace MWWorld
         return mEnabled;
     }
 
+    bool RefData::hasSuppressActivate()
+    {
+        return mFlags & Flag_SuppressActivate;
+    }
+
     void RefData::enable()
     {
         if (!mEnabled)

apps/openmw/mwworld/refdata.hpp

@@ -111,6 +111,8 @@ namespace MWWorld
 
             bool isEnabled() const;
 
+            bool hasSuppressActivate();
+
             void enable();
 
             void disable();