Fixes multiple bugs related to BBEditor and time signatures

[sdasda7777]

This pull request fixes multiple bugs related to BBEditor and time signatures, stemming from incorrect or outdated usage of method getBar():

• Lines 212 and 228: Fix the issue of incorrect BB track being displayed when time signature is not n/n (Changing time signature causes incorrect BB track to be displayed in the BBEditor #6079)
• Line 575: Fixes the issue that if new BB subtrack is added when time signature isn't n/n, due to the way getBar() works the result wouldn't be the one the formula using it was hoping for, and, for example, subtrack added at time signature 8/4 would only appear on every other BB track (subtrack for track 1 would appear on track 2, for 2 on 4, etc.).

[LmmsBot]

🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩

Windows

• Windows 32-bit: lmms-1.3.0-alpha.1.160+ge5b900752-mingw-win32.exe (build link)
• Windows 64-bit: lmms-1.3.0-alpha.1.160+ge5b900752-mingw-win64.exe (build link)

macOS

• macOS : lmms-1.3.0-alpha.1.160+ge5b900752-mac10.14.dmg (build link)

🤖

{"platform_name_to_artifacts": {"Windows": [{"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://output.circle-artifacts.com/output/job/bbe1645b-f462-4adf-9b93-7e1bb2e71891/artifacts/0/lmms-1.3.0-alpha.1.160+ge5b900752-mingw-win32.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/17609?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://output.circle-artifacts.com/output/job/6bbbc35a-a91e-4dad-a80b-fdac4660be08/artifacts/0/lmms-1.3.0-alpha.1.160+ge5b900752-mingw-win64.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/17607?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "macOS": [{"artifact": {"title": {"title": "", "platform_name": "macOS"}, "link": {"link": "https://output.circle-artifacts.com/output/job/949cbbbe-c60a-4900-bc3f-3d51a2df766f/artifacts/0/lmms-1.3.0-alpha.1.160+ge5b900752-mac10.14.dmg"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/17608?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}]}, "commit_sha": "66d2886be1d878f0bc1c1b39141573832b1e4f7c"}

[PhysSong]

It seems like Track::createTCOsForBB needs modification as well to make the behavior consistent.

[sdasda7777]

You're right, that seems to do the trick

[rubiefawn]

I fixed the comically absurd diff† by cherry-picking‡ the two commits that actually matter for this PR onto current master branch.

• 1c8aff1 -> 79c3c54
• 5e40405 -> Removed (see Fixes multiple bugs related to BBEditor and time signatures #6096 (comment))
• 66d2886 -> 3f0c3ba

†

‡ Rewrote the equivalent, since files and classes have all been renamed since the original commits were made lol