it is dangerous to set defaults to mutable objects like [] or {}

plus, we don't actually use this

diff --git a/sfa/client/sfaadmin.py b/sfa/client/sfaadmin.py
index 2ddf609..cf88a3a 100755 (executable)
--- a/sfa/client/sfaadmin.py
+++ b/sfa/client/sfaadmin.py
@@ -81,9 +81,9 @@ class RegistryCommands(Commands):
             save_records_to_file(outfile, records)  
 
 
-    def _record_dict(self, xrn=None, type=None, email='', key=None, 
-                     slices=[], researchers=[], pis=[], 
-                     url=None, description=None, extras={}):
+    def _record_dict(self, xrn, type, email, key, 
+                     slices, researchers, pis, 
+                     url, description, extras):
         record_dict = {}
         if xrn:
             if type: