Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

ScriptBlock.GetNewClosure() unexpectedly tries to include unbound parameters that have validation attributes, causing errors #3144

Copy link
Copy link
@mklement0

Description

@mklement0
Issue body actions

Steps to reproduce

function foo { param([ValidateNotNull()] $bar) & {'OK'}.GetNewClosure() }
foo

Expected behavior

Function call foo should succeed, and string 'OK' should be output.

Actual behavior

The .GetNewClosure() call fails with the following error message:

Exception calling "GetNewClosure" with "0" argument(s): "The attribute cannot be added because variable bar with value would no longer be valid."

That is, an attempt is made to include the unbound -bar parameter value - the $bar variable - in the closure, which apparently then defaults to $null, which violates its validation attribute.

Passing a valid -bar value makes the problem go away; e.g., foo -bar ''.

I suspect this is a bug: If the function itself treats $bar in the absence of a -bar parameter value as nonexistent, so should .GetNewClosure().

The problem seems closely related to this bug report on uservoice.com, however, it has been marked as fixed in v5.

An example of how the problem might affect a real-world scenario can be found here.

Environment data

PowerShell v6.0.0-alpha (v6.0.0-alpha.15) on Darwin Kernel Version 16.4.0: Thu Dec 22 22:53:21 PST 2016; root:xnu-3789.41.3~3/RELEASE_X86_64
PowerShell v5.1.14393.693 on Microsoft Windows 10 Pro (64-bit; v10.0.14393)
Reactions are currently unavailable

Metadata

Metadata

Assignees

No one assigned

    Labels

    Resolution-FixedThe issue is fixed.The issue is fixed.WG-Enginecore PowerShell engine, interpreter, and runtimecore PowerShell engine, interpreter, and runtime

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.