From 74d0c3bc711fe8c7d2baa20d6e46c598b22c9fda Mon Sep 17 00:00:00 2001 From: Thierry Parmentelat Date: Wed, 30 Apr 2014 12:43:49 +0200 Subject: [PATCH] it is dangerous to set defaults to mutable objects like [] or {} plus, we don't actually use this --- sfa/client/sfaadmin.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sfa/client/sfaadmin.py b/sfa/client/sfaadmin.py index 2ddf609f..cf88a3a3 100755 --- 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: -- 2.43.0