Knights and Merchants.net Forum

Spawner is a group ID, not a unit ID. UnitHungerSet is expecting a unit ID. You need to use a for-loop and run UnitHungerSet on every unit in the group (using GroupMember)

Is there a way to automatically detect all units on the map?

That will be added in the next RC (States.PlayerGetAllUnits which returns an array of Unit IDs). The same will be provided for houses and groups.

Alright, then I will just wait to implement this, because just selecting all units will be easier, and will be better in my case

I'd like to share my custom procedure to manage overlay text on the screen. I did that because I find that adding and removing texts manually through Actions.SetOverlayText, and manipulating durations to show the text is cumbersome. It gets even more messy when several notifications are overlapping in the time that they need to be shown. My custom ScreenText procedure has five parameters to change the way any new piece of text is treated:

1) show a new piece of text on a new line, at the same line, put it on the top row or clear all current overlay text while adding a new line
2) make any piece of new text visible to a specific player, or all players at once
3) set the duration to show the new piece of text
4) extract the text that needs to be shown through a entry in the mission libx file
5) alternatively, do not use a libx file and feed custom strings to this procedure

Because I am using a global multi-dimensional array and a array of strings to queue the texts, KaM gives a warning. I would be very interested if somebody here could tell me how to change/improve my script to avoid this warning. The script is below. Just Copy-Paste this in any script file, it should work just fine. I welcome any feedback or suggestions for improvement!
To illustrate how this custom procedure works, paste this:

That's an awesome idea Tef

We plan to implement such a function into Remake scripts, so you could have all this with something like DisplayText(player, timeout, text); and have a list of expiring messages.
Not sure if we could fit that into closest release, but one after it will have it.

We don't want to allow global string variables, because that would make MP saves incompatible if they were saved from different locales.

Thanks for helping find the bug that allowed scripts to execute even if they had fatal warnings (like unsupported global variables, which can cause inconsistencies/errors after loading a save if the script is allowed to run). In the next release only allowed global variables will work, but we will support a lot more types as global variables such as records, multidimensional and dynamic arrays, enumerators, sets, etc.

Because of different languages, right? But I still have no clue how to avoid global strings in some cases. I challenge you:

Suppose there is Team A (0,1,2,3) and Team B (4,5,6,7). Player 0 is doing something that causes a timer to run, which needs the 'playername' of player 0 to be shown only to Team A as overlay text + the status of the timer. But, while the timer runs, also additional notifications need to be shown. So the 'timer-text (=player name, + timer)' and ' other notifications' all need to be shown simultaneously. Dynamic stuff like player names or timers cannot be put into a libx file, so you need dynamic strings. However, local strings won't work, because the procedure 'forgets' any older information when local strings are used. Must I now start coding strings as integers? How to solve this case?!

We realize that essentially you've made an extension to let mapmakers display short messages that auto-remove themselves after some time. We plan to add function similar to those you have made, but directly into the Remake scripting module, so all global strings you have used will be managed by Remake. The script will only have to request ScreenText part.

Hey, i encountered a problem while scripting a function which search for soldiers of a player in an area.

if ((GroupID <> -1) and (States.GroupOwner(GroupID) = PlayerID)) then begin
Result := true;
end;

I always use short-circuit evaluation in other languages, but in pascal script it let the game freeze or bring down the performance so the mission doesn't start.
If i however write two "if constructs" it works flawless? Its my fault or does not pascal support this feature?

Just curious, because for me its more readable

Thanks for bringing that up Philymaster. Short circuit evaluation is enabled by default in Delphi and Lazarus, we use it all the time in the KaM Remake code. However, the PascalScript interpreter has it disabled by default for some reason... I've enabled it now so in the next RC code like you wrote will work

The freezing and low performance is possibly because it is generating a lot of errors that are written to the log files. Open up the latest file in the Logs folder and scroll to the bottom, see if there are script errors (in this case it will complain that you used GroupOwner with -1 as the GroupID, because it's not using short circuit evaluation)

Thanks Lewin, i'm looking forward to the new release.
Unfortunately i have two new problems. (Or one problem and one question)

I have written a function which returns an array with all group ids of a player and wanted to use the for each loop (for ... in ... do) and your script validator expected a regular for. Is this feature also deactivated?

Also i found a other problem with the procedures GroupOrderStorm and GroupOrderWalk.

My code was:
Actions.GroupOrderStorm( ... ); // Enemy Group 1
Actions.GroupOrderStorm( ... ); // Enemy Group 2
Actions.GroupOrderWalk( ... ); // Enemy Group 3

The first and second group spawn in front of the players groups so they attack after few tiles running. The third group is far away and on the way to the player, but once one of the group start to fight the third group don't walk anymore and just stop. I've tested it with multiple groups and all just stop.

I assume its a bug and not desired? I will script a refresh procedure which will be called every second and will test if they stop multiple times then.

The script compiler doesn't seem to support for..in loops. You can use an iterator instead fairly easily: for I:=0 to Length(array)-1 do array

The second problem might be an issue with the AI trying to take control of that group to help in the battle, I'll look into it.

returns an error, it expects a semicolon after (P) in the second line here. I'm trying to iterate through all the houses of a player, and each time save ID as an auxilarry variable.

(before you ask, I just compiled 5252 to have all the new needed scripting commands, so I have the PlayerGetAllHouses state)

Don't use States.PlayerGetAllHouses "inline" like that. For a start that's incredibly inefficient, it's regenerating the list of houses every time you use States.PlayerGetAllHouses. Secondly, it's possible that the PascalScript interpreter can't handle syntax like "States.PlayerGetAllHouses(P)".

Instead, declare an array as a local variable:
var Houses: array of Integer;

Then do:
Houses := States.PlayerGetAllHouses(P);

And the rest of your code becomes:
for I := 0 to Length(Houses)-1 do
ID := Houses;

That will be 100 times more efficient/faster because it's only generating the list of houses once, and should compile fine.