Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002249Zandronum[All Projects] Bugpublic2015-05-24 18:062018-09-30 22:26
ReporterStrikerMan780 
Assigned ToDusk 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version3.0-beta 
Target Version3.0Fixed in Version3.0 
Summary0002249: [MENUDEF] NumberFields don't update correctly.
DescriptionQuoting from a post I made earlier: Changing the Skill and Botskill in the Skirmish menu changes the value of the skills, but it doesn't update in the menu. Ie. If botskill was initially at 1, and you click on it or hit the right arrow, the skill will change to 2 internally, but the value displayed in the menu still says 1.
Attached Files

- Relationships

-  Notes
User avatar (0012377)
Torr Samaho (administrator)
2015-05-25 08:23

 Dusk, you implemented the NumberField, didn't you? Can you look into this?
User avatar (0012378)
Dusk (developer)
2015-05-25 11:34
edited on: 2015-05-25 11:45

 I think I know what's causing this. Skill and botskill are latched CVars so changing their values does not take immediate effect when changed from in-game. I'll have to make it use temporary menu CVars as a workaround.

(Now that I think of it, having the skirmish menu set skill and botskill directly was a rather bad idea in retrospect. Oh well.)
User avatar (0012379)
Dusk (developer)
2015-05-25 11:44

 Fixed
User avatar (0012382)
cobalt (updater)
2015-05-25 12:42

 Issue addressed by commit e0a5057c3829: Do not change skill and botskill directly in the skirmish menu, instead set temporary CVars that take effect when the skirmish starts (fixes 2249).
Committed by Teemu Piippo [Dusk] on Monday 25 May 2015 14:43:45

Changes in files:

 src/menu/multiplayermenu.cpp | 11 +++++++++++
 wadsrc/static/menudef.za | 4 ++--
 2 files changed, 13 insertions(+), 2 deletions(-)

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2015-05-24 18:06 StrikerMan780 New Issue
2015-05-24 18:48 Dusk Product Version => 3.0
2015-05-24 18:48 Dusk Summary [3.0 Alpha][MENUDEF] NumberFields don't update correctly. => [MENUDEF] NumberFields don't update correctly.
2015-05-24 19:37 Dusk Product Version 3.0 => 3.0-beta
2015-05-25 08:23 Torr Samaho Note Added: 0012377
2015-05-25 08:24 Torr Samaho Assigned To => Dusk
2015-05-25 08:24 Torr Samaho Status new => assigned
2015-05-25 11:34 Dusk Note Added: 0012378
2015-05-25 11:44 Dusk Note Added: 0012379
2015-05-25 11:44 Dusk Status assigned => needs review
2015-05-25 11:45 Dusk Note Edited: 0012378 View Revisions
2015-05-25 12:42 cobalt Status needs review => needs testing
2015-05-25 12:42 cobalt Target Version => 3.0
2015-05-25 12:42 cobalt Note Added: 0012382
2015-05-25 12:45 Dusk Status needs testing => resolved
2015-05-25 12:45 Dusk Fixed in Version => 3.0
2015-05-25 12:45 Dusk Resolution open => fixed
2018-09-30 22:26 Blzut3 Status resolved => closed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker