From 41abaad213510b45a2d0c3378cfdea2f07f61e16 Mon Sep 17 00:00:00 2001
From: Vladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Date: Wed, 13 Dec 2017 16:12:04 +0100
Subject: [PATCH] =?UTF-8?q?cleanup=20=E2=80=94=20extract=20configurationMo?=
 =?UTF-8?q?dule?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../execution/configurations/ModuleBasedConfiguration.java | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java b/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
index 68da24686f69..de7d3cf0ea96 100644
--- a/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
+++ b/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
@@ -127,10 +127,11 @@ public abstract class ModuleBasedConfiguration<ConfigurationModule extends RunCo
 
       // AbstractPythonRunConfiguration calls in the constructor and so, there is a chance that newly created configuration will have module, but old haven't
       // and so, on readExternal module will be lost
-      String moduleName = StringUtil.nullize(configuration.getConfigurationModule().getModuleName());
+      RunConfigurationModule configurationModule = configuration.getConfigurationModule();
+      String moduleName = StringUtil.nullize(configurationModule.getModuleName());
       configuration.readExternal(element);
-      if (moduleName != null && StringUtil.nullize(configuration.getConfigurationModule().getModuleName()) == null) {
-        configuration.getConfigurationModule().setModuleName(moduleName);
+      if (moduleName != null && StringUtil.nullize(configurationModule.getModuleName()) == null) {
+        configurationModule.setModuleName(moduleName);
       }
       return configuration;
     }