Knights and Merchants.net Forum

@Ben: Please be more specific in error's description. What is exactly happening, do you get an error message, or houses aren't blocked/unlocked?

Nothing in the dynamic script is working at all. I am just getting this error:

That is because HouseAllow is not a state, it is an action. You need to change it to Actions.HouseAllow(...).

Yeah looks like I didn't write that example script properly, I was doing it in a hurry without checking. Remember it this way:

States are when you query the game without changing anything. For example: "How many warriors does player X have?", "How long has the game been running?", "What type of house is this?"

Actions are when you change something in the game, for example "Add a group of warriors for player X", "Show a message to the player", "Allow the player to build this house".

Events (OnSomething) are when the game lets you know that some event occurred, for example "A house was built", "a unit died", "game tick occurred".

Thanks, Krom and Lewin. I'll see if I can get it working now.

And thanks also, Lewin, for your explanation. That helps a bit.

EDIT:

Okay, it is finally recognizing the code, but now it's doing something funky. It is disabling all buildings except the inn, which shouldn't be happening because the inn's ID (27) isn't in my code: Totally confused...

Thanks for reporting, there was a bug in HouseAllow which made it work the opposite way expected. It works as "HouseBlock". So for now if you switch all cases of "True" to "False" it will work
But next RC it will be fixed so that HouseAllow(A, B, True) makes the house allowed, and False makes it blocked.

I thought that this was what was happening, so I actually tried switching everything to False, but it still didn't work.
I must have overlooked something. I'll go try again.

Don't switch everything to false, swap true with false and false with true.
Also note that the example I gave you wasn't complete, you need to customise what happens in OnHouseBuilt so building the school allows the inn, etc. You also need to allow everything when they build the sawmill to make it use the "normal" unlock order after that.

Yeah that's what I meant. I figured it out; though, and it is working fine Yeah I did it the hard way: manually doing each unlock. I guess that I wasn't thinking. Oh well. More experience I guess

Also, if I just wanted to disable just two buildings, and not, say 9-27, could I just do two IDs instead of a range?

Yes that might be smarter in this case. Just run Actions.HouseAllow twice with fixed IDs (instead of I from the loop), and remove the for loop completely.

Thanks again.

Sado and I were trying to figure out if we could use the OnTick even to run after more than just 1 tick. For example, say I wanted a unit to be created after 1000 ticks. How would I do this?

I'm having troubles too. This error occurs: [Error] (8:26): Type mismatch
I'm using the exact same script as before, when it worked just fine. What's changed?
Also, can't it just ignore the scripts that aren't working? Currently nothing works if there's just one tiny mistake; that's pretty annoying, I can't do a thing right now.

Would be nice to actually have a tool that'd check script's integrity without running the map again and again just to see what was the mistake.

We changed quite a lot of stuff since you originally wrote that script. In future releases we'll try to keep backwards compatibility better, but you were using an "alpha" version of the scripting demo back then Here's the stuff which affects your script
- The error you got says there's a mistake on line 8 at character 26. You can see that's this: Actions.ShowMsg, the second parameter needs to be a string (text). You can fetch text out of the LIBX files using States.Text(X). So replace "Actions.ShowMsg(0, aMsg);" with "Actions.ShowMsg(0, States.Text(aMsg));" This allows you to combine text more easily, like I did for the overlay in Annie's Hill. There are more cases where you used "Actions.ShowMsg" in the script, so make sure you fix all of them.
- States.ArmyCount was renamed to States.StatArmyCount
- The "begin .... end." at the bottom of the document (the main bit of code that runs every tick) needs to be replaced with "procedure OnTick;
begin
...
end;"
So basically add the procedure line at the top and replace the final "." after the end with a ";". We changed it to work like this to be more consistent (everything happens through events)

When you get an error like "(8:26): Type mismatch", check what commands you use on that line (8) and lookup in the wiki to make sure you are using them correctly (in this case you provided an integer when it expected a string, so that's a type mismatch)

After fixing those 3 things your script runs without errors for me, and shows me a message
Unfortunately no, that's the way programming usually works. Because that one mistake could cause nothing else to work in the script. The closest thing we could do is remove every line that causes an error (there's a joke Javascript library that does that...) but that wouldn't be very helpful, since often it will just cause more errors and confuse the user when the wrong stuff happens, for example if we removed an if statement that was broken the conditional code would run every time.
Yes it would, I'm planning to write one for the next RC


EDIT:
Your procedure OnTick will be run EVERY tick of the game, 10 times per second for the entire game. If you want a unit to be created after 1000 ticks use this: So every tick the game will check "is this tick 1000"? If yes, it will run the GiveUnit code. If no, it won't do anything in this case. But you could add more stuff into OnTick:

But this only works once. I thought that the OnTick would work as a loop, but with the gametime action, it doesn't work like that. So how do I make this a loop?