Shawn Walker wrote:
Brock Pytlik wrote:

Have you checked modules/server/api.py search() and the web-based search in the templates provided by the depot to be certain that everything is still works correctly? I would think this would be a bump in the version of pkg.server.api at the least.
I'll check. All I need to do is start a repo, point a browser at it and see what happens when I do a search?

Yes, but don't forget to set your --content-root to your proto/root_`uname -p`/usr/share/lib/pkg directory.

Also, please make one last sweep to ensure you're not referencing "authority" or "auth" anywhere; those should both be "publisher" or "pub" respectively.

Cheers,
Gonna need your help here to get this working as the template files apparently need to be changed and I can't really parse mako at this point. Guidance on how to translate the new code in client (ie process_v_1_search and the loop around it) into code that mako understands would be appreciated.

This should be the rough process:

src/modules/server/api.py
===================
* increment the CURRENT_API_VERSION variable

* Update the search methods in CatalogInterface to use the new search code, removing ones that no longer apply (such as search_done())

src/web/index.shtml
===================
* increment the CLIENT_API_VERSION variable

src/web/en/base.html
===================
* increment the CLIENT_API_VERSION variable
Strangely, this doesn't seem to actually need to be done.

src/web/search.shtml
===================
* go to about line 87 and change the call to catalog.search() as needed

* go to line 111 and adjust the code here, note that it expects the returned values to be in the same format as they used to be when you called pkg.server.catalog.Catalog.search()
Yeah, that's no longer true.
I've got it working so that it returns packages.

You don't need to worry about adding the ability to switch between showing the results as packages or as actions. I'll take care of that. Just get it working again where it shows actions.

Cheers,
I'll be sending out a webrev to cover this bit once I'm sure it's again passing the test suites.
Brock
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to