Fix Audyssey update crashing when AppCommand0300.xml endpoint is not …

…defined / push to version 0.9.7
ol-iver · Nov 22, 2020 · 053727b · 053727b
1 parent 0df34f3
commit 053727b
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 12 deletions.
diff --git a/HELP.md b/HELP.md
@@ -29,7 +29,7 @@ DATA
     __title__ = 'denonavr'
 
 VERSION
-    0.9.6
+    0.9.7
 
 ====================================================================================
 

diff --git a/README.md b/README.md
@@ -1,10 +1,10 @@
 # denonavr
-[![Version](https://img.shields.io/badge/version-v0.9.6-orange.svg)](https://github.com/scarface-4711/denonavr)
+[![Version](https://img.shields.io/badge/version-v0.9.7-orange.svg)](https://github.com/scarface-4711/denonavr)
 [![Build Status](https://travis-ci.com/scarface-4711/denonavr.svg?branch=master)](https://travis-ci.com/scarface-4711/denonavr)
 [![PyPi](https://img.shields.io/pypi/v/denonavr.svg)](https://pypi.org/project/denonavr)
 [![License](https://img.shields.io/github/license/scarface-4711/denonavr.svg)](LICENSE)
 
-Automation Library for Denon AVR receivers - current version 0.9.6
+Automation Library for Denon AVR receivers - current version 0.9.7
 
 ## Installation
 

diff --git a/README.rst b/README.rst
@@ -4,7 +4,7 @@ denonavr
 .. |Build Status| .. image:: https://travis-ci.com/scarface-4711/denonavr.svg?branch=master
     :target: https://travis-ci.com/scarface-4711/denonavr
 
-Automation Library for Denon AVR receivers - current version 0.9.6
+Automation Library for Denon AVR receivers - current version 0.9.7
 
 Installation
 ------------

diff --git a/denonavr/__init__.py b/denonavr/__init__.py
@@ -17,7 +17,7 @@
 logging.getLogger(__name__).addHandler(logging.NullHandler())
 
 __title__ = "denonavr"
-__version__ = "0.9.6"
+__version__ = "0.9.7"
 
 
 def discover():

diff --git a/denonavr/audyssey.py b/denonavr/audyssey.py
@@ -95,9 +95,9 @@ def update(self):
         if response is None:
             return False
 
-        try:
-            audyssey_params = response.find("./cmd/list")
-        except (AttributeError, IndexError, TypeError):
+        audyssey_params = response.find("./cmd/list")
+
+        if audyssey_params is None:
             return False
 
         for param in audyssey_params:
@@ -132,8 +132,11 @@ def _set_audyssey(self, parameter, value):
         if response is None:
             return False
 
-        if response.find("cmd").text == "OK":
-            return True
+        try:
+            if response.find("cmd").text == "OK":
+                return True
+        except AttributeError:
+            pass
 
         return False
 

diff --git a/doc/XML_data_dump.txt b/doc/XML_data_dump.txt
@@ -164,7 +164,7 @@ Upon success the AVR replies with: (needs more testing)
 <?xml version="1.0" encoding="utf-8"?>
 <tx>
   <cmd id="3">
-    <name>GetAudyssyInfo</name>
+    <name>GetAudysseyInfo</name>
     <list>
       <param name="eqname"></param>
       <param name="eqvalue"></param>

diff --git a/setup.py b/setup.py
@@ -4,7 +4,7 @@
 from setuptools import find_packages, setup
 
 setup(name='denonavr',
-      version='0.9.6',
+      version='0.9.7',
       description='Automation Library for Denon AVR receivers',
       long_description='Automation Library for Denon AVR receivers',
       url='https://github.com/scarface-4711/denonavr',
@@ -27,4 +27,5 @@
           "Programming Language :: Python :: 3.6",
           "Programming Language :: Python :: 3.7",
           "Programming Language :: Python :: 3.8",
+          "Programming Language :: Python :: 3.9"
           ])
-Original file line number
+Diff line change
@@ Expand Up / @@ -29,7 +29,7 @@ DATA @@
         __title__ = 'denonavr'
     VERSION
-.9.6
+.9.7
     ====================================================================================
@@ Expand Down @@