Players should be able to have multiple units

CheeseLord / warts

WAcky Real Time Strategy

MIT License

1 stars 0 forks source link

Players should be able to have multiple units #16

Closed kronmillerg closed 7 years ago

kronmillerg commented 7 years ago

Things that need to be done for this:

messages need to support: create new unit for player, delete unit (?), issue order to set of units
client needs to have controls corresponding to each of those messages. Probably handle the last one via "select all" and "deselect all" controls for now. Ultimately, need some way to select an arbitrary subset.
server needs to handle those messages
Remove all calls to playerToUnit

kronmillerg commented 7 years ago

Plan of attack:

Have the client request two units initially. Control each with one of the mouse buttons -- so left-clicking orders the first one and right-clicking orders the second one.
Add real support in the server for multiple units. Test it using the client functionality mentioned above.
- This will probably require fixing #12
Improve the client's UI to something more real -- maybe support selecting units or something like that.

kronmillerg commented 7 years ago

This issue cannot be closed until we've eradicated all calls to playerToUnit. Among other things, of course.

kronmillerg commented 7 years ago

The definition of playerToUnit is gone now, and all calls have been removed except for one that's commented out in centerViewOnSelf. That method is dead anyway, so as far as I'm concerned we've met our goal of "eradicate all calls to playerToUnit".

kronmillerg commented 7 years ago

There's plenty more to do with the UI for controlling multiple units, but very little of it is really necessary before we can close off this issue. (We should file separate issues for those other things.)

At this point I think the only thing left to do before we can close this issue is to provide some method through the GUI of adding and removing units. (That is, add controls for order_new and order_del.)