Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

non commandable units #23

Merged
merged 2 commits into from Aug 7, 2014
Merged

Conversation

Copy link
Contributor

@titison titison commented Jul 28, 2014

Added a boolean to units.

commandable | if this is false, you cant give the units commands.

This is mainly made for the spawn attack, because you may want for example spawn bees in a beehive which you are not able to select or control. I also recommend to accept my other pull request about the spawn attack together with this one.

New Syntax:

<unit>
    <parameters>
                ...
        <time value="25"/>  
        <multi-selection value="true"/>
        <commandable value="false"/>        
        <cellmap value="false"/>
        <levels/>
                ...
    </parameters>
...
</unit>

1. selectable	| if this is false, you cant select the unit.
2. commandable	| if this is false, you cant give the units commands.

I advise to set a non-selectable unit to non-commandable too if you want to make a normal unit morph into it.
@titison
Copy link
Contributor Author

@titison titison commented Jul 29, 2014

Dont add this yet pls.
I may have spotted a bug if you try to attack those units. I need to check this.

EDIT: I checked it. Everything is alright.

@titison
Copy link
Contributor Author

@titison titison commented Aug 5, 2014

Spotted another mistake. The AI can still use thos units. Need to fix that.

@titison titison changed the title non selectable and non commandable units non commandable units Aug 7, 2014
titiger added a commit that referenced this issue Aug 7, 2014
@titiger titiger merged commit e5d2bbb into MegaGlest:develop Aug 7, 2014
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants