Fix class_desc_deselect handling of @GlobalScope.X links

Fix bug where links to class enums and class constants of the form @GlobalScope.X were broken.

For an enum or constant with name "example_name", links of both forms @GlobalScope.example_name and example_name will now be correctly handled, including where example_name contains "." (e.g. Variant.Type).

1 files changed, 26 insertions, 21 deletions

diff --git a/editor/editor_help.cpp b/editor/editor_help.cpp
index 77c4fc71a4..dba0864fcb 100644
--- a/editor/editor_help.cpp
+++ b/editor/editor_help.cpp
@@ -127,30 +127,35 @@ void EditorHelp::_class_desc_select(const String &p_select) {
 		if (table->has(link)) {
 			// Found in the current page.
 			class_desc->scroll_to_paragraph((*table)[link]);
-		} else if (topic == "class_enum") {
-			// Try to find the enum in @GlobalScope.
-			const DocData::ClassDoc &cd = doc->class_list["@GlobalScope"];
-
-			for (int i = 0; i < cd.constants.size(); i++) {
-				if (cd.constants[i].enumeration == link) {
-					// Found in @GlobalScope.
-					emit_signal(SNAME("go_to_help"), topic + ":@GlobalScope:" + link);
-					break;
+		} else {
+			// Look for link in @GlobalScope.
+			// Note that a link like @GlobalScope.enum_name will not be found in this section, only enum_name will be.
+			if (topic == "class_enum") {
+				const DocData::ClassDoc &cd = doc->class_list["@GlobalScope"];
+
+				for (int i = 0; i < cd.constants.size(); i++) {
+					if (cd.constants[i].enumeration == link) {
+						// Found in @GlobalScope.
+						emit_signal(SNAME("go_to_help"), topic + ":@GlobalScope:" + link);
+						return;
+					}
 				}
-			}
-		} else if (topic == "class_constant") {
-			// Try to find the constant in @GlobalScope.
-			const DocData::ClassDoc &cd = doc->class_list["@GlobalScope"];
-
-			for (int i = 0; i < cd.constants.size(); i++) {
-				if (cd.constants[i].name == link) {
-					// Found in @GlobalScope.
-					emit_signal(SNAME("go_to_help"), topic + ":@GlobalScope:" + link);
-					break;
+			} else if (topic == "class_constant") {
+				const DocData::ClassDoc &cd = doc->class_list["@GlobalScope"];
+
+				for (int i = 0; i < cd.constants.size(); i++) {
+					if (cd.constants[i].name == link) {
+						// Found in @GlobalScope.
+						emit_signal(SNAME("go_to_help"), topic + ":@GlobalScope:" + link);
+						return;
+					}
 				}
 			}
-		} else if (link.contains(".")) {
-			emit_signal(SNAME("go_to_help"), topic + ":" + link.get_slice(".", 0) + ":" + link.get_slice(".", 1));
+
+			if (link.contains(".")) {
+				int class_end = link.find(".");
+				emit_signal(SNAME("go_to_help"), topic + ":" + link.substr(0, class_end) + ":" + link.substr(class_end + 1, link.length()));
+			}
 		}
 	} else if (p_select.begins_with("http")) {
 		OS::get_singleton()->shell_open(p_select);