ERP5LoginUserManager: avoid pass id as None because recent ZODB's BTree reject None as a key

product/ERP5Security/ERP5LoginUserManager.py

@@ -267,7 +267,7 @@ class ERP5LoginUserManager(BasePlugin):
       # users so code checking if a user login exists before allowing it to be
       # reused, preventing misleading logins from being misused.
       result.append({
-        'id': None,
+        'id': special_user_name,
         'login': special_user_name,
         'pluginid': plugin_id,
 

[Jérome Perrin]

I don't think this change is harmful, if I understand well the intention of this block was to return zope users to make sure they cannot be taken in this plugin (shouldn't it be another "special users" plugin ?).

My concern is that after this change the whole comment above becomes wrong, because there's no reason about complaining that these users have user id None and setting them a user id.

I will try to revert this change and see which BTree breaks.