SUMMARY Choosing "offline" in the combobox in ktp-contactlist does nothing. Switching to offline mode is possible through the desktop-applet, though. STEPS TO REPRODUCE 1. Open ktp-contactlist 2. Verify that you are online 3. Try to go offline using the combobox in ktp-contactlist OBSERVED RESULT Status is still online, as shown by the desktop-applet. Also contacts are still shown in the contact list and I am able to exchange text messages with my contacts. EXPECTED RESULT Connection status should change to offline. SOFTWARE/OS VERSIONS Windows: MacOS: Linux/KDE Plasma: Arch linux packages, also tested git-master of ktp-contactlist (available in About System) KDE Plasma Version: 18.12.2 KDE Frameworks Version: 5.55.0 Qt Version: 5.12.1 ADDITIONAL INFORMATION Caused by the index check in GlobalPresenceChooser::onUserActivatedComboChange(). Patch at https://phabricator.kde.org/differential/diff/51686/ fixes it most of the time, though in some cases (10-20% of times) it will go offline and go back online immediately afterwards. I can't find the reason for this right now...
Same here. The patch looks good and the original code looks weird. The return on ((index == count() - 3) introduced by https://git.reviewboard.kde.org/r/130189/ What is the reason for the extra condition? (David, James?)
Yeah, that condition makes no sense. Even if it did, that's a terrible way to code it with just arbitrary indexes.
(In reply to Alexander Blinne from comment #0) > Patch at https://phabricator.kde.org/differential/diff/51686/ fixes it most > of the time, though in some cases (10-20% of times) it will go offline and > go back online immediately afterwards. I can't find the reason for this > right now... I applied your patch and the Presence starts to work but it seems that there is some race-condition so if I switch to online and quickly go back to offline before a Connection Manager connected then it has some chance to report "I'm connected" before it was told to stop trying. Would you mind to submit the fix? This patch is a definitely the right thing to do even if the issue still reproducible. There is no other way to fix it and I'm hitting the issue on my own so I would like to have your fix in 18.12.3 and we have literally one day to do that as the release tag will be made at the Monday. If you can't submit the patch today then I would like to commit it with you as the author and me as a committer.
(In reply to Alexandr Akulich from comment #3) > If you can't submit the patch today then I would like to commit it with you > as the author and me as a committer. I don't think I can submit the patch as I don't have a developer account, am I right? I don't have much experience with the KDE workflow, unfortunately...
Git commit 6dcdbdc872c2d8fb8e7838dcf5880c5b27b93c47 by Alexander Akulich, on behalf of Alexander Blinne. Committed on 03/03/2019 at 22:43. Pushed by akulichalexandr into branch 'Applications/18.12'. Restore the "Offline" option in presence combobox There is no valid reason to show the option but do nothing on activated. Reviewed By: akulichalexander M +1 -1 global-presence-chooser.cpp https://commits.kde.org/ktp-contact-list/6dcdbdc872c2d8fb8e7838dcf5880c5b27b93c47