search

partkeepr / PartKeepr

Open Source Inventory Management
http://www.partkeepr.org
GNU General Public License v3.0
1.4k stars 403 forks source link

Full Text Search doesn't work when SKU is included #960

Open HolgerHeckeroth opened 6 years ago

HolgerHeckeroth commented 6 years ago

Hi Felicia,

I found a bug regarding the full text search, that I could reproduce in your demo system. If I include the distributors SKU in the search via the full text search preferences, parts are not found anymore if they have no Distributor specified.

System Information

How to reproduce

in your demo shop

  1. activate SKU in search configuration
  2. search for a part (e.g. in the resistor 1% category search for '1,2' )
  3. only parts with a distributor show up

Cheers, Holger

BTW: Brilliant software. Just stumbled across it last weekend and I love it.

ely-az commented 6 years ago

I'm was having the same issue and found a work around. If you go to System Preferences->Required Fields->Part Distributor and uncheck all the boxes it will fix this problem. You may have to do the same for the Part Manufacturer section as well. I did both at the same time and it fixed the search issue.

I still think this is a bug though. I have several items in the DB without a distributor and/or manufacturer. I should be able to require certain fields for inputting Manufacturer and Distributor but still have those parts available in search that do not have one or the other listed.

christianlupus commented 4 years ago

@HolgerHeckeroth @ely-az Can you verify this is still an issue on the current branch?

HolgerHeckeroth commented 4 years ago

@christianlupus I stopped using Partkeepr around a year or so. So I'm not able to check if the issue still exists in the current branch.

ely-az commented 4 years ago

Not sure what happened but my work around no longer works. I get zero results from the search engine now? I haven't looked into it much. For now we are using the filter at the bottom which still works fine. I am on GIT dev commit 8a6380e. I see there is a newer commit so will try that out and report back.