Fix another premature disposal

MDoerner · MDoerner · commit 2d0dfc253ee2 · 2018-11-26T23:51:41.000+01:00
diff --git a/Rubberduck.Core/UI/UnitTesting/Commands/AddTestModuleCommand.cs b/Rubberduck.Core/UI/UnitTesting/Commands/AddTestModuleCommand.cs
@@ -125,13 +125,13 @@ private string DeclarationFormatFor(string declarationFormat, string type, IUnit
 
         private IVBProject GetProject()
         {
-            using (var activeProject = _vbe.ActiveVBProject)
+            //No using because the wrapper gets returned potentially. 
+            var activeProject = _vbe.ActiveVBProject;
+            if (!activeProject.IsWrappingNullReference)
             {
-                if (!activeProject.IsWrappingNullReference)
-                {
-                    return activeProject;
-                }
+                return activeProject;
             }
+            activeProject.Dispose();
             
             using (var projects = _vbe.VBProjects)
             {

Original file line number	Diff line number	Diff line change
`@@ -125,13 +125,13 @@ private string DeclarationFormatFor(string declarationFormat, string type, IUnit`
`125`	`125`
`126`	`126`	`private IVBProject GetProject()`
`127`	`127`	`{`
`128`		`- using (var activeProject = _vbe.ActiveVBProject)`
	`128`	`+ //No using because the wrapper gets returned potentially.`
	`129`	`+ var activeProject = _vbe.ActiveVBProject;`
	`130`	`+ if (!activeProject.IsWrappingNullReference)`
`129`	`131`	`{`
`130`		`- if (!activeProject.IsWrappingNullReference)`
`131`		`- {`
`132`		`- return activeProject;`
`133`		`- }`
	`132`	`+ return activeProject;`
`134`	`133`	`}`
	`134`	`+ activeProject.Dispose();`
`135`	`135`
`136`	`136`	`using (var projects = _vbe.VBProjects)`
`137`	`137`	`{`