Merge #68: [GUI] Fix possible division-by-zero bug in superblock time…

… left

83aac5a [GUI] Fix possible division-by-zero bug in superblock time left (Fuzzbawls)

Pull request description:

  Re-work the function that converts seconds to days, hours, minutes to avoid a division-by-zero situation when the time left until the next superblock is less than 1 day.

  Also, no longer show seconds in the text as they are always zero.

ACKs for top commit:
  Liquid369:
    LGTM ACK 83aac5a

Tree-SHA512: 0278d3429b2001840ac7d21736963f0b84469e1ae85d97e8e7188095edf4ee2ed9a6d992d9205c0617dea8a442e771294709504f2f1749811ae8909817b63ee0

src/misc.py

@@ -474,14 +474,14 @@ def saveCacheSettings(cache):
     settings.setValue('cache_isTestnetRPC', cache.get('isTestnetRPC'))
 
 
-def sec_to_time(seconds):
-    days = seconds // 86400
-    seconds %= days * 86400
-    hrs = seconds // 3600
-    seconds %= hrs * 3600
-    mins = seconds // 60
-    seconds %= mins * 60
-    return f"{days} days, {hrs} hrs, {mins} mins, {seconds} secs"
+def sec_to_time(secs):
+    days = secs // 86400
+    hours = (secs - days * 86400) // 3600
+    minutes = (secs - days * 86400 - hours * 3600) // 60
+    result = ("{0} day{1}, ".format(days, "s" if days != 1 else "")) + \
+             ("{0} hour{1}, ".format(hours, "s" if hours != 1 else "")) + \
+             ("{0} minute{1} ".format(minutes, "s" if minutes != 1 else ""))
+    return result
 
 
 def splitString(text, n):