Forum OpenACS Q&A: various bugs in content repository

Posted by Deds Castillo on
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

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.

Posted by Dave Bauer on
Where are these bugs on HEAD or 5.1 branch?
Posted by Deds Castillo on

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.

Posted by Lars Pind on
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.