fix last bug and make code prettier

jasp-stats · Oct 10, 2024 · 4c71cd9 · 4c71cd9
1 parent a904529
commit 4c71cd9
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/Desktop/data/datasetpackage.cpp b/Desktop/data/datasetpackage.cpp
@@ -2540,7 +2540,7 @@ Column * DataSetPackage::requestComputedColumnCreation(const std::string & colum
 
 bool DataSetPackage::requestColumnCreation(const std::string & columnName, Analysis * analysis, columnType type)
 {
-	if(DataSetPackage::pkg()->isColumnNameFree(columnName))
+	if(!DataSetPackage::pkg()->isColumnNameFree(columnName))
 		return false;
 
 	createComputedColumn(columnName, type, computedColumnType::analysisNotComputed, analysis);

diff --git a/QMLComponents/boundcontrols/boundcontrolbase.cpp b/QMLComponents/boundcontrols/boundcontrolbase.cpp
@@ -83,16 +83,17 @@ void BoundControlBase::handleComputedColumn(const Json::Value& value)
 		std::string			newName  = value.asString(),
 							orgName  = orgValue.asString();
 
-		if (newName.empty() && !orgName.empty())
-			emit _control->requestComputedColumnDestruction(orgName);
-
-		else if (newName != orgName)
+		if (newName != orgName)
 		{
-			if (_isComputedColumn)	emit _control->requestComputedColumnCreation(newName);
-			else					emit _control->requestColumnCreation(newName, _columnType);
-
 			if (!orgName.empty())
 				emit _control->requestComputedColumnDestruction(orgName);
+
+			if(!newName.empty())
+			{
+				if (_isComputedColumn)	emit _control->requestComputedColumnCreation(newName);
+				else					emit _control->requestColumnCreation(newName, _columnType);
+			}
+
 		}
 	}
 }

diff --git a/QMLComponents/controls/textinputbase.cpp b/QMLComponents/controls/textinputbase.cpp
@@ -214,7 +214,8 @@ void TextInputBase::setUp()
 
 void TextInputBase::setDisplayValue()
 {
-	setProperty("displayValue", _value);
+	if(property("displayValue") != _value)
+		setProperty("displayValue", _value);
 }
 
 void TextInputBase::rScriptDoneHandler(const QString &result)