VOLTTRON / volttron

VOLTTRON Distributed Control System Platform
https://volttron.readthedocs.io/
Other
453 stars 215 forks source link

fix for issue 3053. Added check to determine if variable meta is a dict #3054

Closed ntenney closed 1 year ago

ntenney commented 1 year ago

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #3053

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

This code has been run against databases containing jsonb type and strings for metadata. In addition, I used this code against a local list that simulated the results coming from the database, and also tested it against a situation that should never arise in the wild (strings mixed with objects in the metadata result column). This was tested using the 8.2 deploy of campus.

craig8 commented 1 year ago

@ntenney Thank you!