HiddenFilePathFilter: Fix hidden sequences

Changes.md

@@ -1,6 +1,10 @@
 1.3.16.x (relative to 1.3.16.1)
 ========
 
+Fixes
+-----
+
+- File Browser : Windows only : Fixed bug in `HiddenFilePathFilter` that caused sequences to be treated as though they are hidden files (#5810).
 
 
 1.3.16.1 (relative to 1.3.16.0)

python/GafferTest/HiddenFilePathFilterTest.py

@@ -39,6 +39,8 @@
 import subprocess
 import unittest
 
+import IECore
+
 import Gaffer
 import GafferTest
 
@@ -72,6 +74,55 @@ def test( self ) :
 
 		self.assertEqual( p.children(), [ visibleFile ] )
 
+	def testSequence( self ) :
+
+		hiddenSequence = IECore.FileSequence( ( self.temporaryDirectory() / ".hidden.#.txt 1-3" ).as_posix() )
+		visibleSequence = IECore.FileSequence( ( self.temporaryDirectory() / "visible.#.txt 1-3" ).as_posix() )
+
+		for frame in range( 3 ) :
+			hiddenFile = Gaffer.FileSystemPath( hiddenSequence.fileNameForFrame( frame ) )
+			with open( hiddenFile.nativeString(), "w", encoding = "utf-8" ) as f :
+				f.write( "Hidden sequence")
+			if os.name == "nt" :
+				subprocess.check_call( [ "attrib", "+H", hiddenFile.nativeString() ] )
+
+			visibleFile = Gaffer.FileSystemPath( visibleSequence.fileNameForFrame( frame ) )
+			with open( visibleFile.nativeString(), "w", encoding = "utf-8" ) as f :
+				f.write( "Visible sequence" )
+
+		p = Gaffer.FileSystemPath( pathlib.Path( hiddenSequence.fileName ).parent, None, True )
+
+		self.assertEqual(
+			sorted( [ str( i ) for i in p.children() ] ),
+			sorted(
+				[ hiddenSequence.fileNameForFrame( i ) for i in range( 3 ) ] +
+				[ visibleSequence.fileNameForFrame( i ) for i in range( 3 ) ] +
+				[ hiddenSequence.fileName, visibleSequence.fileName ]
+			)
+		)
+
+		h = Gaffer.HiddenFilePathFilter()
+		p.setFilter( h )
+
+		self.assertEqual(
+			sorted( [ str( i ) for i in p.children() ] ),
+			sorted(
+				[ hiddenSequence.fileNameForFrame( i ) for i in range( 3 ) ] +
+				[ hiddenSequence.fileName ]
+			)
+		)
+
+		h.setInverted( True )
+
+		self.assertEqual(
+			sorted( [ str( i ) for i in p.children() ] ),
+			sorted(
+				[ visibleSequence.fileNameForFrame( i ) for i in range( 3 ) ] +
+				[ visibleSequence.fileName ]
+			)
+		)
+
+
 
 if __name__ == "__main__":
 	unittest.main()

src/Gaffer/HiddenFilePathFilter.cpp

@@ -39,13 +39,16 @@
 
 #include "Gaffer/Path.h"
 
+#include "IECore/FileSequenceFunctions.h"
+
 #ifdef _MSC_VER
 
 #include <windows.h>
 
 #endif
 
 using namespace Gaffer;
+using namespace IECore;
 
 IE_CORE_DEFINERUNTIMETYPED( HiddenFilePathFilter );
 
@@ -105,10 +108,24 @@ bool HiddenFilePathFilter::remove( PathPtr path ) const
 	}
 	return invert( true );
 #else
-	DWORD fileAttributes = GetFileAttributes( path->string().c_str() );
-	if( fileAttributes & FILE_ATTRIBUTE_HIDDEN )
+	if( const auto filePath = runTimeCast<FileSystemPath>( path.get() ) )
 	{
-		return invert( false );
+		std::string file;
+		if( const auto sequence = filePath->fileSequence() )
+		{
+			std::vector<FrameList::Frame> frames;
+			sequence->getFrameList()->asList( frames );
+			file = sequence->fileNameForFrame( frames[0] );
+		}
+		else
+		{
+			file = path->string();
+		}
+		DWORD fileAttributes = GetFileAttributes( file.c_str() );
+		if( fileAttributes & FILE_ATTRIBUTE_HIDDEN )
+		{
+			return invert( false );
+		}
 	}
 	return invert( true );
 #endif