Commit 47a15045 authored by Vincent Pelletier's avatar Vincent Pelletier Committed by Boxiang Sun

all: Avoid remaining direct calls to {recursiveI,i}mmediateReindexObject

Also, simplify scripts while doing so.
parent 5f0c0f5f
""" This script reindex all the objects created before updating local roles """ """ This script reindex all the objects created before updating local roles """
module_list = ['document_module',
'image_module',
'knowledge_pad_module',
'organisation_module',
'person_module',
'review_module',
'test_page_module',
'web_page_module',
'web_site_module']
context.portal_types.Folder_reindexAll()
portal = context.getPortalObject() portal = context.getPortalObject()
for module_id in module_list: portal.portal_types.Folder_reindexAll()
stack = [getattr(portal, module_id)] stack = [
for obj in stack: portal.document_module,
for child in obj.objectValues(): portal.image_module,
stack.append(child) portal.knowledge_pad_module,
obj.updateLocalRolesOnSecurityGroups() portal.organisation_module,
portal.person_module,
portal.review_module,
portal.test_page_module,
portal.web_page_module,
portal.web_site_module,
]
for obj in stack:
stack.extend(obj.objectValues())
obj.updateLocalRolesOnSecurityGroups()
...@@ -46,13 +46,8 @@ if person_list: ...@@ -46,13 +46,8 @@ if person_list:
# create Person account # create Person account
person_module = portal.getDefaultModule(portal_type='Person') person_module = portal.getDefaultModule(portal_type='Person')
person = person_module.newContent(portal_type='Person', **kw) person = person_module.newContent(reindex_on_commit=True, portal_type='Person', **kw)
person.validate() person.validate()
# do not immediate reindex object
# this means that when creating an account the new one will *NOT*
# be available immediately and we should consider sending two email to user
# that 1) his account will be created and when created 2)-> send account info
#person.immediateReindexObject()
# Create default career # Create default career
person.newContent(portal_type='Career', person.newContent(portal_type='Career',
......
...@@ -2,12 +2,5 @@ foo1 = context.foo_module.newContent(portal_type='Foo') ...@@ -2,12 +2,5 @@ foo1 = context.foo_module.newContent(portal_type='Foo')
foo2 = context.foo_module.newContent(portal_type='Foo') foo2 = context.foo_module.newContent(portal_type='Foo')
foo1.setTitle('hasAccessUnauthorized') foo1.setTitle('hasAccessUnauthorized')
foo1.setSuccessorValue(foo2) foo1.setSuccessorValue(foo2)
foo2.activate().manage_permission('Access contents information', 'Manager', 0)
foo1.immediateReindexObject()
foo2.immediateReindexObject()
foo2.activate().manage_permission( 'Access contents information', 'Manager', 0)
return 'Done' return 'Done'
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment