0002512: cvars set with consolecommand conflict with cvarinfo

MantisBT - Zandronum

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0002512

Zandronum

[All Projects] Bug

public

2015-11-04 03:34

2018-09-30 22:03

Reporter

WaTaKiD

Assigned To

Torr Samaho

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

fixed

Platform

Microsoft

Windows

OS Version

XP/Vista/7

Product Version

3.0-beta

Target Version

3.0

Fixed in Version

3.0

Summary

0002512: cvars set with consolecommand conflict with cvarinfo

Description

before zan supported cvarinfo, mod cvars were set/saved using consolecommand, and they would be placed in the ini under [Doom.UnknownConsoleVariables]

however when zan did start to support cvarinfo, zan wouldnt load due to these cvars already existing, as cvarinfo puts them under [Doom.NetServerInfo.Mod]

thus users had to go into their ini and remove the cvars from [Doom.UnknownConsoleVariables] in order to play the mod

Steps To Reproduce

load up zan 1.3 with a fresh config and'http://www.best-ever.org/download?file=metroiddreadnought-v1.pk3 [^]'

now load up zan 2.x or 3.x using that same config and wad

it should error and say a cvar already exists

Additional Information

with 3.0-151004-2012 containing the backports mentioned here:'http://zandronum.com/forum/showthread.php?tid=5134&pid=71751#pid85895 [^]'

i was able to start a server on tspg-painkiller using a config that had just:
set metroid_permabfg 1

whereas in 2.x the server wouldnt start, so thats definitely an improvement, however a client's ini with these cvars under [Doom.UnknownConsoleVariables] is still prone to the error

at the very least if there isnt a suitable solution to this, perhaps a message could be printed instructing users to clear such cvars from their ini, as suggested here:'http://zandronum.com/forum/showthread.php?tid=5134&pid=85858#pid85858 [^]'