Skip to content

Commit 85428e3

Browse files
author
Vitaliy Boyko
committedApr 9, 2020
Code review fixes 2
1 parent 65a9262 commit 85428e3

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed
 

‎src/com/magento/idea/magento2plugin/project/ConfigurableProvider.java

-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ public String getName() {
2222

2323
@NotNull
2424
public PhpFrameworkConfigurable createConfigurable(@NotNull Project project) {
25-
2625
return new SettingsForm(project);
2726
}
2827
}

‎src/com/magento/idea/magento2plugin/project/UsagesProvider.java

-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ public String getName() {
1818
}
1919

2020
public boolean isEnabled(@NotNull Project project) {
21-
2221
return Settings.getInstance(project).pluginEnabled;
2322
}
2423
}

‎src/com/magento/idea/magento2plugin/util/magento/MagentoBasePathUtil.java

+6-8
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,12 @@ public class MagentoBasePathUtil {
1616
public static boolean isMagentoFolderValid(String path) {
1717
if (StringUtil.isEmptyOrSpaces(path)) {
1818
return false;
19-
} else {
20-
VirtualFile file = LocalFileSystem.getInstance().findFileByPath(path);
21-
if (file != null && file.isDirectory()) {
22-
return VfsUtil.findRelativeFile(file, Package.APP, Package.MODULE_BASE_AREA_DIR, ModuleDiXml.FILE_NAME) != null &&
23-
VfsUtil.findRelativeFile(file, new String[]{Package.VENDOR}) != null;
24-
} else {
25-
return false;
26-
}
2719
}
20+
VirtualFile file = LocalFileSystem.getInstance().findFileByPath(path);
21+
if (file != null && file.isDirectory()) {
22+
return VfsUtil.findRelativeFile(file, Package.APP, Package.MODULE_BASE_AREA_DIR, ModuleDiXml.FILE_NAME) != null &&
23+
VfsUtil.findRelativeFile(file, new String[]{Package.VENDOR}) != null;
24+
}
25+
return false;
2826
}
2927
}

0 commit comments

Comments
 (0)