Fix rule based labeling does not permit moving labels using interactive tool

nyalldawson · nyalldawson · commit 0e2ceb90fe3c · 2018-10-01T16:07:40.000+10:00
Fixes #19979
diff --git a/src/core/qgsrulebasedlabeling.cpp b/src/core/qgsrulebasedlabeling.cpp
@@ -55,14 +55,12 @@ QList<QgsAbstractLabelProvider *> QgsRuleBasedLabelProvider::subProviders()
 ////////////////////
 
 QgsRuleBasedLabeling::Rule::Rule( QgsPalLayerSettings *settings, int scaleMinDenom, int scaleMaxDenom, const QString &filterExp, const QString &description, bool elseRule )
-  : mParent( nullptr )
-  , mSettings( settings )
+  : mSettings( settings )
   , mMaximumScale( scaleMinDenom )
   , mMinimumScale( scaleMaxDenom )
   , mFilterExp( filterExp )
   , mDescription( description )
   , mElseRule( elseRule )
-  , mIsActive( true )
 
 {
   initFilter();
diff --git a/src/core/qgsrulebasedlabeling.h b/src/core/qgsrulebasedlabeling.h
@@ -324,17 +324,18 @@ class CORE_EXPORT QgsRuleBasedLabeling : public QgsAbstractVectorLayerLabeling
         void updateElseRules();
 
       private:
-        Rule *mParent; // parent rule (NULL only for root rule)
+        Rule *mParent = nullptr; // parent rule (NULL only for root rule)
         std::unique_ptr<QgsPalLayerSettings> mSettings;
         double mMaximumScale = 0;
         double mMinimumScale = 0;
-        QString mFilterExp, mDescription;
-        bool mElseRule;
+        QString mFilterExp;
+        QString mDescription;
+        bool mElseRule = false;
         RuleList mChildren;
         RuleList mElseRules;
-        bool mIsActive; // whether it is enabled or not
+        bool mIsActive = true; // whether it is enabled or not
 
-        QString mRuleKey; // string used for unique identification of rule within labeling
+        QString mRuleKey = QUuid::createUuid().toString(); // string used for unique identification of rule within labeling
 
         std::unique_ptr<QgsExpression> mFilter;
 
