Convert profile "Recent Items" to jsonindex

Bug #1650317 reported by Paul Everitt
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KARL4
Fix Released
Medium
Jim Fulton

Bug Description

Carlos will work with Jim to convert the catalog query for this view into the new jsonindex-style query.

Constraints:

- Don't remove yet the previous implementation. Instead, use the new one.
- Like Carlos's "sqlcatalogification", we're doing raw SQL instead of SQLAlchemy

Let's do this in a way where Jim learns some, but not much, about KARL views. And Carlos learns some about these new queries, so that later, he can convert more views (or help convert the catalog itself if we go that route.)

I'm not sure the test implications on this. We could use some unit test coverage, obviously. But it seems we could use some integration test coverage as well.

Carlos, you could start by making a branch that tries to isolate out the existing query and makes it easy to have a different query. It would be great if we had a function where Jim didn't have to know anything about the KARL pile and could just take input parameters, do a query, and return an iterator or generator or something.

Carlos, you could also start on the test part. Jim won't have his other stuff (LP #1650315) done this week, so you're just being abstract.

Changed in karl4:
importance: Undecided → Medium
assignee: nobody → Carlos de la Guardia (cguardia)
milestone: none → 026
Revision history for this message
Carlos de la Guardia (cguardia) wrote :

I believe Jim has started work on this?

Changed in karl4:
status: New → In Progress
Changed in karl4:
assignee: Carlos de la Guardia (cguardia) → Jim Fulton (jim-zope)
milestone: 026 → 027
Revision history for this message
Jim Fulton (jim-zope) wrote :

I believe this is done.

Changed in karl4:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.