From 8934a7d832b8e1375f0bbdd71031f37ae42f3929 Mon Sep 17 00:00:00 2001 From: Kelly Brazil Date: Mon, 4 Nov 2019 11:10:58 -0800 Subject: [PATCH] fix dictionary iteration --- jc/parsers/arp.py | 2 +- jc/parsers/df.py | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/jc/parsers/arp.py b/jc/parsers/arp.py index bde1598b..f271eacd 100644 --- a/jc/parsers/arp.py +++ b/jc/parsers/arp.py @@ -60,7 +60,7 @@ $ arp -a | jc --arp -p def process(proc_data): # in BSD style, change name to null if it is a question mark for entry in proc_data: - if entry['name'] and entry['name'] == '?': + if 'name' in entry and entry['name'] == '?': entry['name'] = None return proc_data diff --git a/jc/parsers/df.py b/jc/parsers/df.py index 60702849..a1b8edfe 100644 --- a/jc/parsers/df.py +++ b/jc/parsers/df.py @@ -56,7 +56,7 @@ def process(proc_data): entry[k] = None # change 'used' to int - if entry['used']: + if 'used' in entry: try: used_int = int(entry['used']) entry['used'] = used_int @@ -64,7 +64,7 @@ def process(proc_data): entry['used'] = None # change 'available' to int - if entry['available']: + if 'available' in entry: try: available_int = int(entry['available']) entry['available'] = available_int @@ -72,7 +72,7 @@ def process(proc_data): entry['available'] = None # remove percent sign from 'use_percent' and change to int - if entry['use_percent']: + if 'use_percent' in entry: try: use_percent_int = entry['use_percent'].rstrip('%') use_percent_int = int(use_percent_int)