The code in current Zope does indeed use a "setattr(classobj, '%s__roles__' % name, access)" instead as proposed by this patch.
The code in current Zope does indeed use a "setattr(classobj, '%s__roles__' % name, access)" instead as proposed by this patch.