Opened 8 years ago

Closed 7 years ago

#4230 closed defect (fixed)

Options warning: incompatible value for WaterUgly

Reported by: Vladislav Belov Owned by: Imarok
Priority: Should Have Milestone: Alpha 22
Component: UI & Simulation Keywords:
Cc: Patch: Phab:D89

Description (last modified by elexis)

If reset all options to default and ugly water enabled then next warning appears after reloading: warning: Incompatible renderer option value for WaterUgly

It's printed by gui/options/options.js

if (Engine["Renderer_Get" + keyRenderer + "Enabled"]() !== checked)
{
	warn("Incompatible renderer option value for " + keyRenderer);
	Engine["Renderer_Set" + keyRenderer + "Enabled"](checked);
}

I think it happens because () !== checked not inverted for invertedboolean type, but I'm not sure yet.

Change History (5)

comment:1 by Vladislav Belov, 8 years ago

Description: modified (diff)

comment:2 by Vladislav Belov, 8 years ago

It doesn't work only for invertedboolean values. Another issue: if waterugly is enabled (config contains waterugly = true), then game doesn't apply this settings (water isn't ugly) until the options window won't opened.

Last edited 8 years ago by Vladislav Belov (previous) (diff)

comment:3 by elexis, 8 years ago

Milestone: Alpha 21Backlog

No patch, no cookies.

comment:4 by elexis, 7 years ago

Description: modified (diff)
Milestone: BacklogAlpha 22
Patch: Phab:D89

comment:5 by Imarok, 7 years ago

Owner: set to Imarok
Resolution: fixed
Status: newclosed

In 19822:

Fix incompatible value for WaterUgly warning on opening the options screen.

Reviewed by: vladislavbelov, Imarok. Patch by echotangoecho. Fixes: #4230.
Differential Revision: https://code.wildfiregames.com/D89

Note: See TracTickets for help on using tickets.