Merge pull request #1018 from Iamwade/1000-fixed-bug-report

bohdan-harniuk · web-flow · commit 49be8d437c04 · 2022-02-23T20:13:30.000+02:00
1000: Fixed ArrayIndexOutOfBoundsException: Index 5 out of bounds for length 5(GetModuleNameByDirectoryUtil.java:47)
diff --git a/src/com/magento/idea/magento2plugin/util/magento/GetModuleNameByDirectoryUtil.java b/src/com/magento/idea/magento2plugin/util/magento/GetModuleNameByDirectoryUtil.java
@@ -44,7 +44,11 @@ private GetModuleNameByDirectoryUtil() {}
             final String moduleNamePath =  matcher.group(0);
 
             if (!moduleNamePath.isEmpty()) {
-                return moduleNamePath.split("/")[5];
+                final String[] moduleNamePathParts = moduleNamePath.split("/");
+
+                if (moduleNamePathParts.length >= 6) { //NOPMD
+                    return moduleNamePath.split("/")[5];
+                }
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,11 @@ private GetModuleNameByDirectoryUtil() {}`
`44`	`44`	`final String moduleNamePath = matcher.group(0);`
`45`	`45`
`46`	`46`	`if (!moduleNamePath.isEmpty()) {`
`47`		`- return moduleNamePath.split("/")[5];`
	`47`	`+ final String[] moduleNamePathParts = moduleNamePath.split("/");`
	`48`	`+`
	`49`	`+ if (moduleNamePathParts.length >= 6) { //NOPMD`
	`50`	`+ return moduleNamePath.split("/")[5];`
	`51`	`+ }`
`48`	`52`	`}`
`49`	`53`	`}`
`50`	`54`