D22363: Remove cursor theme option

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> filipf wrote in advanceconfig.cpp:205
> I need the identifier to delete the entry in the config file (done in sddmauthhelper.cpp), the value is not important so I just set whatever. Any smarter way of doing this?

args[QStringLiteral("kde_settings.conf/Theme/CursorTheme")] = QVariant();

Then in sddmauthhelper, something like

      if (fileName == QLatin1String("kde_settings.conf")) {
              if (iterator.value().isValid()) {
                          sddmConfig->group(groupName).writeEntry(keyName, iterator.value());
              } else {
                                 sddmConfig->group(groupName).deleteEntry(keyName);
  
            }
              sddmOldConfig->group(groupName).deleteEntry(keyName);
  }
          }

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

To: filipf, ngraham, davidedmundson
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190720/b39d39bf/attachment.html>