Set-LMDeviceGroup

[aggie87]

There may be a problem sending an empty hash to the -Properties parameter when using the -PropertyMethod Refresh

Set-LMDeviceGroup -id 3406 -Properties @{} -Debug -PropertiesMethod Refresh               
DEBUG: Invoked Command: Set-LMDeviceGroup
DEBUG: Bound Parameters: [Id:3406] [Properties:System.Collections.Hashtable] [Debug:True] [PropertiesMethod:Refresh]
DEBUG: Invoked URL: https://xxxxsandbox.logicmonitor.com/santaba/rest/device/groups/3406?opType=refresh
DEBUG: Request Payload: 
{}
DEBUG: Request Headers: [User-Agent:Logic.Monitor-PowerShell-...] [Authorization:xxxxxxxxxxxxxxxxxxxx...] [Content-Type:application/json] [X-Version:3]

shouldn't the payload should look like the following to "clear out" the custom properties?

Set-LMDeviceGroup -id 3406 -Properties @{} -Debug -PropertiesMethod Refresh               
DEBUG: Invoked Command: Set-LMDeviceGroup
DEBUG: Bound Parameters: [Id:3406] [Properties:System.Collections.Hashtable] [Debug:True] [PropertiesMethod:Refresh]
DEBUG: Invoked URL: https://xxxxsandbox.logicmonitor.com/santaba/rest/device/groups/3406?opType=refresh
DEBUG: Request Payload: 
{
    "customProperties": [
    ]
}
DEBUG: Request Headers: [User-Agent:Logic.Monitor-PowerShell-...] [Authorization:xxxxxxxxxxxxxxxxxxxx...] [Content-Type:application/json] [X-Version:3]

[stevevillardi]

Let me take a look. I have a feeling it's because of the way we are removing unset keys from the payload so we don't accidentally clear something that's null. In this case however we need to handle it a bit differently as it's intended to be cleared. Thanks for bringing it to my attention.

[stevevillardi]

I've gone through the format logic for all of the cmdlets that prepare payload content to centralize them under a single helper function. This also includes the fix for the issue described above. It will be available in the next release but I need to do some more unit testing to ensure it does not introduce any additional issues.

159f24e#diff-61e31fb2af2edab99d33ac65121a67780038b2b2b800b667ed41dee3b101f9e2