Merge pull request #90 from ldmoser/setPresets

andrewkaufman · andrewkaufman · commit 219ffb4b2f2c · 2013-08-26T14:54:47.000-07:00
Adding setPresets() to Parameter class.
diff --git a/include/IECore/CompoundParameter.h b/include/IECore/CompoundParameter.h
@@ -1,6 +1,6 @@
 //////////////////////////////////////////////////////////////////////////
 //
-//  Copyright (c) 2007-2012, Image Engine Design Inc. All rights reserved.
+//  Copyright (c) 2007-2013, Image Engine Design Inc. All rights reserved.
 //
 //  Redistribution and use in source and binary forms, with or without
 //  modification, are permitted provided that the following conditions are
@@ -71,11 +71,17 @@ class CompoundParameter : public Parameter
 		/// of all the child objects.
 		/// \threading It is not safe to call this from multiple concurrent threads.
 		virtual const Object *defaultValue() const;
+		/// \deprecated Use getPresets instead.
+		virtual const PresetsContainer &presets() const;
 		/// If true was passed to adoptChildPresets at construction, then update the presets
 		/// with the intersection of the presets of all the child parameters, otherwise returns
-		/// an empty container. Please note that the map returned may differ between one call
+		/// an empty container or the presets defined by setPresets(). 
+		/// Please note that the map returned may differ between one call
 		/// to presets() and the next.
-		virtual const PresetsContainer &presets() const;
+		virtual const PresetsContainer &getPresets() const;
+		/// Defines presets for this Parameter.
+		/// Throws an exception if true was passed to adoptChildPresets at construction.
+		virtual void setPresets( const PresetsContainer &presets );
 		/// Implemented to return true only if all children have presetsOnly() true and
 		/// true was passed to adoptChildPresets at construction.
 		virtual bool presetsOnly() const;
diff --git a/include/IECore/Parameter.h b/include/IECore/Parameter.h
@@ -78,8 +78,12 @@ class Parameter : public RunTimeTyped
 		const std::string &description() const;
 		/// Returns the default value for this parameter.
 		virtual const Object *defaultValue() const;
-		/// Returns the presets for this parameter.
+		/// \deprecated Use getPresets instead.
 		virtual const PresetsContainer &presets() const;
+		/// Returns the presets for this parameter.
+		virtual const PresetsContainer &getPresets() const;
+		/// Overrides the presets for this parameter.
+		virtual void setPresets( const PresetsContainer &presets );
 		/// Returns true if this parameter only accepts
 		/// parameters present as presets.
 		virtual bool presetsOnly() const;
diff --git a/src/IECore/CompoundParameter.cpp b/src/IECore/CompoundParameter.cpp
@@ -1,6 +1,6 @@
 //////////////////////////////////////////////////////////////////////////
 //
-//  Copyright (c) 2007-2012, Image Engine Design Inc. All rights reserved.
+//  Copyright (c) 2007-2013, Image Engine Design Inc. All rights reserved.
 //
 //  Redistribution and use in source and binary forms, with or without
 //  modification, are permitted provided that the following conditions are
@@ -73,27 +73,32 @@ const Object *CompoundParameter::defaultValue() const
 }
 
 const Parameter::PresetsContainer &CompoundParameter::presets() const
+{
+	return getPresets();
+}
+
+const Parameter::PresetsContainer &CompoundParameter::getPresets() const
 {
 	if( !m_adoptChildPresets )
 	{
-		return Parameter::presets();
+		return Parameter::getPresets();
 	}
 
 	// naughty? nah! it gives the right semantics to an outside observer
-	PresetsContainer &pr = const_cast<PresetsContainer &>( Parameter::presets() );
+	PresetsContainer &pr = const_cast<PresetsContainer &>( Parameter::getPresets() );
 	pr.clear();
 	if( !m_namesToParameters.size() )
 	{
 		return pr;
 	}
 
 	// get a references for each child preset map.
-	// we only want to call presets() once for
+	// we only want to call getPresets() once for
 	// each child as the map returned may change between calls.
 	vector<const PresetsContainer *> childPresets;
 	for( size_t i=0; i<m_parameters.size(); i++ )
 	{
-		childPresets.push_back( &(m_parameters[i]->presets()) );
+		childPresets.push_back( &(m_parameters[i]->getPresets()) );
 	}
 	
 	// find the intersection of all the child preset names
@@ -146,6 +151,15 @@ const Parameter::PresetsContainer &CompoundParameter::presets() const
 	return pr;
 }
 
+void CompoundParameter::setPresets( const PresetsContainer &presets )
+{
+	if (m_adoptChildPresets)
+	{
+		throw Exception( "CompoundParameter cannot override presets when initialized with adoptChildPresets to true.");
+	}
+	Parameter::setPresets(presets);
+}
+
 bool CompoundParameter::presetsOnly() const
 {
 	if( !m_adoptChildPresets || !m_parameters.size() )
diff --git a/src/IECore/Parameter.cpp b/src/IECore/Parameter.cpp
@@ -92,10 +92,20 @@ const Object *Parameter::defaultValue() const
 }
 
 const Parameter::PresetsContainer &Parameter::presets() const
+{
+	return getPresets();
+}
+
+const Parameter::PresetsContainer &Parameter::getPresets() const
 {
 	return m_presets;
 }
 
+void Parameter::setPresets( const PresetsContainer &presets )
+{
+	m_presets = presets;
+}
+
 bool Parameter::presetsOnly() const
 {
 	return m_presetsOnly;
@@ -145,7 +155,7 @@ bool Parameter::valueValid( const Object *value, std::string *reason ) const
 	{
 		return true;
 	}
-	const PresetsContainer &pr = presets();
+	const PresetsContainer &pr = getPresets();
 	for( PresetsContainer::const_iterator it = pr.begin(); it!=pr.end(); it++ )
 	{
 		if( it->second->isEqualTo( value ) )
@@ -201,7 +211,7 @@ void Parameter::setValidatedValue( ObjectPtr value )
 
 void Parameter::setValue( const std::string &presetName )
 {
-	const PresetsContainer &pr = presets();
+	const PresetsContainer &pr = getPresets();
 	
 	PresetsContainer::const_iterator it;
 	for( it=pr.begin(); it != pr.end(); it++ )
@@ -251,7 +261,7 @@ std::string Parameter::getCurrentPresetName() const
 	// didn't have to do a copy of the value. but that breaks with CompoundParameter
 	// as it builds the value dynamically in getValue().
 	const Object *currentValue = getValue();
-	const PresetsContainer &pr = presets();
+	const PresetsContainer &pr = getPresets();
 	PresetsContainer::const_iterator it;
 	for( it=pr.begin(); it!=pr.end(); it++ )
 	{
diff --git a/src/IECoreMaya/FromMayaMeshConverter.cpp b/src/IECoreMaya/FromMayaMeshConverter.cpp
@@ -601,10 +601,10 @@ IECore::PrimitivePtr FromMayaMeshConverter::doPrimitiveConversion( MFnMesh &fnMe
 			unsigned int interpolationIndex = interpolationPlug.asInt(MDGContext::fsNormal, &st);
 			if ( st )
 			{
-				if ( interpolationIndex < m_interpolation->presets().size() - 1 )
+				if ( interpolationIndex < m_interpolation->getPresets().size() - 1 )
 				{
 					// convert interpolation index to the preset value
-					interpolation = staticPointerCast< StringData >( m_interpolation->presets()[interpolationIndex].second )->readable();
+					interpolation = staticPointerCast< StringData >( m_interpolation->getPresets()[interpolationIndex].second )->readable();
 				}
 				else
 				{
diff --git a/src/IECoreMaya/ToMayaMeshConverter.cpp b/src/IECoreMaya/ToMayaMeshConverter.cpp
@@ -495,7 +495,7 @@ bool ToMayaMeshConverter::setMeshInterpolationAttribute( MObject &object, std::s
 	int interpolationValue = 0;
 
 	FromMayaMeshConverter fromMaya(object);
-	const IECore::Parameter::PresetsContainer &presets = fromMaya.interpolationParameter()->presets();
+	const IECore::Parameter::PresetsContainer &presets = fromMaya.interpolationParameter()->getPresets();
 	IECore::Parameter::PresetsContainer::const_iterator it;
 
 	if ( interpolation != "default" )
diff --git a/src/IECoreNuke/PresetsOnlyParameterHandler.cpp b/src/IECoreNuke/PresetsOnlyParameterHandler.cpp
@@ -1,6 +1,6 @@
 //////////////////////////////////////////////////////////////////////////
 //
-//  Copyright (c) 2010-2011, Image Engine Design Inc. All rights reserved.
+//  Copyright (c) 2010-2013, Image Engine Design Inc. All rights reserved.
 //
 //  Redistribution and use in source and binary forms, with or without
 //  modification, are permitted provided that the following conditions are
@@ -49,7 +49,8 @@ void PresetsOnlyParameterHandler::knobs( const IECore::Parameter *parameter, con
 	if( f.makeKnobs() )
 	{
 		m_names.clear();
-		for( Parameter::PresetsContainer::const_iterator it = parameter->presets().begin(); it!=parameter->presets().end(); it++ )
+		const Parameter::PresetsContainer &presets = parameter->getPresets();
+		for( Parameter::PresetsContainer::const_iterator it = presets.begin(); it!=presets.end(); it++ )
 		{
 			if( it->second->isEqualTo( parameter->defaultValue() ) )
 			{
@@ -79,12 +80,12 @@ void PresetsOnlyParameterHandler::setParameterValue( IECore::Parameter *paramete
 	{
 		presetIndex = (int)m_knob->get_value();
 	}
-	parameter->setValue( parameter->presets()[m_storage].second );
+	parameter->setValue( parameter->getPresets()[m_storage].second );
 }
 
 void PresetsOnlyParameterHandler::setKnobValue( const IECore::Parameter *parameter )
 {
-	const Parameter::PresetsContainer &presets = parameter->presets();
+	const Parameter::PresetsContainer &presets = parameter->getPresets();
 	std::string currentPresetName = parameter->getCurrentPresetName();
 	size_t presetIndex = 0;
 	for( Parameter::PresetsContainer::const_iterator it = presets.begin(); it!=presets.end(); it++, presetIndex++ )
diff --git a/src/IECorePython/ParameterBinding.cpp b/src/IECorePython/ParameterBinding.cpp
@@ -1,6 +1,6 @@
 //////////////////////////////////////////////////////////////////////////
 //
-//  Copyright (c) 2007-2010, Image Engine Design Inc. All rights reserved.
+//  Copyright (c) 2007-2013, Image Engine Design Inc. All rights reserved.
 //
 //  Redistribution and use in source and binary forms, with or without
 //  modification, are permitted provided that the following conditions are
@@ -75,21 +75,26 @@ static void validate( Parameter &that, ObjectPtr value )
 	that.validate( value.get() );
 }
 
-static dict presets( Parameter &that )
+static dict getPresets( Parameter &that )
 {
 	dict result;
-	const Parameter::PresetsContainer &p = that.presets();
+	const Parameter::PresetsContainer &p = that.getPresets();
 	for( Parameter::PresetsContainer::const_iterator it=p.begin(); it!=p.end(); it++ )
 	{
 		result[it->first] = it->second->copy();
 	}
 	return result;
 }
 
+static void setPresets( Parameter &p, const object &presets )
+{
+	p.setPresets( parameterPresets<Parameter::PresetsContainer>( presets ) );
+}
+
 static boost::python::tuple presetNames( const Parameter &that )
 {
 	boost::python::list result;
-	const Parameter::PresetsContainer &p = that.presets();
+	const Parameter::PresetsContainer &p = that.getPresets();
 	for( Parameter::PresetsContainer::const_iterator it=p.begin(); it!=p.end(); it++ )
 	{
 		result.append( it->first );
@@ -100,7 +105,7 @@ static boost::python::tuple presetNames( const Parameter &that )
 static boost::python::tuple presetValues( const Parameter &that )
 {
 	boost::python::list result;
-	const Parameter::PresetsContainer &p = that.presets();
+	const Parameter::PresetsContainer &p = that.getPresets();
 	for( Parameter::PresetsContainer::const_iterator it=p.begin(); it!=p.end(); it++ )
 	{
 		result.append( it->second->copy() );
@@ -161,7 +166,9 @@ void bindParameter()
 		.def( "validate", (void (Parameter::*)() const)&Parameter::validate )
 		.def( "validate", &validate )
 		.add_property( "presetsOnly", &Parameter::presetsOnly )
-		.def( "presets", &presets, "Returns a dictionary containing presets for the parameter." )
+		.def( "presets", &getPresets, "Deprecated function. Use getPresets() instead." )
+		.def( "getPresets", &getPresets, "Returns a dictionary containing presets for the parameter." )
+		.def( "setPresets", &setPresets, "Sets the presets for the parameter from a dictionary." )
 		.def( "presetNames", &presetNames, "Returns a tuple containing the names of all presets for the parameter." )
 		.def( "presetValues", &presetValues, "Returns a tuple containing the values of all presets for the parameter." )
 		.def( "userData", &userData )
diff --git a/test/IECore/CompoundParameterTest.py b/test/IECore/CompoundParameterTest.py
@@ -179,7 +179,7 @@ def testPresets( self ) :
 
 		self.assertEqual( p.presetsOnly, True )
 
-		pr = p.presets()
+		pr = p.getPresets()
 		self.assertEqual( len( pr ), 3 )
 		self.assert_( "one" in pr.keys() )
 		self.assert_( "two" in pr.keys() )
@@ -191,6 +191,7 @@ def testPresets( self ) :
 
 		p.setValue( "four" )
 		self.assertEqual( p.getCurrentPresetName(), "four" )
+		self.assertRaises( RuntimeError, p.setPresets, [] )		# CompoundParameter created with adoptChildPresets=True does not allow overriding presets
 
 		p = CompoundParameter(
 			name = "c",
@@ -220,7 +221,7 @@ def testPresets( self ) :
 		)
 
 		self.assertEqual( p.presetsOnly, False )
-		self.assertEqual( len( p.presets() ), 0 )
+		self.assertEqual( len( p.getPresets() ), 0 )
 
 	def testLateValidation( self ) :
 
@@ -346,18 +347,18 @@ def testAddParametersPresets( self ) :
 			members = []
 		)
 
-		self.assertEqual( p.presets(), {} )
+		self.assertEqual( p.getPresets(), {} )
 
 		p.addParameter( IntParameter( name = "i", description = "d", defaultValue = 10, presets = ( ( "one", 1 ), ( "two", 2 ) ) ) )
 
-		self.assertEqual( len( p.presets() ), 2 )
-		self.assertEqual( p.presets(), { "one" : CompoundObject( { "i" : IntData( 1 ) } ), "two" : CompoundObject( { "i" : IntData( 2 ) } ) } )
+		self.assertEqual( len( p.getPresets() ), 2 )
+		self.assertEqual( p.getPresets(), { "one" : CompoundObject( { "i" : IntData( 1 ) } ), "two" : CompoundObject( { "i" : IntData( 2 ) } ) } )
 
 		fParam = FloatParameter( name = "f", description = "d", defaultValue = 20, presets = ( ( "one", 1 ), ) )
 		p.addParameter( fParam )
 
-		self.assertEqual( len( p.presets() ), 1 )
-		self.assertEqual( p.presets(), { "one" : CompoundObject( { "i" : IntData( 1 ), "f" : FloatData( 1 ) } ) } )
+		self.assertEqual( len( p.getPresets() ), 1 )
+		self.assertEqual( p.getPresets(), { "one" : CompoundObject( { "i" : IntData( 1 ), "f" : FloatData( 1 ) } ) } )
 
 		p.insertParameter( IntParameter( name = "x", description = "x", defaultValue = 10 ), fParam )
 		self.assertEqual( p.keys(), [ "i", "x", "f" ] )
@@ -586,7 +587,7 @@ def testAdoptChildPresets( self ) :
 			],
 		)
 		
-		self.assertEqual( len( c.presets() ), 2 )
+		self.assertEqual( len( c.getPresets() ), 2 )
 		self.assertEqual( c.presetsOnly, True )
 		
 		# no adoption of presets
@@ -618,7 +619,7 @@ def testAdoptChildPresets( self ) :
 			adoptChildPresets = False,
 		)
 		
-		self.assertEqual( len( c.presets() ), 0 )
+		self.assertEqual( len( c.getPresets() ), 0 )
 		self.assertEqual( c.presetsOnly, False )
 		
 		# no adoption of presets without use of keyword parameters
@@ -652,10 +653,35 @@ def testAdoptChildPresets( self ) :
 			False,
 		)
 		
-		self.assertEqual( len( c.presets() ), 0 )
+		self.assertEqual( len( c.getPresets() ), 0 )
 		self.assertEqual( c.presetsOnly, False )
 		self.assertEqual( c.userData()["ud"].value, 10 )
 
+		# when adoptChildPresets we can also set presets explicitly...
+		c['a'].setValue("one")
+		c['b'].setValue("two")
+		p1 = c.getValue().copy()
+		c['a'].setValue("two")
+		c['b'].setValue("one")
+		p2 = c.getValue().copy()
+		c.setValue( c.defaultValue )
+
+		c.setPresets(
+			[
+				( "p1", p1 ),
+				( "p2", p2 ),
+			]
+		)
+		pr = c.getPresets()
+		self.assertEqual( len( pr ), 2 )
+		self.assertEqual( pr["p1"], p1 )
+		self.assertEqual( pr["p2"], p2 )
+		self.assertEqual( c.presetNames(), ( "p1", "p2" ) )
+		c.setValue("p1")
+		self.assertEqual( c.getValue(), p1 )
+		c.setValue("p2")
+		self.assertEqual( c.getValue(), p2 )
+		
 	def testDerivingInPython( self ) :
 	
 		class DerivedCompoundParameter( CompoundParameter ) :
diff --git a/test/IECore/EXRImageWriter.py b/test/IECore/EXRImageWriter.py
@@ -223,11 +223,11 @@ def testCompressionParameter( self ):
 		img = r.read()
 
 		w = Writer.create( img, "test/IECore/data/exrFiles/output.exr" )
-		w['compression'].setValue( w['compression'].presets()['zip'] )
+		w['compression'].setValue( w['compression'].getPresets()['zip'] )
 		w.write()
 
 		w = EXRImageWriter()
-		w['compression'].setValue( w['compression'].presets()['zip'] )
+		w['compression'].setValue( w['compression'].getPresets()['zip'] )
 
 	def testBlindDataToHeader( self ) :
 
diff --git a/test/IECore/Parameters.py b/test/IECore/Parameters.py

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`//////////////////////////////////////////////////////////////////////////`
`2`	`2`	`//`
`3`		`-// Copyright (c) 2007-2012, Image Engine Design Inc. All rights reserved.`
	`3`	`+// Copyright (c) 2007-2013, Image Engine Design Inc. All rights reserved.`
`4`	`4`	`//`
`5`	`5`	`// Redistribution and use in source and binary forms, with or without`
`6`	`6`	`// modification, are permitted provided that the following conditions are`
`@@ -73,27 +73,32 @@ const Object *CompoundParameter::defaultValue() const`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`const Parameter::PresetsContainer &CompoundParameter::presets() const`
	`76`	`+{`
	`77`	`+ return getPresets();`
	`78`	`+}`
	`79`	`+`
	`80`	`+const Parameter::PresetsContainer &CompoundParameter::getPresets() const`
`76`	`81`	`{`
`77`	`82`	`if( !m_adoptChildPresets )`
`78`	`83`	`{`
`79`		`- return Parameter::presets();`
	`84`	`+ return Parameter::getPresets();`
`80`	`85`	`}`
`81`	`86`
`82`	`87`	`// naughty? nah! it gives the right semantics to an outside observer`
`83`		`- PresetsContainer &pr = const_cast<PresetsContainer &>( Parameter::presets() );`
	`88`	`+ PresetsContainer &pr = const_cast<PresetsContainer &>( Parameter::getPresets() );`
`84`	`89`	`pr.clear();`
`85`	`90`	`if( !m_namesToParameters.size() )`
`86`	`91`	`{`
`87`	`92`	`return pr;`
`88`	`93`	`}`
`89`	`94`
`90`	`95`	`// get a references for each child preset map.`
`91`		`- // we only want to call presets() once for`
	`96`	`+ // we only want to call getPresets() once for`
`92`	`97`	`// each child as the map returned may change between calls.`
`93`	`98`	`vector<const PresetsContainer *> childPresets;`
`94`	`99`	`for( size_t i=0; i<m_parameters.size(); i++ )`
`95`	`100`	`{`
`96`		`- childPresets.push_back( &(m_parameters[i]->presets()) );`
	`101`	`+ childPresets.push_back( &(m_parameters[i]->getPresets()) );`
`97`	`102`	`}`
`98`	`103`
`99`	`104`	`// find the intersection of all the child preset names`
`@@ -146,6 +151,15 @@ const Parameter::PresetsContainer &CompoundParameter::presets() const`
`146`	`151`	`return pr;`
`147`	`152`	`}`
`148`	`153`
	`154`	`+void CompoundParameter::setPresets( const PresetsContainer &presets )`
	`155`	`+{`
	`156`	`+ if (m_adoptChildPresets)`
	`157`	`+ {`
	`158`	`+ throw Exception( "CompoundParameter cannot override presets when initialized with adoptChildPresets to true.");`
	`159`	`+ }`
	`160`	`+ Parameter::setPresets(presets);`
	`161`	`+}`
	`162`	`+`
`149`	`163`	`bool CompoundParameter::presetsOnly() const`
`150`	`164`	`{`
`151`	`165`	`if( !m_adoptChildPresets \|\| !m_parameters.size() )`
Original file line number	Diff line number	Diff line change
`@@ -92,10 +92,20 @@ const Object *Parameter::defaultValue() const`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`const Parameter::PresetsContainer &Parameter::presets() const`
	`95`	`+{`
	`96`	`+ return getPresets();`
	`97`	`+}`
	`98`	`+`
	`99`	`+const Parameter::PresetsContainer &Parameter::getPresets() const`
`95`	`100`	`{`
`96`	`101`	`return m_presets;`
`97`	`102`	`}`
`98`	`103`
	`104`	`+void Parameter::setPresets( const PresetsContainer &presets )`
	`105`	`+{`
	`106`	`+ m_presets = presets;`
	`107`	`+}`
	`108`	`+`
`99`	`109`	`bool Parameter::presetsOnly() const`
`100`	`110`	`{`
`101`	`111`	`return m_presetsOnly;`
`@@ -145,7 +155,7 @@ bool Parameter::valueValid( const Object value, std::string reason ) const`
`145`	`155`	`{`
`146`	`156`	`return true;`
`147`	`157`	`}`
`148`		`- const PresetsContainer &pr = presets();`
	`158`	`+ const PresetsContainer &pr = getPresets();`
`149`	`159`	`for( PresetsContainer::const_iterator it = pr.begin(); it!=pr.end(); it++ )`
`150`	`160`	`{`
`151`	`161`	`if( it->second->isEqualTo( value ) )`
`@@ -201,7 +211,7 @@ void Parameter::setValidatedValue( ObjectPtr value )`
`201`	`211`
`202`	`212`	`void Parameter::setValue( const std::string &presetName )`
`203`	`213`	`{`
`204`		`- const PresetsContainer &pr = presets();`
	`214`	`+ const PresetsContainer &pr = getPresets();`
`205`	`215`
`206`	`216`	`PresetsContainer::const_iterator it;`
`207`	`217`	`for( it=pr.begin(); it != pr.end(); it++ )`
`@@ -251,7 +261,7 @@ std::string Parameter::getCurrentPresetName() const`
`251`	`261`	`// didn't have to do a copy of the value. but that breaks with CompoundParameter`
`252`	`262`	`// as it builds the value dynamically in getValue().`
`253`	`263`	`const Object *currentValue = getValue();`
`254`		`- const PresetsContainer &pr = presets();`
	`264`	`+ const PresetsContainer &pr = getPresets();`
`255`	`265`	`PresetsContainer::const_iterator it;`
`256`	`266`	`for( it=pr.begin(); it!=pr.end(); it++ )`
`257`	`267`	`{`
Original file line number	Diff line number	Diff line change
`@@ -601,10 +601,10 @@ IECore::PrimitivePtr FromMayaMeshConverter::doPrimitiveConversion( MFnMesh &fnMe`
`601`	`601`	`unsigned int interpolationIndex = interpolationPlug.asInt(MDGContext::fsNormal, &st);`
`602`	`602`	`if ( st )`
`603`	`603`	`{`
`604`		`- if ( interpolationIndex < m_interpolation->presets().size() - 1 )`
	`604`	`+ if ( interpolationIndex < m_interpolation->getPresets().size() - 1 )`
`605`	`605`	`{`
`606`	`606`	`// convert interpolation index to the preset value`
`607`		`- interpolation = staticPointerCast< StringData >( m_interpolation->presets()[interpolationIndex].second )->readable();`
	`607`	`+ interpolation = staticPointerCast< StringData >( m_interpolation->getPresets()[interpolationIndex].second )->readable();`
`608`	`608`	`}`
`609`	`609`	`else`
`610`	`610`	`{`