jgrulich added a comment.

  We were actually saving already the value of current BT state, it was just 
not used. Your code assumes there is just one BT adapter, while in theory there 
might be more of them. What I would just is just simply not add property like 
m_tmpBluetoothEnabled and use m_bluetoothAdapters to check if the specific BT 
adapter was enabled before, that's why we save the object path together with 
the value. This should be done in the else if (enable && 
m_bluetoothAdapters.value(objPath)) branch. Just check whether it was enabled 
before and enable it in that case.
  
  Sorry for not looking into this properly before, I'm attending the LAS 
conference and didn't have that much time.

REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D25279

To: ngraham, #plasma, jgrulich
Cc: davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to