MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002485VCMIGUI - Adventure Mappublic2016-09-19 00:412016-09-19 00:43
ReporterSXX 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusnewResolutionopen 
PlatformOSOS Version
Product Version0.98g 
Target VersionFixed in Version 
Summary0002485: CGarrisonWindow need improvement for CGGarrison with removableUnits = false
DescriptionI fixed server-side for locked CGGarrison (0002303), but client-side code still lacking. Checks GUI have currently messed up and not let you do a lot of things you should be able to do.

If you just patch this code:
 new CGarrisonWindow(up,down,removableUnits);

With this:
new CGarrisonWindow(up,down,true);

Then all checks will be done on server only and you'll see what should be allowed and what's not.

Conecept is simple:

- You can arrange stacks inside locked garrison as you want: split, merge, swap.
- You can also add troops to garrison use any possible mechanic except direct swap (since I don't want to compare stacks).

Of course there is no way something can be taken out of garrison.
TagsNo tags attached.
Attached Files

- Relationships
related to 0002303resolvedSXX You can take the troops from the garrison with "Troops are removable" unchecked 

-  Notes
(0006689)
SXX (administrator)
2016-09-19 00:43

So of course we can just remove client-side checks and do server do it's thing, but then we'll have compains in log which isn't good. So probably we should move checks code in lib instead and use it on both client and server.

- Issue History
Date Modified Username Field Change
2016-09-19 00:41 SXX New Issue
2016-09-19 00:41 SXX Product Version 0.99 => 0.98g
2016-09-19 00:42 SXX Relationship added related to 0002303
2016-09-19 00:43 SXX Note Added: 0006689

Site | Forums | Wiki | Slack | GitHub


Copyright © 2000 - 2020 MantisBT Team
Hosting provided by DigitalOcean