PythonCommand : Fix repr() and str() for variables dict

Changes.md

@@ -5,6 +5,7 @@ Fixes
 -----
 
 - Viewer : Fixed context handling bug in the shader view (#5654).
+- PythonCommand : Fixed misleading results for `repr( variables )` and `str( variables )`, which would suggest the dictionary was empty when it was not.
 
 1.3.11.0 (relative to 1.3.10.0)
 ========

python/GafferDispatch/PythonCommand.py

@@ -158,6 +158,16 @@ def __getitem__( self, key ) :
 		self.__update()
 		return dict.__getitem__( self, key )
 
+	def __repr__( self ) :
+
+		self.__update()
+		return dict.__repr__( self )
+
+	def __str__( self ) :
+
+		self.__update()
+		return dict.__str__( self )
+
 	def __update( self ) :
 		frame = self.__context.get( "frame", "NO FRAME" )
 

python/GafferDispatchTest/PythonCommandTest.py

@@ -428,5 +428,15 @@ def testContextModificationsDontLeak( self ) :
 
 		self.assertEqual( Gaffer.Context.current(), Gaffer.Context() )
 
+	def testVariablesRepr( self ) :
+
+		command = GafferDispatch.PythonCommand()
+		command["variables"].addChild( Gaffer.NameValuePlug( "testKey", "testValue" ) )
+		command["command"].setValue( "self.variablesStr = str( variables ); self.variablesRepr = repr( variables )" )
+		command["task"].execute()
+
+		self.assertEqual( command.variablesStr, str( { "testKey" : "testValue" } ) )
+		self.assertEqual( command.variablesRepr, repr( { "testKey" : "testValue" } ) )
+
 if __name__ == "__main__":
 	unittest.main()