Forum OpenACS Q&A: Re: Search package with xowiki

Collapse
3: Re: Search package with xowiki (response to 2)
Posted by Eduardo Santos on 01/09/07 02:30 PM
Thank you very much Gustaf. The procedure ::xowiki::fix_all_package_ids really fixed the problem, but I still have a concern: I'm using the OpenACS version 5.2.3, which is the last stable one. Is there any patch that I have to apply to correct this problem with the package_id? Is there any other core package for which I'm going to have the same problem?

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.

Collapse
4: Re: Search package with xowiki (response to 3)
Posted by Gustaf Neumann on 01/09/07 02:36 PM
what happens, if you add a new page, and maybe edit it. do the new entries have the package_id? if yes, than everything is ok, which should be the case in 5.2.3...
Collapse
5: Re: Search package with xowiki (response to 4)
Posted by Eduardo Santos on 01/09/07 05:36 PM
Let me tell you what I did now. It seems like the problem persists.

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?

Collapse
6: Re: Search package with xowiki (response to 5)
Posted by Eduardo Santos on 01/09/07 05:40 PM
I forgot something. When I edit a page that already had the package_id stored, the new revision loses the package_id, as you presumed.
Collapse
7: Re: Search package with xowiki (response to 5)
Posted by Eduardo Santos on 01/09/07 05:45 PM
Just another question, if you don't mind. In the file search/www/search.tcl, in the line 111 there's the following code: 
# 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]
    }
}
What's the purpose for this scope var? I couldn't find any definition for it, and I'm losing the the package_id in the search results page. So, if I have to go to page number two in my results, the next page doesn't take the package_id, and the search is performed in the entire site.
Collapse
8: Re: Search package with xowiki (response to 6)
Posted by Gustaf Neumann on 01/09/07 11:41 PM
This was a very nasty problem that took me quite a while to find. Apparently, the problem exists only on sites updating since many versions, but not on fresh installs of oacs-5-2 from the repository. The same problem exist(ed) on openacs.org, where i first updated openacs to the latest fixes of openacs 5.2. But this did not help either.

Apparently the long running sites have "only" 4 versions of content_revision__new (do a \df content_revision__new), while new installs have 7.

To fix the problem, source from psql packages/acs-content-repository/sql/postgresql/content-revision.sql
and ignore the errors at the end. Do one more ::xowiki::fix_all_package_ids, and you are set.

It looks as if there is somewhere a missing update script.

Collapse
9: Re: Search package with xowiki (response to 7)
Posted by Gustaf Neumann on 01/10/07 12:42 AM
i thought, you are using openacs 5.2? the code above is from 5.3....

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.

Collapse
10: Re: Search package with xowiki (response to 8)
Posted by Eduardo Santos on 01/10/07 12:26 PM
Thank you very much Gustaf; it seems to work right now. Aparently you are right about the missing upgrade script, but I couldn't find any new definition for the function content_revision__new in any of the upgrades. I tried to see if in some of the revisions for the file content-revision.sql somebody put more definitions for the function, and I couldn't find it either. The best thing I found is between the dotlrn 2-2-1 final and the HEAD. You can see the diff:

http://cvs.openacs.org/cvs/openacs-4/packages/acs-content-repository/sql/postgresql/content-revision.sql?r1=1.45&r2=1.35

I don't think it's related to this bug anyway, but maybe it's worth to take a look. However, sourcing the file seems to solve everything right now. My search is working just fine.

Collapse
11: Re: Search package with xowiki (response to 9)
Posted by Eduardo Santos on 01/10/07 12:40 PM
I'm using oacs 5-2, but I've installed the search package from the HEAD, because there are a lot of new search procs wich I wanted to use. As it doesn't depend on any other packages, I can use it without any problem.

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.

Collapse
12: Re: Search package with xowiki (response to 11)
Posted by Gustaf Neumann on 01/10/07 01:40 PM
good news.

please report the bug + fix for search.tcl via bug-tracker, it might help some pleople and makes sure, changes are nit forgotten.

Collapse
13: Re: Search package with xowiki (response to 12)
Posted by Eduardo Santos on 01/10/07 05:37 PM
Bug reported and patch uploaded in bugtracker:

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