Mixxx

BPM sorting doesn't respect the decimal point

Bug #1912112 reported by Marco on 2021-01-17

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Mixxx	Fix Released	Medium	Daniel Schürmann	Mixxx 2.3.0

Bug Description

If you sort the music library by BPM, it seems that it only sorts integers in hierarchical order, but not the values after the decimal point (see attached picture).

Tested with recent commit 7e52e3b9dc5f5d0ca3b2713154c7c67474bc9ac0 (Gentoo Linux x86_64, Qt 5.15.2).

Regards,
Marco

Tags:

Revision history for this message

Marco (c-moon) wrote on 2021-01-17:

BPM sorting with decimal point Edit (4.8 KiB, image/png)

Revision history for this message

ronso0 (ronso0) wrote on 2021-03-02:

confirmed, also in https://mixxx.discourse.group/t/2-3-bpm-sorting-issue/21512

Uwe Klotz (uklotzde-deactivatedaccount) on 2021-03-02

Changed in mixxx:
importance:	Undecided → Medium
status:	New → Confirmed

ronso0 (ronso0) on 2021-03-02

tags:	added: library
Changed in mixxx:
milestone:	none → 2.3.0

Revision history for this message

Daniel Schürmann (daschuer) wrote on 2021-03-02:

This is a regression since 2.2.4 and should be fixed before release.

Changed in mixxx:
assignee:	nobody → Daniel Schürmann (daschuer)

Revision history for this message

Be (be.ing) wrote on 2021-03-02:

Fixing it before releasing 2.3.0 would be nice, but I don't think it should block the release if it comes to that. Fixing it in 2.3.1 would be okay.

Daniel Schürmann (daschuer) on 2021-03-03

Changed in mixxx:
status:	Confirmed → In Progress

Revision history for this message

Daniel Schürmann (daschuer) wrote on 2021-03-03:

https://github.com/mixxxdj/mixxx/pull/3669

ronso0 (ronso0) on 2021-03-03

Changed in mixxx:
status:	In Progress → Fix Committed

Revision history for this message

Marco (c-moon) wrote on 2021-03-04:

I've just compiled and tested the latest commit 6a563459f7fc2f8e077cb634e8ffd794d5642a29 => works as expected. Thanks for fixing this issue in 2.3.0, guys!

Regards,

Marco

Daniel Schürmann (daschuer) on 2021-06-29