Skip to content

Ignore the write with proposedValue === undefined #11

@bmblb

Description

@bmblb

Feels wrong to merge null and undefined values. undefined is smth that doesn't exist, null on the other hand is smth that exists, but doesn't have value.
This makes write(identifier, undefined) same as write(identifier, null). Latter is definitely user wanting to null the value. First doesn't make sense to call deliberately, rather a handy filter for undefined values - if undefined is passed then do nothing.

if (proposedValue === undefined) proposedValue = null

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions