SC2Mapster

I used the "attach model to unit" trigger event to have my hero riding up on the elevator. Now that it's at the top, I use "move unit instantly" to put the actual hero unit there, but I need to get rid of the actor. I have it saved to a variable, but there's no "destroy actor" trigger event. I tried sending it a "destroy" message, but it didn't work. I see all kinds of trigger actions that can create actors, but none that can destroy one. I even tried the "kill actor model" action and it didn't work, either.

Well, perhaps you could use "remove actor"?

I dunno, could be wrong.

Perhaps the dr could help?

There is no “Remove Actor” trigger action that I could find.

Quote from DrSuperEvil >>

Why you not just using a Model type actor?

Attach Raynor to Origin on (Actor for Space Elevator (Terrain object doodad) [125.00, 29.00])
Variable -Set Actor2 = (Last created actor)

Actor -Kill actor model Actor2

Attach Raynor to Origin on (Actor for Space Elevator (Terrain object doodad) [221.00, 29.00])
Variable -Set Actor = (Last created actor)
Actor -Send message "SetTintColor 255,0,255" to actor Actor

 This is what I did, and nothing changed. He should have turned pink, correct?

        Attach Raynor to Origin on (Actor for Space Elevator (Terrain object doodad) [221.00, 29.00])
        Variable -Set Actor = (Last created actor)
        Actor -Send message "SetTintColor 255,0,255" to actor Actor
        General -If (Conditions) then do (Actions) else do (Actions)
            If
                Actor != No Actor
            Then
                UI -Display "True" for (All players) to Subtitle area
            Else
                UI -Display "False" for (All players) to Subtitle area

When I test this it prints "True", so I know the actor is getting assigned.

Could it be because I'm attaching it to a doodad instead of a unit?

EDIT: Honestly I'm starting to wonder if this whole thing is even worth the effort... I'm going to try an idea for a workaround tomorrow, I'll get back to you.

'No actor' pretty much evaluates to null, so it only tests whether the variable holds a reference, but doesn't check whether this reference is valid.

You could use "Actor Get Text" to display information about what that actor reference essentially holds.

Also, if you inspect internal implementation of this function:

actor libNtve_gf_AttachModelToUnit (unit lp_unit, string lp_model, string lp_attachPoint) {
    // Variable Declarations
    actorscope lv_scope;
    actor lv_a;

    // Automatic Variable Declarations
    // Variable Initialization

    // Implementation
    lv_scope = ActorScopeFromUnit(lp_unit);
    ActorCreate(lv_scope, "SiteHosted", lp_attachPoint, "", "");
    lv_a = libNtve_gf_ActorLastCreated();
    ActorCreate(lv_scope, "ModelHosted", (lp_model), "", "");
    lv_a = libNtve_gf_ActorLastCreated();
    return lv_a;
}

 You'll realize that it in facts creates two actors, where one is 'SiteHosted', which is some sort of link I assume (not well aware about Actors). Thus by using LastCreated action you only get the referece to ModelHosted. Although this is the essential one in fact, so it surprising if it doesn't respond to 'Destroy' message.

Anyway, you can also utilize actor cheats to get some more information about currently living actors (what could help finding out the cause), i.e.:
ActorInfoDisplay
ActorLiveDump

Could it be because I'm attaching it to a doodad instead of a unit?

You should not be capable of doing this, since it would not pass type checking, I'm assuming you're using "Attach Model To Actor". Which should actually work, because it has different implementation (if that's not the deprecated version without postfix '2'):

actor libNtve_gf_AttachModelToActor2 (actor lp_actor, string lp_model, string lp_attachPoint) {
    // Variable Declarations
    string lv_message;
    actorscope lv_scope;
    actor lv_a;

    // Automatic Variable Declarations
    // Variable Initialization
    lv_message = ("AttachModel {} ");

    // Implementation
    lv_message = ("AttachModel {} " + (lp_model) + " " + lp_attachPoint);
    ActorSendAsText(lp_actor, StringToText(lv_message));
    lv_a = libNtve_gf_ActorLastCreated();
    return lv_a;
}

 ---

Oh, wait. I just realized the implementation above uses Message to create the actor, as oposed to "ActorCreate" function. Thus you have to use "Last Created Actor by Message" to obtain valid reference in this scenario.