Port from QStandardPaths::DataLocation to QStandardPaths::AppDataLocation

They are the same on Linux (but DataLocation is deprecated). On Windows this enables the use of the roaming path.

NO_CHANGELOG

effects/cube/cube.kcfg

@@ -34,7 +34,7 @@
             <default code="true">QColor(KColorScheme(QPalette::Active, KColorScheme::Window).background().color())</default>
         </entry>
         <entry name="CapPath" type="String">
-            <default code="true">QStandardPaths::locate(QStandardPaths::DataLocation, QStringLiteral(&quot;cubecap.png&quot;))</default>
+            <default code="true">QStandardPaths::locate(QStandardPaths::AppDataLocation, QStringLiteral(&quot;cubecap.png&quot;))</default>
         </entry>
         <entry name="TexturedCaps" type="Bool">
             <default>true</default>

effects/trackmouse/trackmouse.cpp

@@ -267,8 +267,8 @@ void TrackMouseEffect::slotMouseChanged(const QPoint&, const QPoint&,
 
 void TrackMouseEffect::loadTexture()
 {
-    QString f[2] = {QStandardPaths::locate(QStandardPaths::DataLocation, QStringLiteral("tm_outer.png")),
-                    QStandardPaths::locate(QStandardPaths::DataLocation, QStringLiteral("tm_inner.png"))};
+    QString f[2] = {QStandardPaths::locate(QStandardPaths::AppDataLocation, QStringLiteral("tm_outer.png")),
+                    QStandardPaths::locate(QStandardPaths::AppDataLocation, QStringLiteral("tm_inner.png"))};
     if (f[0].isEmpty() || f[1].isEmpty())
         return;
 

[Vlad Zahorodnii]

On Windows this enables the use of the roaming path.

Well, we don't advertise the support for windows in kwin. ;-)

---

Thank you for fixing it.

[David Faure]

I know, this was the same commit log I used in all repos where I ported this, to explain why the change was made.

For Unix-only code, it means the change preserves behavior 100%.