Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unpacking RESTful sensor JSON results into attributes. #10753

Merged
merged 32 commits into from
Dec 3, 2017
Merged

Unpacking RESTful sensor JSON results into attributes. #10753

Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
dbaa626
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
fb19f87
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
0cb9def
Merge branch 'rest-json-attrs' of https://github.com/nickovs/home-ass…
nickovs May 6, 2017
e08f649
Added requirement that RESTful JSON results used as attributes must be
nickovs May 6, 2017
de07ae5
Expanded test coverage to test REFTful JSON attributes with and
nickovs May 6, 2017
227a5ad
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
769c7ff
Added requirement that RESTful JSON results used as attributes must be
nickovs May 6, 2017
0f25f6d
Expanded test coverage to test REFTful JSON attributes with and
nickovs May 6, 2017
a22a6d4
Merge branch 'rest-json-attrs' of github.com:nickovs/home-assistant i…
nickovs May 7, 2017
6bd57cc
sensor.envirophat: add missing requirement (#7451)
imrehg May 5, 2017
d635e26
PyPI Openzwave (#7415)
JshWright May 5, 2017
44cc762
Add datadog component (#7158)
nunofgs May 5, 2017
67e8f52
Fix object type for default KNX port
JshWright May 5, 2017
d23ccce
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
f5bbf6b
Added requirement that RESTful JSON results used as attributes must be
nickovs May 6, 2017
a281132
Expanded test coverage to test REFTful JSON attributes with and
nickovs May 6, 2017
dda455d
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
3a99f4e
Added requirement that RESTful JSON results used as attributes must be
nickovs May 6, 2017
22bc0fc
Expanded test coverage to test REFTful JSON attributes with and
nickovs May 6, 2017
d19a81b
Added support for extracting JSON attributes from RESTful values
nickovs May 6, 2017
4ef54a1
Added requirement that RESTful JSON results used as attributes must be
nickovs May 6, 2017
0b9711c
Expanded test coverage to test REFTful JSON attributes with and
nickovs May 6, 2017
8c5548e
Merge branch 'rest-json-attrs' of github.com:nickovs/home-assistant i…
nickovs Nov 20, 2017
0795823
Brought up to date with most recent upstream master
nickovs Nov 20, 2017
19eab69
Fixed breaks cause by manual upstream merge.
nickovs Nov 22, 2017
9dd2a5a
Added one extra blank line to make PyLint happy.
nickovs Nov 22, 2017
2dbcff5
Switched json_attributes to be a list of keys rather than a boolean.
nickovs Nov 28, 2017
1d36270
Added an explicit default value to the json_attributes config entry.
nickovs Dec 2, 2017
02158b0
Removed self.update() from __init__() body.
nickovs Dec 2, 2017
a537b45
Merge branch 'rest-json-attrs' of github.com:nickovs/home-assistant i…
nickovs Dec 2, 2017
3a9650c
Expended unit tests for error cases of json_attributes processing.
nickovs Dec 3, 2017
0bf5ea3
Align quotes
fabaff Dec 3, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Expanded test coverage to test REFTful JSON attributes with and 
without a value template.
  • Loading branch information
@nickovs
nickovs committed Nov 20, 2017
commit 0b9711cfff002072d7c2c992d3b6252940fb173a
12 changes: 0 additions & 12 deletions homeassistant/components/sensor/rest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,18 +133,6 @@ def update(self):
value = self._value_template.render_with_possible_json_value(
value, STATE_UNKNOWN)

if self._json_attrs:
self._attributes = None
try:
attrs = json.loads(value)
if isinstance(attrs, dict):
self._attributes = json.loads(value)
else:
_LOGGER.warning('JSON result was not a dictionary')
except ValueError:
_LOGGER.warning('REST result could not be parsed as JSON')
_LOGGER.debug('Erroneous JSON: %s', value)

self._state = value

@property
Expand Down