permission plugin: Do not add the ipapermissionv2 for output

As with the flags, the objectclass should be returned as it is
on the entry.

https://fedorahosted.org/freeipa/ticket/4257

Reviewed-By: Martin Kosek <mkosek@redhat.com>
This commit is contained in:
Petr Viktorin 2014-03-17 15:53:06 +01:00 committed by Martin Kosek
parent 7872b92050
commit 3dcad00b94
3 changed files with 6 additions and 7 deletions

View File

@ -624,7 +624,7 @@ class permission(baseldap.LDAPObject):
from the ACI corresponding to ``entry``. from the ACI corresponding to ``entry``.
If None, ``entry`` itself is filled If None, ``entry`` itself is filled
:param output_only: :param output_only:
If true, the flags are not updated to V2. If true, the flags & objectclass are not updated to V2.
Used for the -find and -show commands. Used for the -find and -show commands.
:param cached_acientry: :param cached_acientry:
Optional pre-retreived entry that contains the existing ACI. Optional pre-retreived entry that contains the existing ACI.
@ -665,7 +665,6 @@ class permission(baseldap.LDAPObject):
if not output_only: if not output_only:
target_entry['ipapermissiontype'] = ['SYSTEM', 'V2'] target_entry['ipapermissiontype'] = ['SYSTEM', 'V2']
if 'ipapermissionv2' not in entry['objectclass']: if 'ipapermissionv2' not in entry['objectclass']:
target_entry['objectclass'] = list(entry['objectclass']) + [ target_entry['objectclass'] = list(entry['objectclass']) + [
u'ipapermissionv2'] u'ipapermissionv2']

View File

@ -806,7 +806,7 @@ class test_old_permission(Declarative):
'dn': DN(('cn','Add user to default group'), 'dn': DN(('cn','Add user to default group'),
api.env.container_permission, api.env.basedn), api.env.container_permission, api.env.basedn),
'cn': [u'Add user to default group'], 'cn': [u'Add user to default group'],
'objectclass': objectclasses.permission, 'objectclass': objectclasses.system_permission,
'member_privilege': [u'User Administrators'], 'member_privilege': [u'User Administrators'],
'attrs': [u'member'], 'attrs': [u'member'],
'targetgroup': u'ipausers', 'targetgroup': u'ipausers',

View File

@ -1163,7 +1163,7 @@ class test_permission(Declarative):
'dn': DN(('cn','Add user to default group'), 'dn': DN(('cn','Add user to default group'),
api.env.container_permission, api.env.basedn), api.env.container_permission, api.env.basedn),
'cn': [u'Add user to default group'], 'cn': [u'Add user to default group'],
'objectclass': objectclasses.permission, 'objectclass': objectclasses.system_permission,
'member_privilege': [u'User Administrators'], 'member_privilege': [u'User Administrators'],
'attrs': [u'member'], 'attrs': [u'member'],
'targetgroup': [u'ipausers'], 'targetgroup': [u'ipausers'],