From b443d591a29bc18daea36a3871908a4c1c277990 Mon Sep 17 00:00:00 2001 From: Luke Pulverenti Date: Sun, 28 Apr 2013 01:29:27 -0400 Subject: fixes #200 - MB3 Locking Folders for a long time --- MediaBrowser.Controller/IO/FileSystem.cs | 159 ++++++++----------------------- 1 file changed, 41 insertions(+), 118 deletions(-) (limited to 'MediaBrowser.Controller/IO/FileSystem.cs') diff --git a/MediaBrowser.Controller/IO/FileSystem.cs b/MediaBrowser.Controller/IO/FileSystem.cs index 7b031744e..cbe561fdf 100644 --- a/MediaBrowser.Controller/IO/FileSystem.cs +++ b/MediaBrowser.Controller/IO/FileSystem.cs @@ -1,8 +1,7 @@ -using System; -using System.Collections.Generic; +using MediaBrowser.Model.Logging; +using System; using System.Collections.Specialized; using System.IO; -using System.Runtime.InteropServices; using System.Text; namespace MediaBrowser.Controller.IO @@ -13,153 +12,77 @@ namespace MediaBrowser.Controller.IO public static class FileSystem { /// - /// Gets information about a path + /// Gets the file system info. /// /// The path. - /// System.Nullable{WIN32_FIND_DATA}. - /// path - /// GetFileData failed for + path - public static WIN32_FIND_DATA? GetFileData(string path) + /// FileSystemInfo. + public static FileSystemInfo GetFileSystemInfo(string path) { - if (string.IsNullOrEmpty(path)) + // Take a guess to try and avoid two file system hits, but we'll double-check by calling Exists + if (Path.HasExtension(path)) { - throw new ArgumentNullException("path"); - } - - WIN32_FIND_DATA data; - var handle = NativeMethods.FindFirstFileEx(path, FINDEX_INFO_LEVELS.FindExInfoBasic, out data, - FINDEX_SEARCH_OPS.FindExSearchNameMatch, IntPtr.Zero, FindFirstFileExFlags.NONE); + var fileInfo = new FileInfo(path); - var getFilename = false; - - if (handle == NativeMethods.INVALID_HANDLE_VALUE && !Path.HasExtension(path)) - { - if (!path.EndsWith("*", StringComparison.OrdinalIgnoreCase)) + if (fileInfo.Exists) { - NativeMethods.FindClose(handle); - - handle = NativeMethods.FindFirstFileEx(Path.Combine(path, "*"), FINDEX_INFO_LEVELS.FindExInfoBasic, out data, - FINDEX_SEARCH_OPS.FindExSearchNameMatch, IntPtr.Zero, FindFirstFileExFlags.NONE); - - getFilename = true; + return fileInfo; } - } - - if (handle == IntPtr.Zero) - { - throw new IOException("GetFileData failed for " + path); - } - NativeMethods.FindClose(handle); - - // According to MSDN documentation, this will default to 1601 for paths that don't exist. - if (data.CreationTimeUtc.Year == 1601) - { - return null; + return new DirectoryInfo(path); } - - if (getFilename) + else { - data.cFileName = Path.GetFileName(path); - } + var fileInfo = new DirectoryInfo(path); - data.Path = path; - return data; - } + if (fileInfo.Exists) + { + return fileInfo; + } - /// - /// Gets all files within a folder - /// - /// The path. - /// The search pattern. - /// IEnumerable{WIN32_FIND_DATA}. - public static IEnumerable GetFiles(string path, string searchPattern = "*") - { - return GetFileSystemEntries(path, searchPattern, includeDirectories: false); + return new FileInfo(path); + } } /// - /// Gets all file system entries within a foler + /// Gets the creation time UTC. /// - /// The path. - /// The search pattern. - /// if set to true [include files]. - /// if set to true [include directories]. - /// IEnumerable{WIN32_FIND_DATA}. - /// path - /// GetFileSystemEntries failed - public static IEnumerable GetFileSystemEntries(string path, string searchPattern = "*", bool includeFiles = true, bool includeDirectories = true) + /// The info. + /// The logger. + /// DateTime. + public static DateTime GetLastWriteTimeUtc(FileSystemInfo info, ILogger logger) { - if (string.IsNullOrEmpty(path)) - { - throw new ArgumentNullException("path"); - } - - var lpFileName = Path.Combine(path, searchPattern); - - WIN32_FIND_DATA lpFindFileData; - var handle = NativeMethods.FindFirstFileEx(lpFileName, FINDEX_INFO_LEVELS.FindExInfoBasic, out lpFindFileData, - FINDEX_SEARCH_OPS.FindExSearchNameMatch, IntPtr.Zero, FindFirstFileExFlags.FIND_FIRST_EX_LARGE_FETCH); - - if (handle == IntPtr.Zero) - { - var hr = Marshal.GetLastWin32Error(); - if (hr != 2 && hr != 0x12) - { - throw new IOException("GetFileSystemEntries failed"); - } - yield break; - } + // This could throw an error on some file systems that have dates out of range - if (IncludeInFindFileOutput(lpFindFileData.cFileName, lpFindFileData.dwFileAttributes, includeFiles, includeDirectories)) + try { - lpFindFileData.Path = Path.Combine(path, lpFindFileData.cFileName); - - yield return lpFindFileData; + return info.LastAccessTimeUtc; } - - while (NativeMethods.FindNextFile(handle, out lpFindFileData) != IntPtr.Zero) + catch (Exception ex) { - if (IncludeInFindFileOutput(lpFindFileData.cFileName, lpFindFileData.dwFileAttributes, includeFiles, includeDirectories)) - { - lpFindFileData.Path = Path.Combine(path, lpFindFileData.cFileName); - yield return lpFindFileData; - } + logger.ErrorException("Error determining LastAccessTimeUtc for {0}", ex, info.FullName); + return DateTime.MinValue; } - - NativeMethods.FindClose(handle); } /// - /// Includes the in find file output. + /// Gets the creation time UTC. /// - /// Name of the c file. - /// The attributes. - /// if set to true [include files]. - /// if set to true [include directories]. - /// true if XXXX, false otherwise - public static bool IncludeInFindFileOutput(string cFileName, FileAttributes attributes, bool includeFiles, bool includeDirectories) + /// The info. + /// The logger. + /// DateTime. + public static DateTime GetCreationTimeUtc(FileSystemInfo info, ILogger logger) { - if (cFileName.Equals(".", StringComparison.OrdinalIgnoreCase)) - { - return false; - } - if (cFileName.Equals("..", StringComparison.OrdinalIgnoreCase)) - { - return false; - } + // This could throw an error on some file systems that have dates out of range - if (!includeFiles && !attributes.HasFlag(FileAttributes.Directory)) + try { - return false; + return info.CreationTimeUtc; } - - if (!includeDirectories && attributes.HasFlag(FileAttributes.Directory)) + catch (Exception ex) { - return false; + logger.ErrorException("Error determining CreationTimeUtc for {0}", ex, info.FullName); + return DateTime.MinValue; } - - return true; } /// -- cgit v1.2.3