pg_properties.cpp

Go to the documentation of this file.

/*
 * This program source code file is part of KICAD, a free EDA CAD application.
 *
 * Copyright (C) 2020-2023 CERN
 * Copyright (C) 2021-2024 KiCad Developers, see AUTHORS.TXT for contributors.
 * @author Maciej Suminski <[email protected]>
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 3
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along
 * with this program. If not, see <http://www.gnu.org/licenses/>.
 */
 
#include <wx/dc.h>
#include <wx/propgrid/propgrid.h>
 
#include <macros.h>
#include <validators.h>
#include <eda_draw_frame.h>
#include <eda_units.h>
#include <properties/color4d_variant.h>
#include <properties/std_optional_variants.h>
#include <properties/eda_angle_variant.h>
#include <properties/pg_properties.h>
#include <properties/pg_editors.h>
#include <properties/property_mgr.h>
#include <properties/property.h>
#include <string_utils.h>
#include <widgets/color_swatch.h>
 
 
class wxAnyToSTD_OPTIONAL_INT_VARIANTRegistrationImpl : public wxAnyToVariantRegistration
{
public:
 wxAnyToSTD_OPTIONAL_INT_VARIANTRegistrationImpl( wxVariantDataFactory factory )
 : wxAnyToVariantRegistration( factory )
 {
 }
 
public:
 static bool IsSameClass(const wxAnyValueType* otherType)
 {
 return AreSameClasses( *s_instance.get(), *otherType );
 }
 
 static wxAnyValueType* GetInstance()
 {
 return s_instance.get();
 }
 
 virtual wxAnyValueType* GetAssociatedType() override
 {
 return wxAnyToSTD_OPTIONAL_INT_VARIANTRegistrationImpl::GetInstance();
 }
private:
 static bool AreSameClasses(const wxAnyValueType& a, const wxAnyValueType& b)
 {
 return wxTypeId(a) == wxTypeId(b);
 }
 
 static wxAnyValueTypeScopedPtr s_instance;
};
 
 
wxAnyValueTypeScopedPtr wxAnyToSTD_OPTIONAL_INT_VARIANTRegistrationImpl::s_instance( new wxAnyValueTypeImpl<std::optional<int>>() );
 
static wxAnyToSTD_OPTIONAL_INT_VARIANTRegistrationImpl s_wxAnyToSTD_OPTIONAL_INT_VARIANTRegistration( &STD_OPTIONAL_INT_VARIANT_DATA::VariantDataFactory );
 
 
class wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistrationImpl : public wxAnyToVariantRegistration
{
public:
 wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistrationImpl( wxVariantDataFactory factory )
 : wxAnyToVariantRegistration( factory )
 {
 }
 
public:
 static bool IsSameClass(const wxAnyValueType* otherType)
 {
 return AreSameClasses( *s_instance.get(), *otherType );
 }
 
 static wxAnyValueType* GetInstance()
 {
 return s_instance.get();
 }
 
 virtual wxAnyValueType* GetAssociatedType() override
 {
 return wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistrationImpl::GetInstance();
 }
private:
 static bool AreSameClasses(const wxAnyValueType& a, const wxAnyValueType& b)
 {
 return wxTypeId(a) == wxTypeId(b);
 }
 
 static wxAnyValueTypeScopedPtr s_instance;
};
 
 
wxAnyValueTypeScopedPtr wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistrationImpl::s_instance( new wxAnyValueTypeImpl<std::optional<double>>() );
 
static wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistrationImpl s_wxAnyToSTD_OPTIONAL_DOUBLE_VARIANTRegistration( &STD_OPTIONAL_DOUBLE_VARIANT_DATA::VariantDataFactory );
 
 
class wxAnyToEDA_ANGLE_VARIANTRegistrationImpl : public wxAnyToVariantRegistration
{
public:
 wxAnyToEDA_ANGLE_VARIANTRegistrationImpl( wxVariantDataFactory factory )
 : wxAnyToVariantRegistration( factory )
 {
 }
 
public:
 static bool IsSameClass(const wxAnyValueType* otherType)
 {
 return AreSameClasses( *s_instance.get(), *otherType );
 }
 
 static wxAnyValueType* GetInstance()
 {
 return s_instance.get();
 }
 
 virtual wxAnyValueType* GetAssociatedType() override
 {
 return wxAnyToEDA_ANGLE_VARIANTRegistrationImpl::GetInstance();
 }
private:
 static bool AreSameClasses(const wxAnyValueType& a, const wxAnyValueType& b)
 {
 return wxTypeId(a) == wxTypeId(b);
 }
 
 static wxAnyValueTypeScopedPtr s_instance;
};
 
 
wxAnyValueTypeScopedPtr wxAnyToEDA_ANGLE_VARIANTRegistrationImpl::s_instance( new wxAnyValueTypeImpl<EDA_ANGLE>() );
 
static wxAnyToEDA_ANGLE_VARIANTRegistrationImpl s_wxAnyToEDA_ANGLE_VARIANTRegistration( &EDA_ANGLE_VARIANT_DATA::VariantDataFactory );
 
 
class wxAnyToCOLOR4D_VARIANTRegistrationImpl : public wxAnyToVariantRegistration
{
public:
 wxAnyToCOLOR4D_VARIANTRegistrationImpl( wxVariantDataFactory factory )
 : wxAnyToVariantRegistration( factory )
 {
 }
 
public:
 static bool IsSameClass(const wxAnyValueType* otherType)
 {
 return AreSameClasses( *s_instance.get(), *otherType );
 }
 
 static wxAnyValueType* GetInstance()
 {
 return s_instance.get();
 }
 
 virtual wxAnyValueType* GetAssociatedType() override
 {
 return wxAnyToCOLOR4D_VARIANTRegistrationImpl::GetInstance();
 }
private:
 static bool AreSameClasses(const wxAnyValueType& a, const wxAnyValueType& b)
 {
 return wxTypeId(a) == wxTypeId(b);
 }
 
 static wxAnyValueTypeScopedPtr s_instance;
};
 
wxAnyValueTypeScopedPtr wxAnyToCOLOR4D_VARIANTRegistrationImpl::s_instance( new wxAnyValueTypeImpl<KIGFX::COLOR4D>() );
 
static wxAnyToCOLOR4D_VARIANTRegistrationImpl s_wxAnyToCOLOR4D_VARIANTRegistration( &COLOR4D_VARIANT_DATA::VariantDataFactory );
 
 
wxPGProperty* PGPropertyFactory( const PROPERTY_BASE* aProperty, EDA_DRAW_FRAME* aFrame )
{
 wxPGProperty* ret = nullptr;
 PROPERTY_DISPLAY display = aProperty->Display();
 
 switch( display )
 {
 case PROPERTY_DISPLAY::PT_SIZE:
 ret = new PGPROPERTY_SIZE( aFrame );
 ret->SetEditor( PG_UNIT_EDITOR::BuildEditorName( aFrame ) );
 break;
 
 case PROPERTY_DISPLAY::PT_AREA:
 ret = new PGPROPERTY_AREA( aFrame );
 ret->SetEditor( PG_UNIT_EDITOR::BuildEditorName( aFrame ) );
 break;
 
 case PROPERTY_DISPLAY::PT_COORD:
 ret = new PGPROPERTY_COORD( aFrame, aProperty->CoordType() );
 ret->SetEditor( PG_UNIT_EDITOR::BuildEditorName( aFrame ) );
 break;
 
 case PROPERTY_DISPLAY::PT_DECIDEGREE:
 case PROPERTY_DISPLAY::PT_DEGREE:
 {
 PGPROPERTY_ANGLE* prop = new PGPROPERTY_ANGLE();
 
 if( display == PROPERTY_DISPLAY::PT_DECIDEGREE )
 prop->SetScale( 10.0 );
 
 ret = prop;
 ret->SetEditor( PG_UNIT_EDITOR::BuildEditorName( aFrame ) );
 break;
 }
 
 case PROPERTY_DISPLAY::PT_RATIO:
 ret = new PGPROPERTY_RATIO();
 break;
 
 default:
 wxFAIL;
 KI_FALLTHROUGH;
 /* fall through */
 case PROPERTY_DISPLAY::PT_DEFAULT:
 {
 // Create a corresponding wxPGProperty
 size_t typeId = aProperty->TypeHash();
 
 // Enum property
 if( aProperty->HasChoices() )
 {
 // I do not know why enum property takes a non-const reference to wxPGChoices..
 ret = new wxEnumProperty( wxPG_LABEL, wxPG_LABEL,
 const_cast<wxPGChoices&>( aProperty->Choices() ) );
 }
 else if( typeId == TYPE_HASH( int ) || typeId == TYPE_HASH( long ) )
 {
 ret = new wxIntProperty();
 }
 else if( typeId == TYPE_HASH( unsigned int ) || typeId == TYPE_HASH( unsigned long ) )
 {
 ret = new wxUIntProperty();
 }
 else if( typeId == TYPE_HASH( float ) || typeId == TYPE_HASH( double ) )
 {
 ret = new wxFloatProperty();
 }
 else if( typeId == TYPE_HASH( bool ) )
 {
 ret = new PGPROPERTY_BOOL();
 }
 else if( typeId == TYPE_HASH( wxString ) )
 {
 ret = new PGPROPERTY_STRING();
 }
 else if( typeId == TYPE_HASH( COLOR4D ) )
 {
 ret = new PGPROPERTY_COLOR4D();
 }
 else
 {
 wxFAIL_MSG( wxString::Format( wxS( "Property %s not supported by PGPropertyFactory" ),
 aProperty->Name() ) );
 ret = new wxPropertyCategory();
 ret->Enable( false );
 }
 break;
 }
 }
 
 if( ret )
 {
 ret->SetLabel( wxGetTranslation( aProperty->Name() ) );
 ret->SetName( aProperty->Name() );
 ret->SetHelpString( wxGetTranslation( aProperty->Name() ) );
 ret->SetClientData( const_cast<PROPERTY_BASE*>( aProperty ) );
 }
 
 return ret;
}
 
 
PGPROPERTY_DISTANCE::PGPROPERTY_DISTANCE( EDA_DRAW_FRAME* aParentFrame,
 ORIGIN_TRANSFORMS::COORD_TYPES_T aCoordType ) :
 m_parentFrame( aParentFrame ),
 m_coordType( aCoordType )
{
}
 
 
PGPROPERTY_DISTANCE::~PGPROPERTY_DISTANCE()
{
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_DISTANCE::StringToDistance( wxVariant& aVariant, const wxString& aText,
 wxPGPropValFormatFlags aFlags ) const
#else
bool PGPROPERTY_DISTANCE::StringToDistance( wxVariant& aVariant, const wxString& aText,
 int aFlags ) const
#endif
{
 // TODO(JE): Are there actual use cases for this?
 wxCHECK_MSG( false, false, wxS( "PGPROPERTY_DISTANCE::StringToDistance should not be used." ) );
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_DISTANCE::DistanceToString( wxVariant& aVariant,
 wxPGPropValFormatFlags aFlags ) const
#else
wxString PGPROPERTY_DISTANCE::DistanceToString( wxVariant& aVariant, int aFlags ) const
#endif
{
 long distanceIU;
 
 if( aVariant.GetType() == wxT( "std::optional<int>" ) )
 {
 auto* variantData = static_cast<STD_OPTIONAL_INT_VARIANT_DATA*>( aVariant.GetData() );
 
 if( !variantData->Value().has_value() )
 return wxEmptyString;
 
 distanceIU = variantData->Value().value();
 }
 else if( aVariant.GetType() == wxPG_VARIANT_TYPE_LONG )
 {
 distanceIU = aVariant.GetLong();
 }
 else
 {
 wxFAIL_MSG( wxT( "Expected int (or std::optional<int>) value type" ) );
 return wxEmptyString;
 }
 
 ORIGIN_TRANSFORMS& transforms = m_parentFrame->GetOriginTransforms();
 
 distanceIU = transforms.ToDisplay( static_cast<long long int>( distanceIU ), m_coordType );
 
 return m_parentFrame->StringFromValue( distanceIU, true, EDA_DATA_TYPE::DISTANCE );
}
 
 
PGPROPERTY_AREA::PGPROPERTY_AREA( EDA_DRAW_FRAME* aParentFrame ) :
 wxIntProperty( wxPG_LABEL, wxPG_LABEL, 0 ),
 m_parentFrame( aParentFrame )
{
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_AREA::StringToValue( wxVariant& aVariant, const wxString& aText,
 wxPGPropValFormatFlags aArgFlags ) const
#else
bool PGPROPERTY_AREA::StringToValue( wxVariant& aVariant, const wxString& aText,
 int aArgFlags ) const
#endif
{
 // TODO(JE): Are there actual use cases for this?
 wxCHECK_MSG( false, false, wxS( "PGPROPERTY_AREA::StringToValue should not be used." ) );
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_AREA::ValueToString( wxVariant& aVariant,
 wxPGPropValFormatFlags aArgFlags ) const
#else
wxString PGPROPERTY_AREA::ValueToString( wxVariant& aVariant, int aArgFlags ) const
#endif
{
 wxCHECK( aVariant.GetType() == wxPG_VARIANT_TYPE_LONGLONG, wxEmptyString );
 
 wxLongLongNative areaIU = aVariant.GetLongLong();
 
 return m_parentFrame->StringFromValue( areaIU.ToDouble(), true, EDA_DATA_TYPE::AREA );
}
 
 
wxValidator* PGPROPERTY_AREA::DoGetValidator() const
{
 return nullptr;
}
 
 
PGPROPERTY_SIZE::PGPROPERTY_SIZE( EDA_DRAW_FRAME* aParentFrame ) :
 wxUIntProperty( wxPG_LABEL, wxPG_LABEL, 0 ),
 PGPROPERTY_DISTANCE( aParentFrame, ORIGIN_TRANSFORMS::NOT_A_COORD )
{
}
 
 
bool PGPROPERTY_SIZE::ValidateValue( wxVariant& aValue, wxPGValidationInfo& aValidationInfo ) const
{
 if( aValue.GetType() == wxT( "std::optional<int>" ) )
 {
 auto* data = static_cast<STD_OPTIONAL_INT_VARIANT_DATA*>( aValue.GetData() );
 
 if( !data->Value().has_value() )
 return wxEmptyString;
 
 wxVariant value( data->Value().value() );
 return wxUIntProperty::ValidateValue( value, aValidationInfo );
 }
 
 return wxUIntProperty::ValidateValue( aValue, aValidationInfo );
}
 
 
wxValidator* PGPROPERTY_SIZE::DoGetValidator() const
{
 return nullptr;
}
 
 
PGPROPERTY_COORD::PGPROPERTY_COORD( EDA_DRAW_FRAME* aParentFrame,
 ORIGIN_TRANSFORMS::COORD_TYPES_T aCoordType ) :
 wxIntProperty( wxPG_LABEL, wxPG_LABEL, 0 ),
 PGPROPERTY_DISTANCE( aParentFrame, aCoordType )
{
}
 
 
wxValidator* PGPROPERTY_COORD::DoGetValidator() const
{
 return nullptr;
}
 
 
PGPROPERTY_RATIO::PGPROPERTY_RATIO() :
 wxFloatProperty( wxPG_LABEL, wxPG_LABEL, 0 )
{
 SetEditor( PG_RATIO_EDITOR::EDITOR_NAME );
}
 
 
const wxPGEditor* PGPROPERTY_RATIO::DoGetEditorClass() const
{
 wxCHECK_MSG( m_customEditor, wxPGEditor_TextCtrl,
 wxT( "Make sure to RegisterEditorClass() for PGPROPERTY_RATIO!" ) );
 return m_customEditor;
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_RATIO::StringToValue( wxVariant& aVariant, const wxString& aText,
 wxPGPropValFormatFlags aArgFlags ) const
#else
bool PGPROPERTY_RATIO::StringToValue( wxVariant& aVariant, const wxString& aText,
 int aArgFlags ) const
#endif
{
 // TODO(JE): Are there actual use cases for this?
 wxCHECK_MSG( false, false, wxS( "PGPROPERTY_RATIO::StringToValue should not be used." ) );
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_RATIO::ValueToString( wxVariant& aVariant,
 wxPGPropValFormatFlags aArgFlags ) const
#else
wxString PGPROPERTY_RATIO::ValueToString( wxVariant& aVariant, int aArgFlags ) const
#endif
{
 double value;
 
 if( aVariant.GetType() == wxT( "std::optional<double>" ) )
 {
 auto* variantData = static_cast<STD_OPTIONAL_DOUBLE_VARIANT_DATA*>( aVariant.GetData() );
 
 if( !variantData->Value().has_value() )
 return wxEmptyString;
 
 value = variantData->Value().value();
 }
 else if( aVariant.GetType() == wxPG_VARIANT_TYPE_DOUBLE )
 {
 value = aVariant.GetDouble();
 }
 else
 {
 wxFAIL_MSG( wxT( "Expected double (or std::optional<double>) value type" ) );
 return wxEmptyString;
 }
 
 return wxString::Format( wxS( "%g" ), value );
}
 
 
bool PGPROPERTY_RATIO::ValidateValue( wxVariant& aValue, wxPGValidationInfo& aValidationInfo ) const
{
 if( aValue.GetType() == wxT( "std::optional<double>" ) )
 {
 auto* data = static_cast<STD_OPTIONAL_DOUBLE_VARIANT_DATA*>( aValue.GetData() );
 
 if( !data->Value().has_value() )
 return wxEmptyString;
 
 wxVariant value( data->Value().value() );
 return wxFloatProperty::ValidateValue( value, aValidationInfo );
 }
 
 return wxFloatProperty::ValidateValue( aValue, aValidationInfo );
}
 
 
wxValidator* PGPROPERTY_RATIO::DoGetValidator() const
{
 return nullptr;
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_ANGLE::StringToValue( wxVariant& aVariant, const wxString& aText,
 wxPGPropValFormatFlags aArgFlags ) const
#else
bool PGPROPERTY_ANGLE::StringToValue( wxVariant& aVariant, const wxString& aText,
 int aArgFlags ) const
#endif
{
 double value = 0.0;
 
 if( !aText.ToDouble( &value ) )
 {
 aVariant.MakeNull();
 return true;
 }
 
 value *= m_scale;
 
 if( aVariant.IsNull() || aVariant.GetDouble() != value )
 {
 aVariant = value;
 return true;
 }
 
 return false;
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_ANGLE::ValueToString( wxVariant& aVariant,
 wxPGPropValFormatFlags aArgFlags ) const
#else
wxString PGPROPERTY_ANGLE::ValueToString( wxVariant& aVariant, int aArgFlags ) const
#endif
{
 if( aVariant.GetType() == wxPG_VARIANT_TYPE_DOUBLE )
 {
 // TODO(JE) Is this still needed?
 return wxString::Format( wxS( "%g\u00B0" ), aVariant.GetDouble() / m_scale );
 }
 else if( aVariant.GetType() == wxS( "EDA_ANGLE" ) )
 {
 wxString ret;
 static_cast<EDA_ANGLE_VARIANT_DATA*>( aVariant.GetData() )->Write( ret );
 return ret;
 }
 else
 {
 wxCHECK_MSG( false, wxEmptyString, wxS( "Unexpected variant type in PGPROPERTY_ANGLE" ) );
 }
}
 
 
wxValidator* PGPROPERTY_ANGLE::DoGetValidator() const
{
 return nullptr;
}
 
 
wxSize PGPROPERTY_COLORENUM::OnMeasureImage( int aItem ) const
{
 // TODO(JE) calculate size from window metrics?
 return wxSize( 16, 12 );
}
 
 
void PGPROPERTY_COLORENUM::OnCustomPaint( wxDC& aDC, const wxRect& aRect,
 wxPGPaintData& aPaintData )
{
 int index = aPaintData.m_choiceItem;
 
 if( index < 0 )
 index = GetIndex();
 
 // GetIndex can return -1 when the control hasn't been set up yet
 if( index < 0 || index >= static_cast<int>( GetChoices().GetCount() ) )
 return;
 
 wxColour color = GetColor( GetChoices().GetValue( index ) );
 
 if( color == wxNullColour )
 return;
 
 aDC.SetPen( *wxTRANSPARENT_PEN );
 aDC.SetBrush( wxBrush( color ) );
 aDC.DrawRectangle( aRect );
 
 aPaintData.m_drawnWidth = aRect.width;
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_STRING::ValueToString( wxVariant& aValue,
 wxPGPropValFormatFlags aFlags ) const
#else
wxString PGPROPERTY_STRING::ValueToString( wxVariant& aValue, int aFlags ) const
#endif
{
 if( aValue.GetType() != wxPG_VARIANT_TYPE_STRING )
 return wxEmptyString;
 
 return UnescapeString( aValue.GetString() );
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_STRING::StringToValue( wxVariant& aVariant, const wxString& aString,
 wxPGPropValFormatFlags aArgFlags ) const
#else
bool PGPROPERTY_STRING::StringToValue( wxVariant& aVariant, const wxString& aString,
 int aFlags ) const
#endif
{
 aVariant = EscapeString( aString, CTX_QUOTED_STR );
 return true;
}
 
 
PGPROPERTY_BOOL::PGPROPERTY_BOOL( const wxString& aLabel, const wxString& aName, bool aValue ) :
 wxBoolProperty( aLabel, aName, aValue )
{
 SetEditor( PG_CHECKBOX_EDITOR::EDITOR_NAME );
}
 
 
const wxPGEditor* PGPROPERTY_BOOL::DoGetEditorClass() const
{
 wxCHECK_MSG( m_customEditor, wxPGEditor_CheckBox,
 wxT( "Make sure to RegisterEditorClass() for PGPROPERTY_BOOL!" ) );
 return m_customEditor;
}
 
 
PGPROPERTY_COLOR4D::PGPROPERTY_COLOR4D( const wxString& aLabel, const wxString& aName,
 COLOR4D aValue, COLOR4D aBackgroundColor ) :
 wxStringProperty( aLabel, aName, aValue.ToCSSString() ),
 m_backgroundColor( aBackgroundColor )
{
 SetEditor( PG_COLOR_EDITOR::EDITOR_NAME );
#if wxCHECK_VERSION( 3, 3, 0 )
 SetFlag( wxPGPropertyFlags::NoEditor );
#else
 SetFlag( wxPG_PROP_NOEDITOR );
#endif
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
bool PGPROPERTY_COLOR4D::StringToValue( wxVariant& aVariant, const wxString& aString,
 wxPGPropValFormatFlags aArgFlags ) const
#else
bool PGPROPERTY_COLOR4D::StringToValue( wxVariant& aVariant, const wxString& aString,
 int aFlags ) const
#endif
{
 aVariant.SetData( new COLOR4D_VARIANT_DATA( aString ) );
 return true;
}
 
 
#if wxCHECK_VERSION( 3, 3, 0 )
wxString PGPROPERTY_COLOR4D::ValueToString( wxVariant& aValue,
 wxPGPropValFormatFlags aFlags ) const
#else
wxString PGPROPERTY_COLOR4D::ValueToString( wxVariant& aValue, int aFlags ) const
#endif
{
 wxString ret;
 
 if( aValue.IsType( wxS( "COLOR4D" ) ) )
 static_cast<COLOR4D_VARIANT_DATA*>( aValue.GetData() )->Write( ret );
 else
 return wxStringProperty::ValueToString( aValue, aFlags );
 
 return ret;
}