Akkarin

Mob Control Commands

Recommended Posts

There's a report in the Docs Bug Tracker (bugreport:4999) that's been there for a while - any devs fancy following this up?

Share this post


Link to post
Share on other sites

hum o.o it appears the documentation still says the unit commands work for mobs while they dont

Share this post


Link to post
Share on other sites

And in my opinion many people wants them to work again.

  • Upvote 1

Share this post


Link to post
Share on other sites

We all do....I used them until they were broken....

  • Upvote 1

Share this post


Link to post
Share on other sites

if anyone can tell me the revision number where they were removed I'd be able to understand the reasons behind that.

Share this post


Link to post
Share on other sites

it appears ultramage removed simply because they werent official, that code however won't work anymore.


Since the interest for these features are huge I'd kindly ask anyone whos interested to redesign (theoretically only -- we'll code it later) the commands in any manner they find efficient and present it in a new topic for discussion

  • Upvote 3

Share this post


Link to post
Share on other sites

+1 I'd like to see this working again as well (and after reading the bug report, so do others).

Share this post


Link to post
Share on other sites

*Intentionally "bump"s into the topic*

Just wondering if anyone was working on the theoretical design before going afk? If not, I can give it ago?

no one has shown interest in doing it until now, go ahead.

Share this post


Link to post
Share on other sites

Making the diff up to date is definitely is a good idea, but I think we should not merge it unless the community demands it.

Share this post


Link to post
Share on other sites

According to the bug report people are asking for it

And in my opinion many people wants them to work again.

^ +1

Share this post


Link to post
Share on other sites

And in my opinion many people wants them to work again.

^ +1

^also gets my +1

Share this post


Link to post
Share on other sites

Implemented a basic form of it in 2cee5b6. It's open for suggestions on expanding it further.

  • Upvote 1

Share this post


Link to post
Share on other sites

That was actually the reason behind me working on the unit control stuff again. I thought it was a cool instance! I worked on all of this over a year ago and even wrote the script for it. It sort of slid away as more pressing things needed to be resolved and no one ever had any super official information for it. I just need to confirm a few things but as a whole it's pretty much done.

Share this post


Link to post
Share on other sites

That's awesome! Can't wait to see it running.

Share this post


Link to post
Share on other sites

That was actually the reason behind me working on the unit control stuff again. I thought it was a cool instance! I worked on all of this over a year ago and even wrote the script for it. It sort of slid away as more pressing things needed to be resolved and no one ever had any super official information for it. I just need to confirm a few things but as a whole it's pretty much done.

 

setunitdata command

Parameter (indexes) for NPCs

	case BL_NPC:
		if (!md) {
			ShowWarning("buildin_setunitdata: Error in finding object BL_NPC!\n");
			return SCRIPT_CMD_FAILURE;
		}


should be

	case BL_NPC:
		if (!nd) {
			ShowWarning("buildin_setunitdata: Error in finding object BL_NPC!\n");
			return SCRIPT_CMD_FAILURE;
		}

Share this post


Link to post
Share on other sites

setunitdata command

Parameter (indexes) for NPCs

	case BL_NPC:
		if (!md) {
			ShowWarning("buildin_setunitdata: Error in finding object BL_NPC!\n");
			return SCRIPT_CMD_FAILURE;
		}


should be

	case BL_NPC:
		if (!nd) {
			ShowWarning("buildin_setunitdata: Error in finding object BL_NPC!\n");
			return SCRIPT_CMD_FAILURE;
		}

thank you, fixed in b71ed058

Share this post


Link to post
Share on other sites