Grimrock

Posted: **Sat Jun 11, 2016 12:57 am**

You need a script entity in your dungeon and set its source to your external lua file. Then choose a specific id for that script entity, so you can use that id anywhere you want.

Posted: **Sat Jun 11, 2016 4:40 pm**

And so I face another spell related issue.

When I have to change more than one thing related to the tileDamager I have to make any other modification after the first one not part of the 'spawn' line. I tried doing it separately, but it shows me an error saying 'tried to index local 'a' (a nil value)'

Now, I've followed some custom spell examples I've seen, and they used a 'local a = spawn(...)', and later changed the spawned object with 'a.tiledamager:setAttackPower(...)'

The function is defined in an external script and is linked into the game through a scrip entity on the first level, where I do the testing.
Here's the part of the function that causes the error:

Code: Select all

	local a = spawn(s, party.level, x, y, direction, party.elevation).tiledamager:setCastByChampion(champ:getOrdinal())
	a.tiledamager:setAttackPower(power)

Removing 'a.tiledamager:setAttackPower(power)' will stop it from crashing, but its needed for the spell to work properly.

Posted: **Sat Jun 11, 2016 5:26 pm**

Try that instead!

Code: Select all

  local a = spawn(s, party.level, x, y, direction, party.elevation)
  a.tiledamager:setCastByChampion(champ:getOrdinal())
  a.tiledamager:setAttackPower(power)

Posted: **Sat Jun 11, 2016 5:43 pm**

Quick 2 questions:
1. What does projectile:pushForward() do??
2. What does champion:getDualClass() do??

Thanks all

Posted: **Sat Jun 11, 2016 6:35 pm**

AndakRainor wrote:Try that instead!

Code: Select all

  local a = spawn(s, party.level, x, y, direction, party.elevation)
  a.tiledamager:setCastByChampion(champ:getOrdinal())
  a.tiledamager:setAttackPower(power)

Huh.. This works... But why didn't the previous one work?

Posted: **Sat Jun 11, 2016 7:27 pm**

zimberzimber wrote:
AndakRainor wrote:Try that instead!
Code: Select all
  local a = spawn(s, party.level, x, y, direction, party.elevation)
  a.tiledamager:setCastByChampion(champ:getOrdinal())
  a.tiledamager:setAttackPower(power)
Huh.. This works... But why didn't the previous one work?

The spawn() function returns a game object.
The setCastByChampion() function always returns nil.

So this will set the variable "a" to the newly spawned game object:
local a = spawn(s, party.level, x, y, direction, party.elevation)

But this will set the variable "a" to nil:
local a = spawn(s, party.level, x, y, direction, party.elevation).tiledamager:setCastByChampion(champ:getOrdinal())

Maybe this shows it more clearly:
local a = spawn("wall_fire", party.level, party.x, party.y, party.facing, party.elevation)
local b = a.tiledamager:setAttackPower(0)
hudPrint("a = " .. tostring(a))
hudPrint("b = " .. tostring(b))

Posted: **Sat Jun 11, 2016 7:42 pm**

Zo Kath Ra wrote: The spawn() function returns a game object.
The setCastByChampion() function always returns nil.

Ah, I see. Thanks for clarifying!

And thanks to everyone who helped with the positioning, obstacle detection, and spell damage/source!

So here's my complete script for summoning tile damagers:

Code: Select all

function tileDamagerSpell(champ, direction, skill, s, power)
	local dx,dy = getForward(party.facing)
	local o = party.map:checkObstacle(party, party.facing)
	local x, y
	
	if party.map:isWall(party.x + dx, party.y + dy) then
		x = party.x
		y = party.y
	else
		x = party.x + dx
		y = party.y + dy
	end
	
	if party.map:getElevation(x, y) == party.elevation then
		if  not o or o == "dynamic_obstacle" then
			local a = spawn(s, party.level, x, y, direction, party.elevation)
			a.tiledamager:setCastByChampion(champ:getOrdinal())
			a.tiledamager:setAttackPower(power)
		else
			local a = spawn(s, party.level, party.x, party.y, direction, party.elevation)
			a.tiledamager:setCastByChampion(champ:getOrdinal())
			a.tiledamager:setAttackPower(power)
		end
		return
	else
		if  not o or o == "dynamic_obstacle" then
			for e in party.map:entitiesAt(x, y) do
				if e.platform and e.platform:isEnabled() and e.elevation == party.elevation then
					if  not o or o == "dynamic_obstacle" then
						local a = spawn(s, party.level, x, y, direction, party.elevation)
						a.tiledamager:setCastByChampion(champ:getOrdinal())
						a.tiledamager:setAttackPower(power)
					else
						local a = spawn(s, party.level, party.x, party.y, direction, party.elevation)
						a.tiledamager:setCastByChampion(champ:getOrdinal())
						a.tiledamager:setAttackPower(power)
					end
					return
				end
			end
		elseif (party.map:getElevation(party.x, party.y) == party.elevation) then
			local a = spawn(s, party.level, party.x, party.y, direction, party.elevation)
			a.tiledamager:setCastByChampion(champ:getOrdinal())
			a.tiledamager:setAttackPower(power)
			return
		end
	end
	playSound("spell_fizzle")
end

Posted: **Sun Jun 12, 2016 12:28 am**

Do you have problems with ItemComponent.onUnequipItem(self, champion, slot)?

I get an attempt to index field 'map' (a nil value) each time, just by clicking the item in the inventory to grab it. I don't use anything from the calling item in the hooked function.

Edit: the problem happens only with Component:addConnector(event, target, action), not if I define the onUnequipItem function directly in the object definition.

Posted: **Sun Jun 12, 2016 1:39 am**

Confirmed. I get that with the following:

Code: Select all

function addItem(item)
	local d = party:spawn(item)
	print("map level: "..d.map:getLevel())
	d.item:addConnector("onEquipItem", self.go.id, "payload")
	d.item:addConnector("onUnEquipItem", self.go.id, "payload")
end
addItem("dagger")	

function payload(...)
	print(...)
end

Posted: **Sun Jun 12, 2016 2:32 am**

If an object not on a map has any connectors, and it triggers them, it will crash, because triggering a connector uses the triggering object's map. The raw hooks in the component definition will still work; they don't use the map. This means that you can imitate connector functionality without needing a map. There is already a Grimrock 2 library that can do this: viewtopic.php?f=22&t=8345

Grimrock

Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer

Re: Ask a simple question, get a simple answer