Hardcoded column name in Colorize nodes

[Adriano Faria]

I'm trying to use the Colorize Node feature in my apps but I'm getting on streams:

Quote

SELECT `rc_id`, `rc_name_furl`, `feature_color` FROM `raffles_categories` WHERE ( rc_id IN(1) )
IPS\Db\Exception: Unknown column 'feature_color' in 'field list' (1054)

That's because you hardcoded the column name in IPS\Node\Colorize to be used in FORUMS app only.

/**
 * Colorize Trait
 */
trait Colorize
{
	/**
	 * @brief	Table column name that holds the feature color hex
	 */
	public static $featureColumnName = 'feature_color';

If I rename my column to remove the table prefix, then it works:

But that's not right. It should've been done like reaction, where you pass the ID.

[bfarber]

Thanks, I've brought this up in our internal tracker

[Adriano Faria]

On 10/10/2017 at 10:10 AM, bfarber said:

Thanks, I've brought this up in our internal tracker

Brandon, as it isn't on 4.2.6 (at least until Beta 3), please make sure it will be on 4.3.

Thank you.

[Adriano Faria]

On 10/10/2017 at 10:10 AM, bfarber said:

Thanks, I've brought this up in our internal tracker

Brandon, just a remind to change this in 4.3.

[Adriano Faria]

Still hardcoded in 4.3. ?

[newbie LAC]

It would be better to open a ticket than wait.

[Midnight Modding]

That's not really a big deal. Sure it's not the best setup, but it's minor compared to other areas I run into, such as methods where there's no way to do something without totally overwriting a large method, such as during the content item creation process.

[newbie LAC]

This has been fixed in 4.3.0. Mark my post as best answer