Grimrock

zimberzimber wrote:OK so as the lazy fuck I am, I really don't feel like redefening every monster attack component, so I went with modifying base_monster with a new 'Null' component, that calls a function through onInit, that iterates through every component the monster has, and do stuff from there.
Now I got two problems:
1) There doesn't seem to be a getPierce/setPierce
2) Monsters that are not placed on the map don't seem to call that function (thing like the spawn function)

Any help with those?

local orig_defineObject = defineObject
defineObject = function(def)
	if def.components then
		for i,c in ipairs(def.components) do
			if c.class == "MonsterAttack" or c.class == "CrowernAttack" then
				c.pierce = math.huge
			end
		end
	end
	orig_defineObject(def)
end

minmay wrote:That was the save editor, which includes a button to remove nonstandard traits from the party to prevent this crash. But that's a tool for users to use, you can't integrate it in your mod obviously. There's no practical way to get rid of all the nonstandard traits from within your mod itself, because you don't know what the names of those traits are.

Isaac wrote:Here is an experimental charge attack for the party; implemented as a magical "Horn of Berserker Charge'.

*It's not necessarily balanced for play, but it does have some usage checks (and experimental limitations). As-is it may well allow the breaking puzzles that can be broken by a faster than expected party —provided they don't need to change direction during the charge.

zimberzimber wrote:Hah, so apparently I'll have to redefine every monster that uses melee attacks after all /̵͇̿​̿/’̿’̿ ̿ ̿̿ (Gotta index the monster when it melee hits a champion in a global table for future interactions)

zimberzimber wrote:So question - Would redefining the whole definition instead of defining an object with the same name and using the previous (original) definition as a base class have a significant difference?

akroma222 wrote:I think this Example (this is minmays work) should demonstrate its doable:

SpoilerShow

Code: Select all

local awardXp = function(self)
       
	local rval = Config.monsterOnDieMap[self.go.name] and Config.monsterOnDieMap[self.go.name](self) or true
    if rval then
		local xp = Config.monsterExpMap[self.go.name]
        if xp then
			xp = math.floor(xp * 1.1^(self:getLevel() - 1))
            for i=1,4 do
				if party.party:getChampion(i):getEnabled() and not party.party:getChampion(i):hasTrait("farmer") then
					party.party:getChampion(i):gainExp(xp)
                end
            end
            self:showDamageText(string.format("+%d xp",xp),"0xffff88")
		end
		-- oh, while we're at it, fix the phantom flames bug
		if self.go.burning then 
			self.go:removeComponent("burning") 
		end
    end
    return rval
end
--------------------------------------this method makes MonsterComponent:getExp() useless.
local orig_defineObject = defineObject
Config.monsterExpMap = {}
Config.monsterOnDieMap = {}
--------------------------------------
	defineObject = function(def)
       
		if def.components then 
			for _,c in ipairs(def.components) do
				if c.class == "Monster" then
					Config.monsterExpMap[def.name] = c.exp
					Config.monsterOnDieMap[def.name] = c.onDie
					c.onDie = awardXp
					c.exp = false 				-- removes the +xp text
				end
			end 
		end
		orig_defineObject(def)
    end

minmay wrote:

akroma222 wrote:I think this Example (this is minmays work) should demonstrate its doable:

SpoilerShow

Code: Select all

local awardXp = function(self)
       
	local rval = Config.monsterOnDieMap[self.go.name] and Config.monsterOnDieMap[self.go.name](self) or true
    if rval then
		local xp = Config.monsterExpMap[self.go.name]
        if xp then
			xp = math.floor(xp * 1.1^(self:getLevel() - 1))
            for i=1,4 do
				if party.party:getChampion(i):getEnabled() and not party.party:getChampion(i):hasTrait("farmer") then
					party.party:getChampion(i):gainExp(xp)
                end
            end
            self:showDamageText(string.format("+%d xp",xp),"0xffff88")
		end
		-- oh, while we're at it, fix the phantom flames bug
		if self.go.burning then 
			self.go:removeComponent("burning") 
		end
    end
    return rval
end
--------------------------------------this method makes MonsterComponent:getExp() useless.
local orig_defineObject = defineObject
Config.monsterExpMap = {}
Config.monsterOnDieMap = {}
--------------------------------------
	defineObject = function(def)
       
		if def.components then 
			for _,c in ipairs(def.components) do
				if c.class == "Monster" then
					Config.monsterExpMap[def.name] = c.exp
					Config.monsterOnDieMap[def.name] = c.onDie
					c.onDie = awardXp
					c.exp = false 				-- removes the +xp text
				end
			end 
		end
		orig_defineObject(def)
    end

Logging on despite the sudden lack of HTTPS, just to say that this terrible indentation is not my work. Here is the original.