From e08e9b0d32c7d61209370c73cf2314e651534071 Mon Sep 17 00:00:00 2001
From: croneter <croneter@users.noreply.github.com>
Date: Wed, 12 Jun 2019 13:17:17 +0200
Subject: [PATCH] Fix ValueError if casting to int/float

---
 resources/lib/transfer.py | 55 +++++++++++++++++++++++++--------------
 resources/lib/utils.py    | 14 +++++-----
 2 files changed, 43 insertions(+), 26 deletions(-)

diff --git a/resources/lib/transfer.py b/resources/lib/transfer.py
index 0c12ba28..94e5e607 100644
--- a/resources/lib/transfer.py
+++ b/resources/lib/transfer.py
@@ -21,32 +21,47 @@ WINDOW_COMMAND = 'plexkodiconnect.command'.encode('utf-8')
 
 def cast(func, value):
     """
-    Cast the specified value to the specified type (returned by func). Currently this
-    only support int, float, bool. Should be extended if needed.
+    Cast the specified value to the specified type (returned by func). Currently
+    this only support int, float, bool. Should be extended if needed.
     Parameters:
         func (func): Calback function to used cast to type (int, bool, float).
         value (any): value to be cast and returned.
+
+    Returns None if something goes wrong
     """
-    if value is not None:
-        if func == bool:
-            return bool(int(value))
-        elif func == unicode:
-            if isinstance(value, (int, long, float)):
-                return unicode(value)
-            else:
-                return value.decode('utf-8')
-        elif func == str:
-            if isinstance(value, (int, long, float)):
-                return str(value)
-            else:
-                return value.encode('utf-8')
-        elif func in (int, float):
+    if value is None:
+        return value
+    elif func == bool:
+        return bool(int(value))
+    elif func == unicode:
+        if isinstance(value, (int, long, float)):
+            return unicode(value)
+        elif isinstance(value, unicode):
+            return value
+        else:
+            return value.decode('utf-8')
+    elif func == str:
+        if isinstance(value, (int, long, float)):
+            return str(value)
+        elif isinstance(value, str):
+            return value
+        else:
+            return value.encode('utf-8')
+    elif func == int:
+        try:
+            return int(value)
+        except ValueError:
             try:
-                return func(value)
+                # Converting e.g. '8.0' fails; need to convert to float first
+                return int(float(value))
             except ValueError:
-                return float('nan')
-        return func(value)
-    return value
+                return
+    elif func == float:
+        try:
+            return float(value)
+        except ValueError:
+            return
+    return func(value)
 
 
 def kodi_window(property, value=None, clear=False):
diff --git a/resources/lib/utils.py b/resources/lib/utils.py
index 15cf3f8a..9d57917d 100644
--- a/resources/lib/utils.py
+++ b/resources/lib/utils.py
@@ -268,11 +268,13 @@ class AttributeDict(dict):
 
 def cast(func, value):
     """
-    Cast the specified value to the specified type (returned by func). Currently this
-    only support int, float, bool. Should be extended if needed.
+    Cast the specified value to the specified type (returned by func). Currently
+    this only support int, float, bool. Should be extended if needed.
     Parameters:
         func (func): Calback function to used cast to type (int, bool, float).
         value (any): value to be cast and returned.
+
+    Returns None if something goes wrong
     """
     if value is None:
         return value
@@ -294,18 +296,18 @@ def cast(func, value):
             return value.encode('utf-8')
     elif func == int:
         try:
-            return func(value)
+            return int(value)
         except ValueError:
             try:
                 # Converting e.g. '8.0' fails; need to convert to float first
                 return int(float(value))
             except ValueError:
-                return float('nan')
+                return
     elif func == float:
         try:
-            return func(value)
+            return float(value)
         except ValueError:
-            return float('nan')
+            return
     return func(value)