Fixed issue with CRefPath.Parse method not correctly parsing indexor …

…properties.
barry-jones · Nov 12, 2014 · 6151a75 · 6151a75
1 parent 42ca29b
commit 6151a75
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 21 deletions.
diff --git a/Source/TheBoxSoftware.API.LiveDocumenter/CRefPath.cs b/Source/TheBoxSoftware.API.LiveDocumenter/CRefPath.cs
@@ -51,22 +51,15 @@ private void Parse()
 			if (this.PathType != CRefTypes.Error)
             {
 				string[] items;
-				if (this.PathType != CRefTypes.Method)
+				int startParams = this.crefPath.IndexOf('(');
+				if (startParams == -1) 
                 {
 					items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1).Split('.');
 				}
-				else
+				else 
                 {
-					int startParams = this.crefPath.IndexOf('(');
-					if (startParams == -1) 
-                    {
-						items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1).Split('.');
-					}
-					else 
-                    {
-						items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1, this.crefPath.IndexOf('(') - 2).Split('.');
-						this.Parameters = this.crefPath.Substring(this.crefPath.IndexOf('('));
-					}
+					items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1, this.crefPath.IndexOf('(') - 2).Split('.');
+					this.Parameters = this.crefPath.Substring(this.crefPath.IndexOf('('));
 				}
 
 				switch (this.PathType)

diff --git a/Source/TheBoxSoftware.Reflection/Comments/CRefPath.cs b/Source/TheBoxSoftware.Reflection/Comments/CRefPath.cs
@@ -195,18 +195,13 @@ private void Parse()
 			if (this.PathType != CRefTypes.Error)
             {
 				string[] items;
-				if (this.PathType != CRefTypes.Method) {
+				int startParams = this.crefPath.IndexOf('(');
+				if (startParams == -1) {
 					items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1).Split('.');
 				}
 				else {
-					int startParams = this.crefPath.IndexOf('(');
-					if (startParams == -1) {
-						items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1).Split('.');
-					}
-					else {
-						items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1, this.crefPath.IndexOf('(') - 2).Split('.');
-						this.Parameters = this.crefPath.Substring(this.crefPath.IndexOf('('));
-					}
+					items = this.crefPath.Substring(this.crefPath.IndexOf(':') + 1, this.crefPath.IndexOf('(') - 2).Split('.');
+					this.Parameters = this.crefPath.Substring(this.crefPath.IndexOf('('));
 				}
 
 				switch (this.PathType) {