Bug fix: M261.x always required a V parameter

Duet3D · Sep 22, 2024 · f556068 · f556068
1 parent c264985
commit f556068
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/Platform/Platform.cpp b/src/Platform/Platform.cpp
@@ -2574,14 +2574,14 @@ GCodeResult Platform::ReceiveI2cOrModbus(GCodeBuffer& gb, const StringRef &reply
 	String<MaxVariableNameLength> varName;
 	bool seenV = false;
 	gb.TryGetQuotedString('V', varName.GetRef(), seenV, false);
-	if (!Variable::IsValidVariableName(varName.c_str()))
-	{
-		reply.printf("variable '%s' is not a valid name", varName.c_str());
-		return GCodeResult::error;
-	}
 	Variable *_ecv_null resultVar = nullptr;
 	if (seenV)
 	{
+		if (!Variable::IsValidVariableName(varName.c_str()))
+		{
+			reply.printf("variable '%s' is not a valid name", varName.c_str());
+			return GCodeResult::error;
+		}
 		auto vset = WriteLockedPointer<VariableSet>(nullptr, &gb.GetVariables());
 		Variable *_ecv_null const v = vset->Lookup(varName.c_str(), false);
 		if (v != nullptr)