Merge pull request #1 from danasmera-clone/master

Made script PEP8 compliant

Author: danasmera

show_firefox_bookmarks.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python
 '''extract a list of URLs from Firefox exported bookmars JSON file '''
- 
 __author__ = "Daniel T."
 __license__ = "GPL"
 __version__ = "0.1.0"
@@ -12,6 +11,7 @@
 import json
 import io
 
+
 def Usage():
     print "{0} Path-to-bookmarks-file".format(sys.argv[0])
     sys.exit(1)
@@ -26,33 +26,32 @@ def Usage():
     print "{0} not found.".format(bookmark_file)
     sys.exit(1)
 
-# Load JSON file 
+# Load JSON file
 fp_data = io.open(bookmark_file, encoding='utf-8')
 try:
     jdata = json.load(fp_data)
 except ValueError:
     print "{0} not valid JSON file".format(bookmark_file)
     sys.exit(1)
 fp_data.close()
- 
- 
 #Recursive function to get the title and URL keys from JSON file
- 
+
+
 def grab_keys(bookmarks_data, bookmarks_list=[]):
-  if 'children' in bookmarks_data:
-    for item in bookmarks_data['children']:
-      bookmarks_list.append({'title': item.get('title', 'No title'),
-                             'uri': item.get('uri', 'None')})
-      grab_keys(item, bookmarks_list)
-  return bookmarks_list
- 
- 
+    if 'children' in bookmarks_data:
+        for item in bookmarks_data['children']:
+            bookmarks_list.append({'title': item.get('title', 'No title'),
+                                   'uri': item.get('uri', 'None')})
+            grab_keys(item, bookmarks_list)
+    return bookmarks_list
+
+
 def main():
-  mydata=grab_keys(jdata)
-  for item in mydata:
-    myurl = item['uri']
-    if myurl.startswith('http') or myurl.startswith('ftp'):
-      print item['uri'], "  ", item['title']
- 
-if __name__=="__main__":
-  main()
+    mydata = grab_keys(jdata)
+    for item in mydata:
+        myurl = item['uri']
+        if myurl.startswith('http') or myurl.startswith('ftp'):
+            print item['uri'], "  ", item['title']
+
+if __name__ == "__main__":
+    main()