Skip to content

Fix admin search for PurchaseOrderLineItem - #12508

Merged
SchrodingersGat merged 1 commit into
inventree:masterfrom
bhumin18:fix-po-lineitem-admin-search
Jul 31, 2026
Merged

Fix admin search for PurchaseOrderLineItem#12508
SchrodingersGat merged 1 commit into
inventree:masterfrom
bhumin18:fix-po-lineitem-admin-search

Conversation

@bhumin18

@bhumin18 bhumin18 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Expands search_fields on PurchaseOrderLineItemAdmin to allow searching by part name, part description, supplier SKU, order reference, and supplier name.

Motivation

Previously only the optional reference field was searchable, making it impossible to find line items by part or order details in the Django Admin panel.

Searching for a part name like Widget returned 0 results, while the same search on SalesOrderLineItemAdmin works correctly because it has part__name in its search_fields.

Changes

  • src/backend/InvenTree/order/admin.py: Expanded search_fields on PurchaseOrderLineItemAdmin
  • src/backend/InvenTree/order/tests.py: Added unit test to verify search_fields configuration

Expand search_fields on PurchaseOrderLineItemAdmin to allow
searching by part name, part description, supplier SKU,
order reference, and supplier name.

Previously, only the optional 'reference' field was searchable,
making it impossible to find line items by part or order details.

- part__part__name: search by underlying Part name
- part__part__description: search by Part description
- part__SKU: search by Supplier SKU
- order__reference: search by Purchase Order reference
- order__supplier__name: search by Supplier name

Also adds a unit test to verify the search_fields are correctly
configured.
@netlify

netlify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit fc08611
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6a6b3ea94f3ae50008fde5c2

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.72%. Comparing base (20ea122) to head (fc08611).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12508      +/-   ##
==========================================
- Coverage   86.72%   86.72%   -0.01%     
==========================================
  Files        1444     1444              
  Lines       96138    96145       +7     
  Branches    11208    11208              
==========================================
+ Hits        83377    83378       +1     
- Misses      12697    12703       +6     
  Partials       64       64              
Flag Coverage Δ
backend 90.77% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 92.18% <100.00%> (-0.01%) ⬇️
Backend General 93.56% <ø> (ø)
Frontend 79.59% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SchrodingersGat
SchrodingersGat merged commit 5d758bb into inventree:master Jul 31, 2026
42 checks passed
@SchrodingersGat

Copy link
Copy Markdown
Member

@bhumin18 thanks for the contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants