Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Incorrect count of units belonging to commander #1057

Closed
wzdev-ci opened this issue Nov 9, 2009 · 22 comments
Closed

Incorrect count of units belonging to commander #1057

wzdev-ci opened this issue Nov 9, 2009 · 22 comments
Labels
closed component: other trivial
Milestone
2.3.0

Comments

@wzdev-ci
Copy link
Contributor

wzdev-ci commented Nov 9, 2009

keyword_commander resolution_invalid type_bug | by gundersen@...

I've only used the commander in the campaign, so I don't know if the problem exists in multiplayer as well.

The problem is that when you select a commander from the commander menu (the star in the hexagon in the bottom left), then the commanders have an incorrect number of units assigned to them. It might say 6/14, but when you click the commander, a larger number of units are selected, and they all follow him around.

This might also be the case only if you bring a commander with you on an away mission.

I have no idea how to fix this problem, unfortunately. I might have a look at the source some other time to try to understand it.

Issue migrated from trac:1057 at 2022-04-15 20:03:32 -0700
@wzdev-ci
Copy link
Contributor Author

Zarel changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

Zarel set resolution to duplicate

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Duplicate of #1248.

@wzdev-ci
Copy link
Contributor Author

Buginator changed milestone from 2.2.5 to 2.3

@wzdev-ci
Copy link
Contributor Author

Buginator commented

Milestone 2.2.5 deleted

@wzdev-ci
Copy link
Contributor Author

Emdek changed resolution from duplicate to ``

@wzdev-ci
Copy link
Contributor Author

Emdek changed status from closed to reopened

@wzdev-ci
Copy link
Contributor Author

Emdek commented

Reopening, still happens in 2.3.0.

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Screenshot would be nice.

@wzdev-ci
Copy link
Contributor Author

anonymous uploaded file wz-commander-bug.png (3158.1 KiB)

wz-commander-bug.png

@wzdev-ci
Copy link
Contributor Author

Emdek uploaded file a06.tar.bz2 (175.7 KiB)

@wzdev-ci
Copy link
Contributor Author

Zarel commented

I don't see anything wrong with the screenshot. Five attached units, four attached artillery; matches the commander exactly.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Apr 26, 2010 •
edited

Emdek commented

Replying to Warzone2100/old-trac-import#1057 (comment:5):

I don't see anything wrong with the screenshot. Five attached units, four attached artillery; matches the commander exactly.
But another commander also has mortars and they are counted.
It is at least inconsistent...

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Erm, give me one screenshot (from 2.3) in which mortars are counted...

@wzdev-ci
Copy link
Contributor Author

Emdek uploaded file wz-commander-mortars.jpg (412.3 KiB)

Commander with mortars
wz-commander-mortars.jpg

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Are you sure there aren't non-mortar units off-screen?

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Apr 26, 2010 •
edited

Emdek commented

Replying to Warzone2100/old-trac-import#1057 (comment:8):

Are you sure there aren't non-mortar units off-screen?
Oops, some locked in jam in entrance...

But it's a bit confusing anyway, couldn't it be somehow shown in the list? Something like 5/10 + n (or maybe even more clear)?

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Yes, I could do something like that...

@wzdev-ci
Copy link
Contributor Author

Zarel changed status from reopened to closed

@wzdev-ci
Copy link
Contributor Author

Zarel changed resolution from `` to invalid

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Apr 26, 2010 •
edited

Emdek commented

Replying to Warzone2100/old-trac-import#1057 (comment:10):

Yes, I could do something like that...
So maybe it should have own ticket or topic in ideas and suggestions (maybe someone will have better idea for visualization) to be not forgotten...

@wzdev-ci
Copy link
Contributor Author

Zarel commented

Feel free to start that Ideas thread.

@wzdev-ci wzdev-ci mentioned this issue Apr 16, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
closed component: other trivial
Projects
None yet
Milestone
2.3.0
Development

No branches or pull requests
1 participant
@wzdev-ci