From a23acd2f1535a94ef3f7477040890b175c3651b6 Mon Sep 17 00:00:00 2001
From: Aleix Pol <aleixpol@kde.org>
Date: Tue, 17 Sep 2019 00:04:26 +0200
Subject: Fix generating properties that start with an uppercase letter

Summary: We were not adjusting the property name to the getter letter-casing.

Test Plan: See added test, also fixes the issue that made me realize this issue.

Reviewers: #frameworks, davidedmundson

Reviewed By: davidedmundson

Subscribers: ngraham, aacid, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D24010
---
 autotests/kconfig_compiler/test13.cpp.ref |  3 +++
 autotests/kconfig_compiler/test13.h.ref   | 11 +++++++++++
 autotests/kconfig_compiler/test13.kcfg    |  1 +
 3 files changed, 15 insertions(+)

(limited to 'autotests')

diff --git a/autotests/kconfig_compiler/test13.cpp.ref b/autotests/kconfig_compiler/test13.cpp.ref
index 3beea1d0..0f62dc96 100644
--- a/autotests/kconfig_compiler/test13.cpp.ref
+++ b/autotests/kconfig_compiler/test13.cpp.ref
@@ -16,6 +16,9 @@ Test13::Test13(  )
   KConfigCompilerSignallingItem  *itemBrightness;
   itemBrightness = new KConfigCompilerSignallingItem(new KConfigSkeleton::ItemDouble( currentGroup(), QStringLiteral( "brightness" ), mBrightness ), this, notifyFunction, signalBrightnessChanged);
   addItem( itemBrightness, QStringLiteral( "brightness" ) );
+  KConfigSkeleton::ItemBool  *itemStartsWithUppercase;
+  itemStartsWithUppercase = new KConfigSkeleton::ItemBool( currentGroup(), QStringLiteral( "StartsWithUppercase" ), mStartsWithUppercase );
+  addItem( itemStartsWithUppercase, QStringLiteral( "StartsWithUppercase" ) );
 }
 
 Test13::~Test13()
diff --git a/autotests/kconfig_compiler/test13.h.ref b/autotests/kconfig_compiler/test13.h.ref
index 441643f7..6c67fc5d 100644
--- a/autotests/kconfig_compiler/test13.h.ref
+++ b/autotests/kconfig_compiler/test13.h.ref
@@ -47,6 +47,16 @@ class Test13 : public KConfigSkeleton
     }
 
 
+    Q_PROPERTY(bool startsWithUppercase READ startsWithUppercase CONSTANT)
+    /**
+      Get StartsWithUppercase
+    */
+    bool startsWithUppercase() const
+    {
+      return mStartsWithUppercase;
+    }
+
+
     enum {
       signalBrightnessChanged = 0x1
     };
@@ -62,6 +72,7 @@ class Test13 : public KConfigSkeleton
     // kamoso
     QUrl mPicturesDir;
     double mBrightness;
+    bool mStartsWithUppercase;
 
   private:
 };
diff --git a/autotests/kconfig_compiler/test13.kcfg b/autotests/kconfig_compiler/test13.kcfg
index c4d36350..cf78c70e 100644
--- a/autotests/kconfig_compiler/test13.kcfg
+++ b/autotests/kconfig_compiler/test13.kcfg
@@ -7,5 +7,6 @@
   <group name="kamoso">
     <entry name="picturesDir" type="Url" />
     <entry name="brightness" type="double" />
+    <entry name="StartsWithUppercase" type="bool" />
   </group>
 </kcfg>
-- 
cgit v1.2.1