improved 4.17 compatibility

Author: Roberto De Ioris

Source/PythonConsole/PythonConsole.Build.cs

@@ -5,10 +5,12 @@
 
 public class PythonConsole : ModuleRules
 {
-
-
+#if WITH_FORWARDED_MODULE_RULES_CTOR
+    public PythonConsole(ReadOnlyTargetRules Target) : base(Target)
+#else
     public PythonConsole(TargetInfo Target)
-	{
+#endif
+    {
         PrivateIncludePaths.AddRange(
             new string[] {
                 "PythonConsole/Private",
@@ -17,19 +19,19 @@ public PythonConsole(TargetInfo Target)
         );
 
         PrivateDependencyModuleNames.AddRange(
-			new string[] {
-				"Core",
-				"CoreUObject", // @todo Mac: for some reason it's needed to link in debug on Mac
+            new string[] {
+                "Core",
+                "CoreUObject", // @todo Mac: for some reason it's needed to link in debug on Mac
 				"Engine",
                 "InputCore",
-				"UnrealEd",
-				"Slate",
-				"SlateCore",
+                "UnrealEd",
+                "Slate",
+                "SlateCore",
                 "EditorStyle",
                 "TargetPlatform",
                 "UnrealEnginePython"
             }
-		);
+        );
 
     }
 }

Source/PythonEditor/PythonEditor.Build.cs

@@ -2,35 +2,39 @@
 
 namespace UnrealBuildTool.Rules
 {
-	public class PythonEditor : ModuleRules
-	{
-		public PythonEditor(TargetInfo Target)
-		{
-			PrivateIncludePaths.AddRange(
-				new string[] {
-					"PythonEditor/Private",
-				}
-				);
+    public class PythonEditor : ModuleRules
+    {
+#if WITH_FORWARDED_MODULE_RULES_CTOR
+        public PythonEditor(ReadOnlyTargetRules Target) : base(Target)
+#else
+        public PythonEditor(TargetInfo Target)
+#endif
+        {
+            PrivateIncludePaths.AddRange(
+                new string[] {
+                    "PythonEditor/Private",
+                }
+                );
 
-			PrivateDependencyModuleNames.AddRange(
-				new string[]
-				{
-					"Core",
-					"CoreUObject",
-					"SlateCore",
-					"Slate",
-					"AssetTools",
-					"UnrealEd",
-					"EditorStyle",
-					"PropertyEditor",
-					"Kismet",  // for FWorkflowCentricApplication
+            PrivateDependencyModuleNames.AddRange(
+                new string[]
+                {
+                    "Core",
+                    "CoreUObject",
+                    "SlateCore",
+                    "Slate",
+                    "AssetTools",
+                    "UnrealEd",
+                    "EditorStyle",
+                    "PropertyEditor",
+                    "Kismet",  // for FWorkflowCentricApplication
 					"InputCore",
-					"DirectoryWatcher",
-					"LevelEditor",
+                    "DirectoryWatcher",
+                    "LevelEditor",
                     "Projects",
                     "UnrealEnginePython"
                 }
-				);
-		}
-	}
+                );
+        }
+    }
 }

Source/UnrealEnginePython/Private/UObject/UEPyActor.cpp

@@ -106,8 +106,14 @@ PyObject *py_ue_actor_destroy_component(ue_PyUObject * self, PyObject * args) {
 	}
 
 	UActorComponent *component = ue_py_check_type<UActorComponent>(py_component);
+	if (!component)
+		return PyErr_Format(PyExc_Exception, "argument is not a UActorComponent");
 
+#if ENGINE_MINOR_VERSION >= 17
+	component->DestroyComponent();
+#else
 	actor->K2_DestroyComponent(component);
+#endif
 
 	Py_INCREF(Py_None);
 	return Py_None;

Source/UnrealEnginePython/UnrealEnginePython.Build.cs

@@ -70,7 +70,11 @@ public class UnrealEnginePython : ModuleRules
 	    "/usr/lib/x86_64-linux-gnu/libpython2.7.so",
     };
 
+#if WITH_FORWARDED_MODULE_RULES_CTOR
+    public UnrealEnginePython(ReadOnlyTargetRules Target) : base(Target)
+#else
     public UnrealEnginePython(TargetInfo Target)
+#endif
     {
 
         PublicIncludePaths.AddRange(