Invalid Colour Name

Aggregate · January 4, 2018, 8:09am

I’m attempting to apply colour to a monster but getting the following error:

I literally just copy-pasted it from COLOR.md, as far as I know it should work. I’ve also had problems with, I think, all of the colour combinations I’ve tried. Assistance would be appreciated.

Also, is there a way to stop COLOR.md from being turned into a link?

Not quite related, but is there a way to give a monster a reusable flashbang, akin to the Flashbang Generator CBM?

NuG · January 4, 2018, 5:18pm

Colors were changed or something recently, I think light was changed to lt and dark to dk(better info in links below).
Error with some Mods is where I just got my info and/or
https://github.com/CleverRaven/Cataclysm-DDA/pull/22535 might be relevant .

Aggregate · January 4, 2018, 10:02pm

They were actually changed from lt_ to light_ etc, and I’m fairly sure that the “COLOR.md” file was added around that time.

Zhilkin · January 4, 2018, 10:54pm

invalid color name error will be thrown when color_from_string function returns c_unset.

Aggregate · January 4, 2018, 11:03pm

Right, nice to know why the error’s thrown (even if I’m not code savvy enough to actually understand it), but it should be working. There are even existing working examples; the zombie hulk and dancer use “color”: “white_magenta”,.

Zhilkin · January 4, 2018, 11:05pm

I agree, but it seems error is thrown for some valid combinations of foreground and background color.

Zhilkin · January 4, 2018, 11:36pm

It seems all possible color combinations are hardcoded.

You can see them in Settings - Colors menu or in color_manager::load_default() function:

github.com

CleverRaven/Cataclysm-DDA/blob/master/src/color.cpp#L207


if( bg_name.empty() ) {  //c_black -> h_black
    hi_name = "h_" + name.substr( 2, name.length() - 2 );
} else {
    hi_name = name + "_" + bg_name;
}


color_id id = name_to_id( hi_name );
return color_array[id].color;
}


void color_manager::load_default()
{
static const auto color_pair = []( const int n ) {
    return nc_color::from_color_pair_index( n );
};
//        Color         Name      Color Pair      Invert
add_color( def_c_black, "c_black", color_pair( 30 ), def_i_black );
add_color( def_c_white, "c_white", color_pair( 1 ).bold(), def_i_white );
add_color( def_c_light_gray, "c_light_gray", color_pair( 1 ), def_i_light_gray );
add_color( def_c_dark_gray, "c_dark_gray", color_pair( 30 ).bold(), def_i_dark_gray );
add_color( def_c_red, "c_red", color_pair( 2 ), def_i_red );

Aggregate · January 5, 2018, 1:10am

That’s unfortunate. Still, thank you for the assistance.

Topic		Replies	Views
Changing colors breaks game loading The Garage	5	674	July 3, 2019
Error with some Mods The Lab	9	2307	December 5, 2017
(R.I.P. OSX) The colour "issue" in recent experimentals The Bunker	5	663	September 28, 2018
Changing background color The Garage	9	312	July 5, 2013
Change Back to Previos Color of Unseen/Explored The Bunker	3	686	November 11, 2018

Invalid Colour Name

Related Topics