Forum OpenACS Q&A: Search package with xowiki

insert into search_observer_queue (select live_revision, now(), 'INSERT' from (
select live_revision from cr_items where content_type like '::xowiki::Page') xowiki);

insert into search_observer_queue (select live_revision, now(), 'INSERT' from (
select live_revision from cr_items where content_type like '::xowiki::Object') xowiki);

insert into search_observer_queue (select live_revision, now(), 'UPDATE' from (
select live_revision from cr_items where content_type like '::xowiki::Page') xowiki);

insert into search_observer_queue (select live_revision, now(), 'UPDATE' from (
select live_revision from cr_items where content_type like '::xowiki::Page') xowiki);

The next step is to perform the search::indexer procedure, wich should index my xowiki pages for search. However, when I try to do that, I get the following error message:

Error: search::indexer: error getting datasource for 1056546
::xowiki::Page: Cannot create object -- illegal name '::' Cannot create object -- illegal name '::'
::xowiki::Package ::xotcl::Class->create
invoked from within
"my create ::$package_id"
(procedure "require" line 10)
::xowiki::Package ::xo::PackageMgr->require
invoked from within
"my require -url $url $package_id"
(procedure "initialize" line 21)
::xowiki::Package ::xo::PackageMgr->initialize
invoked from within
"::xowiki::Package initialize -package_id $package_id -user_id $user_id -parameter $parameter -init_url false -actual_que$ (procedure "instantiate_page_from_id" line 16)
::xowiki::Package->instantiate_page_from_id
invoked from within
"::xowiki::Package instantiate_page_from_id -revision_id $revision_id -user_id 0"
(procedure "::xowiki::datasource" line 4)
invoked from within
"::xowiki::datasource $object_id"
(procedure "AcsSc.ftscontentprovider.datasource.xowiki_page" line 1)
invoked from within
"AcsSc.ftscontentprovider.datasource.xowiki_page 1056546"
("uplevel" body line 1)
invoked from within"uplevel $func_and_args"
(procedure "apply" line 3)
invoked from within
"apply $proc_name $arguments"
(procedure "acs_sc_call" line 6)
invoked from within
"acs_sc_call FtsContentProvider datasource [list $object_id] $object_type"
invoked from within
search::indexer
invoked from within
adp_parse_ad_conn_file
invoked from within
rp_serve_concrete_file /var/lib/aolserver/catir/packages/acs-developer-support/www/shell.adp
invoked from within
rp_serve_abstract_file /var/lib/aolserver/catir/packages/acs-developer-support/www/shell
invoked from within
rp_handler

Any ideas about the problem? I'm using the head version for the search

My concerns have a meaning, because I have a huge and old system. Any upgrade is very difficult to be made, and I usually lose weeks just to test it. So, I don't want to upgrade everything right know. If it's really necessary, I'll do it with the broken package.

1 - I've created a new page in xowiki, inserted it in the search_observer_queue table and then I ran the search::indexer. I couldn't find the page in the search with the package_id.

2 - So I ran the ::xowiki::fix_all_package_ids procedure to create the package_id, did all the index stuff again and the search worked for me.

3 - To be sure about the issue I've created another page and looked for it in the acs_objects table. The page doesn't have the package_id for my xowiki instance. The other ones, for wich I had used the ::xowiki::fix_all_package_ids procedure have the package_id for my xowiki instance stored.

My conclusion is that the package_id is not being stored for new pages. Do I have to schedule the ::xowiki::fix_all_package_ids to run everytime I create a new page in xowiki? Can you tell me wich package is responsible for the bug?

# TODO calculate subsite or dotlrn package_ids
if {![string equal "this" $scope]} {
    # don't send package_id if its not searching this package
    set search_package_id ""
} else {
    set search_node_id [site_node::get_node_id_from_object_id -object_id $search_package_id]
    if {[string equal "dotlrn" [site_node::get_element -node_id $search_node_id -element package_key]]} {
        set search_package_id [site_node::get_children -node_id $search_node_id -element package_id]
    }
}

the code in the search package is not from me. Apparently it is broken. When a package_id is passed in, it should be passed to next etc. as well.

The code works better, if you comment out 'set search_package_id ""' in the cited code. With this change the package_id is passed correctly in the "next" link, but still not in the link for page 2 etc. is missing.

maybe dave can comment on this.

About the query I told you, I did comment it out. There's no reason for that check to be made anyway, and I don't think some bug is coming from there. About the links for the pages it's an easy fix that needs to be made, and I did it already. Thanks for the help again.

https://www.openacs.org/bugtracker/openacs/bug?bug%5fnumber=3061