Manually merge PR123 to address a python3-specific bug in edituser and editgdpuser as reported by AU.

git-svn-id: svn+ssh://svn.code.sf.net/p/migrid/code/trunk@6141 b75ad72c-e7d7-11dd-a971-7dbc132099af

Author: jonasbardino

mig/server/editgdpuser.py

@@ -4,7 +4,7 @@
 # --- BEGIN_HEADER ---
 #
 # editgdpuser - Edit a MiG GDP user
-# Copyright (C) 2003-2022  The MiG Project lead by Brian Vinter
+# Copyright (C) 2003-2024  The MiG Project lead by Brian Vinter
 #
 # This file is part of MiG.
 #
@@ -146,9 +146,9 @@ def usage(name='editgdpuser.py'):
         sys.exit(1)
 
     # Remove empty value fields
-
-    for (key, val) in user_dict.items():
-        if not val:
+    # NOTE: force list copy here as we delete inline below
+    for key in list(user_dict):
+        if not user_dict[key]:
             del user_dict[key]
 
     if account_state:

mig/server/edituser.py

@@ -4,7 +4,7 @@
 # --- BEGIN_HEADER ---
 #
 # edituser - Edit a MiG user
-# Copyright (C) 2003-2022  The MiG Project lead by Brian Vinter
+# Copyright (C) 2003-2024  The MiG Project lead by Brian Vinter
 #
 # This file is part of MiG.
 #
@@ -167,9 +167,9 @@ def usage(name='edituser.py'):
         user_dict['role'] = role
 
     # Remove empty value fields
-
-    for (key, val) in user_dict.items():
-        if not val:
+    # NOTE: force list copy here as we delete inline below
+    for key in list(user_dict):
+        if not user_dict[key]:
             del user_dict[key]
 
     if verbose: