Fix Select-Object with Property = '*'

iSazonov · iSazonov · commit 70c30384c74d · 2016-10-05T17:04:46.000+03:00
1. Now ExcludeProperty imply Property = '*'
2. Don't return property named '*'
diff --git a/src/Microsoft.PowerShell.Commands.Utility/commands/utility/select-object.cs b/src/Microsoft.PowerShell.Commands.Utility/commands/utility/select-object.cs
@@ -337,6 +337,11 @@ private void ProcessExpressionParameter()
             if (ExcludeProperty != null)
             {
                 _exclusionFilter = new MshExpressionFilter(ExcludeProperty);
+                if ((Property == null) || (Property.Length == 0))
+                {
+                    Property = new Object[]{"*"};
+                    _propertyMshParameterList = processor.ProcessParameters(Property, invocationContext);
+                }
             }
         }
 
@@ -414,7 +419,7 @@ private void ProcessParameter(MshParameter p, PSObject inputObject, List<PSNoteP
                 }
             }
 
-            if (expressionResults.Count == 0)
+            if (expressionResults.Count == 0 && !ex.HasWildCardCharacters)
             {
                 //Commented out for bug 1107600
                 //if (!ex.HasWildCardCharacters)

Original file line number	Diff line number	Diff line change
`@@ -337,6 +337,11 @@ private void ProcessExpressionParameter()`
`337`	`337`	`if (ExcludeProperty != null)`
`338`	`338`	`{`
`339`	`339`	`_exclusionFilter = new MshExpressionFilter(ExcludeProperty);`
	`340`	`+ if ((Property == null) \|\| (Property.Length == 0))`
	`341`	`+ {`
	`342`	`+ Property = new Object[]{"*"};`
	`343`	`+ _propertyMshParameterList = processor.ProcessParameters(Property, invocationContext);`
	`344`	`+ }`
`340`	`345`	`}`
`341`	`346`	`}`
`342`	`347`
`@@ -414,7 +419,7 @@ private void ProcessParameter(MshParameter p, PSObject inputObject, List<PSNoteP`
`414`	`419`	`}`
`415`	`420`	`}`
`416`	`421`
`417`		`- if (expressionResults.Count == 0)`
	`422`	`+ if (expressionResults.Count == 0 && !ex.HasWildCardCharacters)`
`418`	`423`	`{`
`419`	`424`	`//Commented out for bug 1107600`
`420`	`425`	`//if (!ex.HasWildCardCharacters)`