admin/PowerShell-PowerShell
1
0
Fork 0
mirror of https://github.com/PowerShell/PowerShell synced 2026-06-08 12:12:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Get-ChildItem -Recurse -ErrorAction Ignore doesn't ignore the errors (#2503)

Browse Source 
Get-ChildItem -Recurse -ErrorAction Ignore doesn't ignore the errors for some specific files under a folder and instead terminate the pipeline.
This commit is contained in:
nitinbps
2016-11-02 11:26:51 -07:00
committed by Dongbo Wang
parent f102fce1dd
commit f4c1569421
1 changed files with 50 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/System.Management.Automation/engine/SessionStateContainer.cs
+50 -5
View File
@@ -1764,6 +1764,53 @@ namespace System.Management.Automation
}
} // GetChildItems
/// <summary>
/// Determines if the item at the specified path is a container.
/// </summary>
///
/// <param name="providerInstance">
/// The provider instance to use.
/// </param>
///
/// <param name="path">
/// The path to the item if it was specified on the command line.
/// </param>
///
/// <param name="context">
/// The context which the core command is running.
/// </param>
///
private bool IsPathContainer(
CmdletProvider providerInstance,
string path,
CmdletProviderContext context)
{
bool itemContainer = false;
try
{
itemContainer = IsItemContainer(providerInstance, path, context);
}
catch (UnauthorizedAccessException accessException)
{
context.WriteError(new ErrorRecord(accessException, "GetItemUnauthorizedAccessError", ErrorCategory.PermissionDenied, path));
}
catch (ProviderInvocationException accessException)
{
// if providerinvocationexception is wrapping access denied error, it is ok to not terminate the pipeline
if (accessException.InnerException != null &&
accessException.InnerException.GetType().Equals(typeof(System.UnauthorizedAccessException)))
{
context.WriteError(new ErrorRecord(accessException, "GetItemUnauthorizedAccessError", ErrorCategory.PermissionDenied, path));
}
else
{
throw;
}
}
return itemContainer;
} // IsPathContainer
/// <summary>
/// Since we can't do include and exclude filtering on items we have to
/// do the recursion ourselves. We get each child name and see if it matches
@@ -1852,9 +1899,8 @@ namespace System.Management.Automation
WildcardOptions.IgnoreCase);
// If the item is a container we have to filter its children
// Use a hint + lazy evaluation to skip a container check
if (skipIsItemContainerCheck || IsItemContainer(providerInstance, path, context))
// Use a hint + lazy evaluation to skip a container check
if (skipIsItemContainerCheck || IsPathContainer(providerInstance, path, context))
{
CmdletProviderContext newContext =
new CmdletProviderContext(context);
@@ -1976,14 +2022,13 @@ namespace System.Management.Automation
}
// Now recurse if it is a container
if (recurse && IsItemContainer(providerInstance, qualifiedPath, context))
if (recurse && IsPathContainer(providerInstance, qualifiedPath, context))
{
// Making sure to obey the StopProcessing.
if (context.Stopping)
{
return;
}
// The item is a container so recurse into it.
ProcessPathItems(providerInstance, qualifiedPath, recurse, context, out childrenNotMatchingFilterCriteria, processMode, skipIsItemContainerCheck: true);
}