Forum OpenACS Development: Re: Creating a Trash for generic objects

Collapse
Posted by Dave Bauer on
The idea is good, but the implementation is completely wrong.

Context_id determines permissions inheritance. You can't go changing the permissions of the object without considereing the consequences. When you add an item back from the trash the permissions may be changed again.

So you would really need to think about how this was implemented.

Also packages in general do not use the context_id to determine if an object appears. In general either the package_id or some sort of parent object (cr_items.parent_id in the content repository) is referenced.

One of the ways I have been thinking about fixing this is to make context_id be used as the parent_id for an object hierarchy. Right now you can not rely on this.