Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OLD-6411: Report - Inventory Levels - Select product doesn't work #78

Closed
cpaschen opened this issue Oct 21, 2012 · 3 comments
Closed

OLD-6411: Report - Inventory Levels - Select product doesn't work #78

cpaschen opened this issue Oct 21, 2012 · 3 comments
Assignees
Labels

Comments

@cpaschen
Copy link
Member

Description

In the Report - Inventory Levels, the Select Product selection doesn't appear to have any effect on the report content.

History

#4 Updated by Chris DioscouriChris Paschen. 8 months ago

Target version changed from Version 0.9.9 - Misc Bug Fixes and New Features to Version 0.9.4 - Plugin Testing and Finalization

#3 Updated by Lukas Polak 9 months ago

Target version changed from Version 0.9.0 - Major Core Fixes to Version 0.9.9 - Misc Bug Fixes and New Features

First, we need to fix #3558.

Lukas

#2 Updated by Chris DioscouriChris Paschen. 9 months ago

Assignee set to Lukas Polak

#1 Updated by Chris DioscouriChris Paschen. 9 months ago

Priority changed from Normal to Low

@ghost ghost assigned efillman Nov 21, 2012
@efillman
Copy link
Member

This bug is strange because the report plug-in has code for searching by product name commented out. Did we try to move towards the product selection drop down? This report uses the products model which does not currently have support for the filter_product_name, I can add that in or just re enable the filter_name, it seems to accomplish the same thing but I am not sure what customers have preferred in the past.

$state['filter_name'] vs $state['filter_product_name']

@ghost ghost assigned rdiaztushman and efillman Dec 11, 2012
@rdiaztushman
Copy link
Member

Try using filter_name, the one that already exists in the products model. I suspect there was a reason for taking it out, but for the life of me I cannot remember.....

@efillman
Copy link
Member

Changed to filter by name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants