D16945: Fix fallback to built-in ui

David Edmundson noreply at phabricator.kde.org
Sun Nov 18 17:05:35 GMT 2018


davidedmundson added a comment.


  Ship it if you like, but I think there's a nicer solution.

INLINE COMMENTS

> greeterapp.cpp:313
>          // on error, load the fallback lockscreen to not lock the user out of the system
> -        if (view->status() == QQmlComponent::Error) {
> +        if (view->status() == QQmlComponent::Error || m_mainQmlPath.isEmpty()) {
>              static const QUrl fallbackUrl(QUrl(QStringLiteral("qrc:/fallbacktheme/LockScreen.qml")));

> view->status() != Ready

should also work, and that covers the case of m_mainQmlPath being set but pointing to an empty file.

We'll never be loading as all files should be local.

REPOSITORY
  R133 KScreenLocker

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

To: graesslin, #plasma
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181118/f472412d/attachment-0001.html>


More information about the Plasma-devel mailing list