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

Sort bom list on products is sorted on product ID not alpabetically #18713

Closed
erikvanberkum opened this issue Sep 14, 2021 · 9 comments
Closed
Labels
Bug This is a bug (something does not work as expected)

Comments

@erikvanberkum
Copy link
Contributor

erikvanberkum commented Sep 14, 2021

Bug

When sorting the bom_list on the field products alphabetically, it's sorted on the product ID number and not on ABC. The expected behavior must be sorting the product list alphabetically.

Environment

  • Versions: 14.0.1 15.0.1 15.0.5 15.0.3 16.0.0 17.0.2 18.0.2 19.0.3, 20.0.3
  • OS: Debian
  • Web server: Apache
  • PHP: 7.4
  • Database: mySQL
  • URL(s): /bom/bom_list.php?sortfield=t.fk_product&sortorder=asc

Expected and actual behavior

List sort on ID not on text, its expected to sort on text.

Steps to reproduce the behavior

Go to BOM list and sort on product

@erikvanberkum erikvanberkum added the Bug This is a bug (something does not work as expected) label Sep 14, 2021
@erikvanberkum
Copy link
Contributor Author

I think the issue is related due to missing Foreign Keys:
fk_product on llx_product rowid

same issue is present on MO list
Also not sorting correctly on products or third party.
Same issue here due to expected missing foreign keys on the table

@Hystepik
Copy link
Contributor

Hello,
It works for me in last version of branch 14

@Hystepik Hystepik added the Works for me / Can't reproduce It seems not possible to reproduce the bug. label Sep 23, 2021
@erikvanberkum
Copy link
Contributor Author

@Hystepik can you check again, maybe my explanation was not clear? I installed a fresh 14.0.1 and made 4 products:

see list -> /product/list.php

product_list

A_product which has a technical database ID of 2 etc., I put them in brackets for all products.

Then when sorting the BOM list, on products, we get the following:
/bom/bom_list.php?sortfield=t.fk_product&sortorder=asc

bom_sort

The image shows clearly that the BOM list when sorting on products, it's sorted by product ID and not alphabetically.

Can you remove the label "Works for me / Can't reproduce" and set it to "Bug"? Or guide me what I am doing wrong.

I further have this issue on my production sever running 14.0.2 and in my development version 15 Alpha

@github-actions github-actions bot added the Issue Stale (automatic label) This issue is stale because it has been open 1 year with no activity. Remove this label to keep open label Oct 3, 2022
@erikvanberkum
Copy link
Contributor Author

This issue is still not solved

@github-actions github-actions bot removed the Issue Stale (automatic label) This issue is stale because it has been open 1 year with no activity. Remove this label to keep open label Oct 4, 2022
@Hystepik Hystepik removed the Works for me / Can't reproduce It seems not possible to reproduce the bug. label Jun 6, 2023
@ksar-ksar
Copy link
Contributor

Hello,

Is it still a relevant issue on V18 ?

@erikvanberkum
Copy link
Contributor Author

This issue is still relevant in version 18.0.2. Product colum is ordered by product id and not by product name.

eldy added a commit that referenced this issue Dec 28, 2023
@eldy eldy closed this as completed in 46511ae Dec 29, 2023
@erikvanberkum
Copy link
Contributor Author

erikvanberkum commented Dec 30, 2023

@Hystepik thank you for solving this issue in the mo_list.php but the issue is not solved in bom_list.php

please reopen issue.

@erikvanberkum
Copy link
Contributor Author

Bug not solved.

@hregis
Copy link
Contributor

hregis commented Dec 31, 2024

@erikvanberkum try with this fix : #32497

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected)
Projects
None yet
Development

No branches or pull requests

4 participants