Comment 10 for bug 1093038

Revision history for this message
Marcel van der Boom (HS-Development BV) (mrb) wrote :

The cause of what Paul noticed in comment #8 is that portals are now just groups with a field is_portal set to True.

- first, on line 129 in portal/share_wizard.py a 'group.id' is passed to the method '_create_shared_data_menu'. I think that needs to be 'group' instead of 'group.id'
- second, when passed via method ' _create_shared_data_menu' to method '_sharing_root_menu' an attempt is made to access 'portal.parent_menu_id.id' but the portal (= res.groups) object does not have a 'parent_menu_id' anymore, this is a leftover when portal objects were ' res.portal'

Quite a bit of the code in portal/share_wizard still thinks the portal object *does* have the parent_menu id.

I'm not sure what the fix should be here, but it looks as if a whole piece of the wizard needs reviewing.