Need help with the "RemoveLocation"...

[brusz]

First i want to know if this is correct:

With the 1

• Trigger
• Events
  • Unit - Unit did something...
• Conditions
  • ...
• Actions
  • Set Pos[(Player number of (Owner of (Triggering unit)))] = (Position of (Triggering unit))
  • Something happens with Pos...
  • Custom script: call RemoveLocation(udg_Pos[GetPlayerId( GetOwningPlayer(GetTriggerUnit())) + 1])

Or does it need to be like this:

Without the 1

• Trigger
• Events
  • Unit - Unit did something...
• Conditions
  • ...
• Actions
  • Set Pos[(Player number of (Owner of (Triggering unit)))] = (Position of (Triggering unit))
  • Something happens with Pos...
  • Custom script: call RemoveLocation(udg_Pos[GetPlayerId( GetOwningPlayer(GetTriggerUnit()))])

The other question is: is there a way to detect a "buying unit"?
Like the "GetTriggerUnit()" but with buying unit instead of triggering unit...

I need to "remove" a location but with the array number being the number of the player that owns the buying unit...

This is part of the trigger:

Trigger with the selling and buying unit

• Trigger
• Events
  • Unit - A unit Sells an item (from shop)
• Conditions
  • (Item-type of (Sold Item)) Equal to Item
• Actions
  • Set BuyerPos[(Player number of (Owner of (Buying unit)))] = (Position of (Buying unit))
  • Something happens with BuyerPos...
  • Custom script: call RemoveLocation(udg_BuyerPos[GetPlayerId( GetOwningPlayer(Get?????Unit()))])

What goes on the ????? and is the "+1" needed?

 

[MortAr]

1. Remove the +1
2. Yes, GetOwningPlayer(GetBuyingUnit())

 

[PurgeandFire]

You need the +1 because "Player Number" in GUI is actually ranged from 1-16, and uses this function:

function GetConvertedPlayerId takes player whichPlayer returns integer
    return GetPlayerId(whichPlayer) + 1
endfunction

GetPlayerId() returns values from 0-15.

So the +1 is required to remove the correct indexed location.

 

[brusz]

So i must use "GetBuyingUnit()" and add the "+1" because im using GUI right?

Thanks for the help, added the rep for your replies.

---------------------------------------------------------------------------

EDIT*

hmm what about the "Picked Player"?

instead of "GetOwningPlayer()" is "GetPickedPlayer()" ???