The Battle for Wesnoth Forums

ChaosRider wrote:Ok, but what was wrong in my first posted in this topic code?

You used tags that don't exist, like [filter_unit] and [not_side].

Telchin wrote:

Code: Select all

  {FOREACH humside i}
      [if]
         [have_unit]
            side=humside[i]
            canrecruit=yes
         [/have_unit]
         [then]
            [gold]
               side=humside[i]
               amount=75
            [/gold]

There are 4 dollar signs missing here.

No, fireball and bolt work just fine for me. Maybe there's something missing in another macro which is called somewhere indirectly, but since the error messages can't handle excessive macro usage very well, I've got no idea. I put a test scenario in front of the prologue so I can debug more easily, ...

Dugi wrote:There is a tag named [move_unit_fake] for it. Use the {MOVE_UNIT id=blablabla x y} macro, that will make it even easier.

There is also a [move_unit] tag, which really moves the unit instead of just moving an image like [move_unit_fake] does.

Line 197: The #endif is missing.

Hi there! Sorry for not contributing much lately, I got stuck with the "stunned" status. The possibility of the effect prolonging when the affected unit is hit by an impact weapon made it way harder, and it is not done yet :augh: Anyway, here're the first two fire/thunder/battle spells. I ...

[filter_adjacent] x,y=$x1,$y1 count=5 [/filter_adjacent] It kind of looks as if you mixed up [filter_adjacent] and [filter_location] here - right now you're searching for five adjacent units which have to be at $x1,$y1. I tried this, and it stored all units from side 1 which are up to 5 hexes away ...

EDIT I made a quick check using the interactive :lua debug command, and could remove one of the labels that way, so the FOREACH... should work, something else was wrong I suppose. Gah :doh: Note to self: read code written by myself at least thrice to notice all glaringly obvious mistakes. Sorry abo...

I tried that too, but it only removes the random name parts. You have still things like " 's Forest" or "ham" left.

Pentarctagon wrote:Would it allow you to do a FOREACH loop on all the map hexes and set the label to an empty string?

Tried it, nope.

Is there any specific reason for those labels to be immutable? IMO it would make much more sense if they weren't.

Code: Select all

[set_variable]
  name=texty
  value=$firer.y
[/set_variable]

I'm assuming that it should be testy, not texty.

I wasn't able to reproduce that. I only got an error about Of_The_Uprising/units not existing. Once I removed the include from the _main.cfg everything worked fine.

Is it intentional that the female sprites still have parts of the beard?

When in the save/load map dialog, double-clicking the double period (..) brings you to the parent directory and double-clicking a folder brings you to that folder, allowing you to navigate around and save/load maps wherever you want.