|Anonymous | Login||2020-01-24 23:10 UTC|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0002485||VCMI||GUI - Adventure Map||public||2016-09-19 00:41||2016-09-19 00:43|
|Target Version||Fixed in Version|
|Summary||0002485: CGarrisonWindow need improvement for CGGarrison with removableUnits = false|
|Description||I 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:
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.
|Tags||No tags attached.|
|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.|
|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|
|Copyright © 2000 - 2020 MantisBT Team|