Forum OpenACS Q&A: various bugs in content repository

Collapse
1: various bugs in content repository
Posted by Deds Castillo on 04/07/04 07:36 PM
While doing some BCMS stuff, I found the following bugs on content repository:

- broken content_revision__copy due to the merge/remerge as done on TIP #53
- broken content_item__move (i'm guessing TIP #53 also)
- (HIGH) broken trigger on update of cr_items as a result of a typo

I posted patch here https://openacs.org/bugtracker/openacs/patch?patch%5fnumber=492

Aside from the above this also adds a content_extlink__get_label.

Note that content_item__move tries to call content_folder__move with 3 params.  There is a purpose for that but the function is not defined so this function still needs to be defined and the original 2 param content_folder__move be recreated as a wrapper.

Collapse
2: Re: various bugs in content repository (response to 1)
Posted by Dave Bauer on 04/07/04 07:56 PM
Where are these bugs on HEAD or 5.1 branch?
Collapse
3: Re: various bugs in content repository (response to 1)
Posted by Deds Castillo on 04/08/04 05:29 PM
Dave,

They're in HEAD.  Patch is also for HEAD.  I updated checkout just before I posted.  I haven't checked on 5.1 but the cr_items trigger might be there as it's present on an upgrade script.

Collapse
4: Re: various bugs in content repository (response to 1)
Posted by Lars Pind on 04/17/04 10:10 PM
Hi Deds,

It would be absolutely fabulous, if you would have a chance to contribute automated test cases for these bugs as well, so we can ensure they don't happen again in the future.

Thanks for the patch.

/Lars