enriquem updated this revision to Diff 48614.
enriquem added a comment.

  Treated otp-flags the same way as password-flags. But we have to be careful 
to consider the case where the option is not yet in the config file (as will 
certainly be the case the firs time it is used if there was a vpn already 
configured), so I added a isEmpty() test not to break the toInt() call.

REPOSITORY
  R116 Plasma Network Management Applet

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D17834?vs=48397&id=48614

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

AFFECTED FILES
  vpn/fortisslvpn/fortisslvpnadvanced.ui
  vpn/fortisslvpn/fortisslvpnauth.cpp
  vpn/fortisslvpn/fortisslvpnauth.ui
  vpn/fortisslvpn/fortisslvpnwidget.cpp
  vpn/fortisslvpn/nm-fortisslvpn-service.h

To: enriquem, jgrulich
Cc: plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to