Acq: general search can't always find invoices

Bug #927685 reported by Lebbeous Fogle-Weekley on 2012-02-06
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Lebbeous Fogle-Weekley

Bug Description

Observed by Ben Ostrowsky and Bill Erickson, affecting Evergreen 2.0+:

If you have an invoice that is related to a purchase order only via invoice *items* (which in Evergreen correspond to misc charges, taxes, etc) and not through any invoice *entries* (which correspond to lineitems - books and book-like things), then you won't see that invoice when using the Acquisitions -> General Search and searching for invoices by a field like Purchase Order ID.

I will refactor the core query used by Acq General Search to find invoices by any of their possible relationships to other Acq objects, rather than exclusively via the acq.invoice_entry table.

Here's a solution. Testing appreciated.

    Acq: improve General Search's ability to find invoices

    Invoices in Evergreen have complex relationships with other items. They
    can be related to lineitems (and ultimately POs and selection lists) by
    invoice *entries*, or to PO items (and ultimately etc etc) by invoice
    *items*, or directly to purchase orders by either of the above mentioned
    acq.invoice_{entry,item} objects.

    This should make general search more able to find invoices related to
    other objects whose fields you might search against in Acquistions
    General search.

    Here's a diagram that I created to help me think about how to write
    these joins:

    At Bill Erickson's suggestion, I wound up putting the joins into an IDL view
    ("acqus") rather than trying to express them in JSON query, which was turnin
    into a real time sink.

    To prevent this change from making Acq General Search slower than it
    was, I've also added indices on all the foreign keys used in the IDL
    view, so that the joins should be as fast as possible.;a=shortlog;h=refs/heads/user/senator/lp-927685

tags: added: pullrequest

The google docs link above was cut off. Here's the whole thing:

Heads up if you've already fetched that branch in the past 10 minutes; just force-pushed small corrections.

Bill Erickson (berick) wrote :

Tested and signed off on the main commit. I've also pushed a follow up commit which fixes the jump-to-invoices button in the PO page. Both commits can be found in working/collab/berick/lp-927685

Changed in evergreen:
status: New → In Progress
Changed in evergreen:
status: In Progress → New
milestone: 2.2.0alpha2 → 2.2.0beta1
Mike Rylander (mrylander) wrote :

Merged to master!

Mike Rylander (mrylander) wrote :

However, 2.1 has merge conflicts. Leaving it in progress as a result.

Changed in evergreen:
milestone: 2.2.0beta1 → none
status: New → In Progress

Lost track of this for a while, but bug #983195 addresses problems in the code that was merged to master but never merged to 2.1 and 2.0.

I am marking this bug as wontfix for 2.0 and 2.1, and will fix everything all at once on bug #983195.

Changed in evergreen:
status: In Progress → Fix Committed
Ben Shum (bshum) on 2012-07-01
Changed in evergreen:
status: Fix Committed → Fix Released
milestone: none → 2.2.0
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers