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

Shift+F12 should toggle AutoDJ regardless if AutoDJ feature was selecter ealier #9915

Closed
mixxxbot opened this issue Aug 23, 2022 · 3 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: ronso0
Date: 2020-04-04T16:39:35Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1870817
Tags: autodj, library


to reproduce (1):
(make sure there are tracks in the AutoDJ playlist, quit Mixxx)

  • start Mixxx
  • don't click anything, just hit Shift+F12 (toggle AutoDJ)
    = nothing happens*
  • open AutoDJ feature
  • hit Shift+F12
    = AutoDJ starts

to reproduce (2):
(make sure there are tracks in the AutoDJ playlist, quit Mixxx)

  • start Mixxx
  • don't click anything, just hit Shift+F12 (toggle AutoDJ)
    = nothing happens*
  • from anywhere, add track to AutoDJ queue
  • hit Shift+F12
    = AutoDJ starts

*Log message is Queue is empty nowfrom AutoDJProcessor::toggleAutoDJ(bool)
https://github.com/mixxxdj/mixxx/blob/master/src/library/autodj/autodjprocessor.cpp#L399

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-04-04T16:41:51Z


Adding m_pAutoDJTableModel->select() at the very top of AutoDJProcessor::getNextTrackFromQueue() fixes the issue.
https://github.com/mixxxdj/mixxx/blob/master/src/library/autodj/autodjprocessor.cpp#L826

I'm not sure if this the correct fix, but m_pAutoDJTableModel->select(); is also called in DlgAutoDJ::onShow()
https://github.com/mixxxdj/mixxx/blob/master/src/library/autodj/dlgautodj.cpp#L219

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-11-10T17:23:05Z


#3242

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.3.0 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant